On Wed, 05 Apr 2023, "Govindapillai, Vinod" <vinod.govindapillai@xxxxxxxxx> wrote: > On Wed, 2023-04-05 at 13:41 +0300, Jani Nikula wrote: >> There's not much point in a static work function having a kernel-doc >> comment. Just clean it up and make it a regular comment. >> >> This fixes the kernel-doc warnings: >> >> drivers/gpu/drm/i915/display/intel_tc.c:1370: warning: Function >> parameter or member 'work' not described in >> 'intel_tc_port_disconnect_phy_work' >> >> drivers/gpu/drm/i915/display/intel_tc.c:1370: warning: Excess function >> parameter 'dig_port' description in 'intel_tc_port_disconnect_phy_work' >> >> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> >> --- > > Thanks. > > Reviewed-by: Vinod Govindapillai <vinod.govindapillai@xxxxxxxxx> Thanks for the review, pushed to drm-intel-next. BR, Jani. > > >> drivers/gpu/drm/i915/display/intel_tc.c | 5 +---- >> 1 file changed, 1 insertion(+), 4 deletions(-) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_tc.c b/drivers/gpu/drm/i915/display/intel_tc.c >> index 2bb02d4e6859..3b60995e9dfb 100644 >> --- a/drivers/gpu/drm/i915/display/intel_tc.c >> +++ b/drivers/gpu/drm/i915/display/intel_tc.c >> @@ -1358,10 +1358,7 @@ void intel_tc_port_lock(struct intel_digital_port *dig_port) >> __intel_tc_port_lock(to_tc_port(dig_port), 1); >> } >> >> -/** >> - * intel_tc_port_disconnect_phy_work: disconnect TypeC PHY from display port >> - * @dig_port: digital port >> - * >> +/* >> * Disconnect the given digital port from its TypeC PHY (handing back the >> * control of the PHY to the TypeC subsystem). This will happen in a delayed >> * manner after each aux transactions and modeset disables. > -- Jani Nikula, Intel Open Source Graphics Center