On Tue, Oct 08, 2019 at 09:06:11AM +0300, Dan Carpenter wrote: > On Tue, Oct 08, 2019 at 01:38:58PM +0800, zhengbin wrote: > > diff --git a/drivers/staging/sm750fb/ddk750_mode.c b/drivers/staging/sm750fb/ddk750_mode.c > > index 9722692..e0230f4 100644 > > --- a/drivers/staging/sm750fb/ddk750_mode.c > > +++ b/drivers/staging/sm750fb/ddk750_mode.c > > @@ -209,12 +209,11 @@ static int programModeRegisters(struct mode_parameter *pModeParam, > > int ddk750_setModeTiming(struct mode_parameter *parm, enum clock_type clock) > > { > > struct pll_value pll; > > - unsigned int uiActualPixelClk; > > > > pll.input_freq = DEFAULT_INPUT_CLOCK; > > pll.clock_type = clock; > > > > - uiActualPixelClk = sm750_calc_pll_value(parm->pixel_clock, &pll); > > + sm750_calc_pll_value(parm->pixel_clock, &pll); > > Get rid of this function as well. Oops. I'm wrong. Sorry for the noise. Leave the function. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel