On Mon, Jul 25, 2011 at 10:28:30AM +0900, Boojin Kim wrote: > +static void s3c64xx_spi_dma_rxcb(void *data) > +{ > + struct s3c64xx_spi_driver_data *sdd > + = (struct s3c64xx_spi_driver_data *)data; You never need explicit casts from void * or to void *. Please remove these.