From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Mon, 04 Sep 2006 18:26:53 +0100 (BST) Received: from h155.mvista.com ([63.81.120.155]:36969 "EHLO imap.sh.mvista.com") by ftp.linux-mips.org with ESMTP id S20039235AbWIDR0t (ORCPT ); Mon, 4 Sep 2006 18:26:49 +0100 Received: from [192.168.1.248] (unknown [10.150.0.9]) by imap.sh.mvista.com (Postfix) with ESMTP id 58F773EBE; Mon, 4 Sep 2006 10:26:43 -0700 (PDT) Message-ID: <44FC625A.5050005@ru.mvista.com> Date: Mon, 04 Sep 2006 21:28:58 +0400 From: Sergei Shtylyov Organization: MontaVista Software Inc. User-Agent: Mozilla/5.0 (X11; U; Linux i686; rv:1.7.2) Gecko/20040803 X-Accept-Language: ru, en-us, en-gb MIME-Version: 1.0 To: Russell King Cc: Pantelis Antoniou , Linux-MIPS Subject: Re: [PATCH] AMD Alchemy: claim UART memory range References: <4432BF48.8030403@ru.mvista.com> <44F2E9F7.6030309@ru.mvista.com> <20060830080157.GA17632@flint.arm.linux.org.uk> In-Reply-To: <20060830080157.GA17632@flint.arm.linux.org.uk> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 12511 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: sshtylyov@ru.mvista.com Precedence: bulk X-list: linux-mips Hello. Russell King wrote: >>> BTW, can anybody enlighten me why 8250_au1x00.c came into being >>>at all? >>>Its only function seems to register the UART platform devices, the >>>thing >>>that is usually done in the board setup code, i. e. I'd rather have >>>it in arch/mips/au1000/common/platform.c (however, 8250.c should >>>have been able to filter out ports with UPIO_AU in case >>>CONFIG_SERIAL_8250_AU1X00 undefined)... >>Seemed like a good idea at the moment to follow the already existing >>convention. > Already existing convention is as per Sergei's mail actually - to have the > platform device registration in arch/*. The others which you thought were > convention there (accent, boca, fourport, hub6, mca) are all for add-in > cards and aren't architecture specific. > Hence, they can't live in arch/*. > So yes, 8250_au1x00.c breaks the established convention because it isn't > an add-in card. Thanks for clarification. Now another question to Pantelis: IIUC, the Alchemy UART platform devices have UPF_SKIP_TEST set because of the Alchemy docs claiming that UARTs other than UART3 don't have MCR/MSR and only UART3 does have the full set of the modem control/status lines? Were they indeed failing the loopback test for you? Asking because on DBAu1550 board all (enabled) UARTs do pass the loopback test if I get rid of this flag (however, Au1550 datasheet says MCR/MSR exists on all UARTs, just no modem pins exist on UART0, and only RTS-/CTS- pair on UART1 -- and the bits having no correspoding pins seem to be tied high internally). If I'm correct, the driver seems inconsistent in how it handles UART_BUG_NOMSR flag, only checking it when deciding whether to enable the modem status interrupts or not while actually it should have been checked in serial8250_set_mctrl() and check_modem_status() as well... It also looks like the driver doesn't use Alchemy UARTs to their full potential currently: UART3 has not only full set of modem lines, but also is capable of the auto flow control (UART1 on Au1550 also is). (Making use of these features howewer are complicated by the auto flow control being only available in the late steppings of Au1500 and UART3 modem pins being multiplexed with GPIO...) WBR, Sergei PS: CCing linux-mips to keep people here informed. :-)