Hi Axel Lin, On Thu, May 08, 2014 at 07:17:07PM +0800, Axel Lin wrote: > The dw_spi_cleanup() function was removed by commit c63f5da00845 > "spi: dw: Don't call kfree for memory allocated by devm_kzalloc". > > commit ec37e8e1f0e3 "spi: dw: migrate to generic queue infrastructure" added > dw_spi_cleanup() but never use it. So now I got below build warning: > > CC [M] drivers/spi/spi-dw.o > drivers/spi/spi-dw.c:609:13: warning: 'dw_spi_cleanup' defined but not used > [-Wunused-function] > > Signed-off-by: Axel Lin <[email protected]>
Acked-by: Baruch Siach <[email protected]> Thanks, baruch > --- > drivers/spi/spi-dw.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/drivers/spi/spi-dw.c b/drivers/spi/spi-dw.c > index 9965e1b..29f3314 100644 > --- a/drivers/spi/spi-dw.c > +++ b/drivers/spi/spi-dw.c > @@ -606,12 +606,6 @@ static int dw_spi_setup(struct spi_device *spi) > return 0; > } > > -static void dw_spi_cleanup(struct spi_device *spi) > -{ > - struct chip_data *chip = spi_get_ctldata(spi); > - kfree(chip); > -} > - > /* Restart the controller, disable all interrupts, clean rx fifo */ > static void spi_hw_init(struct dw_spi *dws) > { > -- > 1.8.3.2 -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - [email protected] - tel: +972.2.679.5364, http://www.tkos.co.il - -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
