On Fri, Jul 19, 2019 at 01:51:36AM +0000, YueHaibing wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/gpu/drm/i915/display/icl_dsi.c: In function > 'gen11_dsi_set_transcoder_timings': > drivers/gpu/drm/i915/display/icl_dsi.c:768:6: warning: > variable 'hfront_porch' set but not used [-Wunused-but-set-variable] > > It is never used and can be removed. > > Reported-by: Hulk Robot <hul...@huawei.com> > Signed-off-by: YueHaibing <yuehaib...@huawei.com>
Thanks. Applied to drm-intel-next-queued. > --- > drivers/gpu/drm/i915/display/icl_dsi.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/icl_dsi.c > b/drivers/gpu/drm/i915/display/icl_dsi.c > index 4d952accfaaa..a42348be0438 100644 > --- a/drivers/gpu/drm/i915/display/icl_dsi.c > +++ b/drivers/gpu/drm/i915/display/icl_dsi.c > @@ -763,7 +763,7 @@ gen11_dsi_set_transcoder_timings(struct intel_encoder > *encoder, > enum transcoder dsi_trans; > /* horizontal timings */ > u16 htotal, hactive, hsync_start, hsync_end, hsync_size; > - u16 hfront_porch, hback_porch; > + u16 hback_porch; > /* vertical timings */ > u16 vtotal, vactive, vsync_start, vsync_end, vsync_shift; > > @@ -772,8 +772,6 @@ gen11_dsi_set_transcoder_timings(struct intel_encoder > *encoder, > hsync_start = adjusted_mode->crtc_hsync_start; > hsync_end = adjusted_mode->crtc_hsync_end; > hsync_size = hsync_end - hsync_start; > - hfront_porch = (adjusted_mode->crtc_hsync_start - > - adjusted_mode->crtc_hdisplay); > hback_porch = (adjusted_mode->crtc_htotal - > adjusted_mode->crtc_hsync_end); > vactive = adjusted_mode->crtc_vdisplay; > > -- Ville Syrjälä Intel
