[PATCH] [POWERPC] Move mpc52xx-psc uart driver to of_device from platform_device

Grant Likely grant.likely at secretlab.ca
Thu Nov 2 07:50:03 EST 2006


On 11/1/06, Nicolas DET <nd at bplan-gmbh.de> wrote:
> Grant Likely wrote:
> > As part of the transition to arch/powerpc, this patch moves the mpc5200 PSC
> > driver over to the OF platform bus infrastructure.
> >
> > This patch is not acceptable for mainline as-is because it breaks arch/ppc
> > support for the mpc52xx.  More rework is needed to allow it to compile for
> > either arch (or alternately, fork the driver)
> >
>
> It has been hard to apply on 2.6.19-rc4 but it seems to perform well.
> I add to add some entries in mpc52xx_uart_match[]

Good news, thanks for the feedback. I take it you are able to use the
serial port with this driver?

>
> In now looks like this:
> static struct of_device_id mpc52xx_uart_match[] = {
>         {
>                 .name = "serial",
>                 .compatible = "mpc52xx-psc",
>         },
>         {
>                 .name = "serial",
>                 .compatible = "mpc52xx-serial",
>         },
>         {
>                 .name = "serial",
>                 .compatible = "mpc5200-psc",
>         },
>         {
>                 .name = "serial",
>                 .compatible = "mpc5200-serial",
>         },
>         {},
> };

I'd rather not do this for the mainline code.  I'd rather define a
naming convention and all of us use it.  We're early enough in this
process that we can do that since there are only two major boards
(lite5200 & Efika) on track to be supported in mainline ATM.

Having 4 compatibility blocks to support 2 boards is over-engineering.  :)

>
> By the way, why not using mpc52xx_uart_match[] to also in
> mpc52xx_console_setup() to track down any devices?

Because I hadn't thought of that, and therefore have not decided if
that's a good idea!  :)  I'll take a look.

g.

-- 
Grant Likely, B.Sc. P.Eng.
Secret Lab Technologies Ltd.
grant.likely at secretlab.ca
(403) 399-0195



More information about the Linuxppc-embedded mailing list