On Wed, May 09, 2012 at 03:34:50AM +0530, Thomas Abraham wrote: > + s3c64xx_spi0_set_platdata("s3c6410-spi", NULL, 0, 1); Shouldn't we just set the name in the struct platform_device rather than requiring the machine to pass it through by hand?