Hi YueHaibing, Thank you for the patch. On Sat, Feb 16, 2019 at 07:34:59AM +0000, YueHaibing wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/gpu/drm/bridge/cdns-dsi.c: In function 'cdns_dsi_bridge_enable': > drivers/gpu/drm/bridge/cdns-dsi.c:986:6: warning: > variable 'bpp' set but not used [-Wunused-but-set-variable] > > It never used since introduction. > > Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/bridge/cdns-dsi.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/cdns-dsi.c b/drivers/gpu/drm/bridge/cdns-dsi.c > index 6166dca6be81..07ee1685fdfe 100644 > --- a/drivers/gpu/drm/bridge/cdns-dsi.c > +++ b/drivers/gpu/drm/bridge/cdns-dsi.c > @@ -785,13 +785,12 @@ static void cdns_dsi_bridge_enable(struct drm_bridge *bridge) > unsigned long tx_byte_period; > struct cdns_dsi_cfg dsi_cfg; > u32 tmp, reg_wakeup, div; > - int bpp, nlanes; > + int nlanes; > > if (WARN_ON(pm_runtime_get_sync(dsi->base.dev) < 0)) > return; > > mode = &bridge->encoder->crtc->state->adjusted_mode; > - bpp = mipi_dsi_pixel_format_to_bpp(output->dev->format); > nlanes = output->dev->lanes; > > WARN_ON_ONCE(cdns_dsi_check_conf(dsi, mode, &dsi_cfg, false)); -- Regards, Laurent Pinchart
