From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Wed, 7 Jan 2009 16:37:28 +0100 Subject: [U-Boot] [PATCH V2] ppc4xx: Fix UART baudrate setup by FDT In-Reply-To: <200901062015.58355.matthias.fuchs@esd-electronics.com> References: <200812281835.40995.mfuchs@ma-fu.de> <200901060907.21508.sr@denx.de> <200901062015.58355.matthias.fuchs@esd-electronics.com> Message-ID: <200901071637.28700.sr@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.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 =====================================================================