Hi Wolfram, On Fri, 30 Oct 2009 20:44:12 +0100 Wolfram Sang wrote: > > static struct of_platform_driver mpc52xx_psc_spi_of_driver = { > .owner = THIS_MODULE, > - .name = "mpc52xx-psc-spi", > + .name = DRIVER_NAME, You no longer need to set either owner or name in the of_platform driver, just in the included struct driver (as is done below), so you could just remove the above two lines. > .match_table = mpc52xx_psc_spi_of_match, > .probe = mpc52xx_psc_spi_of_probe, > .remove = __exit_p(mpc52xx_psc_spi_of_remove), > .driver = { > - .name = "mpc52xx-psc-spi", > + .name = DRIVER_NAME, > .owner = THIS_MODULE, > }, > }; I am hoping that we can remove the owner and name fields from struct of_platform_driver sometime. -- Cheers, Stephen Rothwell sfr@canb.auug.org.au http://www.canb.auug.org.au/~sfr/