Re: [patch 6/6] serial8250: convert to the new platform device interface

From: Tony Lindgren
Date: Mon Jan 16 2006 - 21:34:52 EST


* Kumar Gala <galak@xxxxxxxxxxxxxxxxx> [060116 16:02]:
> On Mon, 16 Jan 2006, Russell King wrote:
>
> > On Mon, Jan 16, 2006 at 04:27:17PM -0600, Kumar Gala wrote:
> > > This patch is breaking arch/ppc & arch/powerpc usage of 8250.c. The
> > > issue appears to be with the order in which platform_driver_register
> > > () is called vs platform_device_add().
> > >
> > > arch/powerpc/kernel/legacy_serial.c registers an 8250 device on the
> > > platform bus before 8250_init() gets called.
> > >
> > > Changing the order of platform_driver_register() vs
> > > platform_device_add() fixes the issue. I'm still not sure what the
> > > correct solution to this is. Ideas? comments?
> >
> > Mea Culpa - should've spotted that - that patch is actually rather
> > broken. platform_driver_register() can't be moved from where it
> > initially was.
>
> This seems to fix my issue on arch/powerpc and arch/ppc, please push to
> Linus ASAP.

This patch fixes problems on omap too.

Tony
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/