[PATCH v5] spi: Add PPC4xx SPI driver

Stephen Rothwell sfr at canb.auug.org.au
Sun Dec 28 08:36:55 EST 2008


Hi David, Stefan,

On Sat, 27 Dec 2008 11:32:25 -0800 David Brownell <david-b at pacbell.net> wrote:
>
> Also:
> 
> > +static struct of_platform_driver spi_ppc4xx_of_driver = {
> > +       .owner = THIS_MODULE,
> > +       .name = DRIVER_NAME,
> 
> I'd hope the PPC folk eliminate this duplication soonish.
> Those fields are obvious duplicates of the driver model
> fields...

We are on the path (have been for some time).  Only the driver model
versions should be filled in for new drivers.  For old drivers we copy
the values into the driver model fields if they are not already set.  We
intend to remove the of_platform_drivers versions of "name" and "owner"
soon.

> > +       .match_table = spi_ppc4xx_of_match,
> > +       .probe = spi_ppc4xx_of_probe,
> > +       .remove = __exit_p(spi_ppc4xx_of_remove),
> > +       .driver = {
> > +               .name = DRIVER_NAME,
> > +               .owner = THIS_MODULE,
> > +       },
> > +};
-- 
Cheers,
Stephen Rothwell                    sfr at canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20081228/a4807f6f/attachment.pgp>


More information about the Linuxppc-dev mailing list