From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761524Ab0HFPjX (ORCPT ); Fri, 6 Aug 2010 11:39:23 -0400 Received: from cantor2.suse.de ([195.135.220.15]:51753 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757394Ab0HFPjV (ORCPT ); Fri, 6 Aug 2010 11:39:21 -0400 Date: Fri, 6 Aug 2010 08:39:03 -0700 From: Greg KH To: Philippe Langlais Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux@arm.linux.org.uk, akpm@linux-foundation.org, STEricsson_nomadik_linux@list.st.com, etienne.carriere@stericsson.com, vincent.guittot@stericsson.com Subject: Re: [PATCH 5/5] U6715 16550A serial driver support Message-ID: <20100806153903.GA6255@suse.de> References: <1281011335-25761-1-git-send-email-philippe.langlais@stericsson.com> <1281011335-25761-6-git-send-email-philippe.langlais@stericsson.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1281011335-25761-6-git-send-email-philippe.langlais@stericsson.com> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Aug 05, 2010 at 02:28:55PM +0200, Philippe Langlais wrote: > UART Features extract from STEricsson U6715 data-sheet (arm926 SoC for mobile phone): > * Fully compatible with industry standard 16C550 and 16C450 from various > manufacturers > * RX and TX 64 byte FIFO reduces CPU interrupts > * Full double buffering > * Modem control signals include CTS, RTS, (and DSR, DTR on UART1 only) > * Automatic baud rate selection > * Manual or automatic RTS/CTS smart hardware flow control > * Programmable serial characteristics: > – Baud rate generation (50 to 3.25M baud) > – 5, 6, 7 or 8-bit characters > – Even, odd or no-parity bit generation and detection > – 1, 1.5 or 2 stop bit generation > * Independent control of transmit, receive, line status, data set interrupts and FIFOs > * Full status-reporting capabilities > * Separate DMA signaling for RX and TX > * Timed interrupt to spread receive interrupt on known duration > * DMA time-out interrupt to allow detection of end of reception > * Carkit pulse coding and decoding compliant with USB carkit control interface [40] > > In 16550A auto-configuration, if the fifo size is 64 then it's an U6 16550A port. > Add set_termios hook & export serial8250_do_set_termios, > this feature is used in our U6715 8250 platform serial driver, to avoid > hack in 8250.c set_termios. > In this driver the input uart clock frequency depends on baud rate and > is computed in our set_termios override function. > > Signed-off-by: Philippe Langlais Acked-by: Greg Kroah-Hartman But isn't this patch needed before the last one in this series in order to keep everything building properly? You might want to move it up to the first one. thanks, greg k-h