[U-Boot] [PATCH V2] ppc4xx: Fix UART baudrate setup by FDT

[Stefan Roese <sr@denx.de>]

Hi Matthias,

On Tuesday 06 January 2009, Matthias Fuchs wrote:
> > >                   UART0: serial at ef600300 {
> > >                                 device_type = "serial";
> > >                                 compatible = "ns16550", "ibm,uart";
> > >                                 ...
> >
> > That could be done as well. Perhaps it's the "better" solution. You might
> > want to ask on the linuxppc-dev list if such a patch is welcome. If yes,
> > then we should go this way.
>
> Changing U-Boot to check for something different than ns16550 will break
> all board that have ns16550 in their dt.

Ah, your talking about compatibility problems with older dts files. 
Understood.

> The Linux kernel only checks for ns16550 compatible nodes under specific
> parents (see arch/powerpc/kernel/legacy_serial.c). For 4xx this is only
> "opb" and not even opb/ebc.
>
> So I think we should do it also this way. In this case we could keep the
> modification in U-Boots' cpu/ppc4xx/fdt.c.
>
> When you are with me I will try to prepare a patch.

Yes, this sounds like a good idea. Please go ahead.

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================