From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e23smtp01.au.ibm.com (E23SMTP01.au.ibm.com [202.81.18.162]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "e23smtp01.au.ibm.com", Issuer "Equifax" (verified OK)) by ozlabs.org (Postfix) with ESMTP id 3256CDE05A for ; Fri, 12 Oct 2007 12:36:30 +1000 (EST) Received: from d23relay03.au.ibm.com (d23relay03.au.ibm.com [202.81.18.234]) by e23smtp01.au.ibm.com (8.13.1/8.13.1) with ESMTP id l9C2acLk010687 for ; Fri, 12 Oct 2007 12:36:38 +1000 Received: from d23av02.au.ibm.com (d23av02.au.ibm.com [9.190.235.138]) by d23relay03.au.ibm.com (8.13.8/8.13.8/NCO v8.5) with ESMTP id l9C2aRmM4653308 for ; Fri, 12 Oct 2007 12:36:27 +1000 Received: from d23av02.au.ibm.com (loopback [127.0.0.1]) by d23av02.au.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id l9C2aRUE021258 for ; Fri, 12 Oct 2007 12:36:27 +1000 Date: Fri, 12 Oct 2007 12:31:49 +1000 From: David Gibson To: Josh Boyer Subject: Re: [PATCH] PowerPC: Fix find_legacy_serial_ports on OPB. Message-ID: <20071012023149.GA21056@localhost.localdomain> References: <20071011152609.GA10320@ru.mvista.com> <200710111750.41852.arnd@arndb.de> <1192118028.5534.123.camel@localhost.localdomain> <470E5CBC.9040604@ru.mvista.com> <1192127513.5534.127.camel@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1192127513.5534.127.camel@localhost.localdomain> Cc: linuxppc-dev@ozlabs.org, Arnd Bergmann List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, Oct 11, 2007 at 01:31:53PM -0500, Josh Boyer wrote: > On Thu, 2007-10-11 at 21:26 +0400, Valentine Barshak wrote: > > Josh Boyer wrote: > > > On Thu, 2007-10-11 at 17:50 +0200, Arnd Bergmann wrote: > > >> On Thursday 11 October 2007, Valentine Barshak wrote: > > >>> Currently find_legacy_serial_ports() can find no serial ports on the OPB. > > >>> Thus no legacy boot console can be initialized. Just the early udbg console > > >>> works, which is initialized with udbg_init_44x_as1() on the UART's physical > > >>> address specified in kernel config. This happens because we look for ns16750 > > >>> and higher serial devices only and expect opb node to have a device type > > >>> property. This patch makes it look for ns16550 compatible devices and use > > >>> of_device_is_compatible() for opb instead of checking device type. > > >>> Lack of legacy serial ports found causes problems for KGDB over serial. > > >>> > > >>> Signed-off-by: Valentine Barshak > > >> The patch would make sense if we were only dealing with flattened device > > >> tree systems at this point. Unfortunately, IBM is shipping hardware that > > >> encodes the serial port in exactly the way that find_legacy_serial_ports > > >> is looking for (parent->type == "opb", compatible = "ns16750" "ns16550" > > >> "ns16450" i8250"). > > >> > > >> Changing the search for ns16750 to ns16550 should be fine, but unnecessary > > >> because AFAIK, all OPB serial imlpementations are actually ns16750 and > > >> should have that in the device tree as well. > > >> > > >> For the device type of the bus, please check for both compatible and > > >> type, so that it still works on machines that are missing the compatible > > >> property. > > > > > > Wait, no. We already had this discussion months ago when David was > > > working on the original Ebony port. It was declared that legacy_serial > > > is not how serial should be done on 4xx and the serial_of driver was > > > supposed to be used instead. > > > > > > Have we changed our stance on that? If not, then perhaps KGDB should be > > > fixed to work with serial_of. > > > > Actually I don't see any reason not to use legacy_serial stuff for early > > console. We could split the kernel configured very early debug output, > > which uses PPC_EARLY_DEBUG_44x_PHYSLOW/PHYSHIGH (since it's really > > dangerous) and early console things by using legacy serial. We could use > > early boot console without PPC_EARLY_DEBUG_44x. > > That was exactly my thinking when this first came up. I'd like to hear > David's opinion on it. Yeah, I think I misinterpreted BenH way back when. This looks ok, and means serial will be initialized earlier than of_serial, which would be nice. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson