On Fri, 02 Sep 2022, cgel....@gmail.com wrote: > From: zhang songyi <zhang.son...@zte.com.cn> > > Return the drm_dp_dpcd_writeb() directly instead of storing it in another > redundant variable.
Please just *stop* sending changes like this. See for example [1]. What's most offending to me is that I've replied to a lot of patches from you, but I've *never* *even* *once* received a reply back on my reviews. If you want to participate in creating a better kernel, then please *participate*. Just throwing random cleanup patches in our general direction with one-way communication is not helping. I'm seriously considering just ignoring *all* patches from you. BR, Jani. [1] https://lore.kernel.org/r/8735dcepcv....@intel.com > > Reported-by: Zeal Robot <zea...@zte.com.cn> > Signed-off-by: zhang songyi <zhang.son...@zte.com.cn> > --- > drivers/gpu/drm/display/drm_dp_helper.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/display/drm_dp_helper.c > b/drivers/gpu/drm/display/drm_dp_helper.c > index 92990a3d577a..09b282b704fa 100644 > --- a/drivers/gpu/drm/display/drm_dp_helper.c > +++ b/drivers/gpu/drm/display/drm_dp_helper.c > @@ -2925,16 +2925,13 @@ EXPORT_SYMBOL(drm_dp_get_pcon_max_frl_bw); > */ > int drm_dp_pcon_frl_prepare(struct drm_dp_aux *aux, bool > enable_frl_ready_hpd) > { > - int ret; > u8 buf = DP_PCON_ENABLE_SOURCE_CTL_MODE | > DP_PCON_ENABLE_LINK_FRL_MODE; > > if (enable_frl_ready_hpd) > buf |= DP_PCON_ENABLE_HPD_READY; > > - ret = drm_dp_dpcd_writeb(aux, DP_PCON_HDMI_LINK_CONFIG_1, buf); > - > - return ret; > + return drm_dp_dpcd_writeb(aux, DP_PCON_HDMI_LINK_CONFIG_1, buf); > } > EXPORT_SYMBOL(drm_dp_pcon_frl_prepare); -- Jani Nikula, Intel Open Source Graphics Center
