From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Wed, 28 Oct 2009 20:52:18 +0100 (CET) Received: from smtp1.linux-foundation.org ([140.211.169.13]:49940 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by ftp.linux-mips.org with ESMTP id S1492882AbZJ1TwM (ORCPT ); Wed, 28 Oct 2009 20:52:12 +0100 Received: from imap1.linux-foundation.org (imap1.linux-foundation.org [140.211.169.55]) by smtp1.linux-foundation.org (8.14.2/8.13.5/Debian-3ubuntu1.1) with ESMTP id n9SJq3vs031307 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 28 Oct 2009 12:52:04 -0700 Received: from akpm.mtv.corp.google.com (localhost [127.0.0.1]) by imap1.linux-foundation.org (8.13.5.20060308/8.13.5/Debian-3ubuntu1.1) with SMTP id n9SJq3Cg025086; Wed, 28 Oct 2009 12:52:03 -0700 Date: Wed, 28 Oct 2009 12:52:03 -0700 From: Andrew Morton To: Manuel Lauss Cc: Ralf Baechle , Linux-MIPS , linux-serial@vger.kernel.org, Manuel Lauss Subject: Re: [PATCH 2/2] MIPS: Alchemy: UARTs are 16550A Message-Id: <20091028125203.c513883e.akpm@linux-foundation.org> In-Reply-To: References: <1256756954-29211-1-git-send-email-manuel.lauss@gmail.com> <1256756954-29211-2-git-send-email-manuel.lauss@gmail.com> <20091028122430.f7670ae2.akpm@linux-foundation.org> X-Mailer: Sylpheed 2.4.8 (GTK+ 2.12.9; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-MIMEDefang-Filter: lf$Revision: 1.188 $ X-Scanned-By: MIMEDefang 2.63 on 140.211.169.13 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: 24561 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: akpm@linux-foundation.org Precedence: bulk X-list: linux-mips On Wed, 28 Oct 2009 20:27:16 +0100 Manuel Lauss wrote: > On Wed, Oct 28, 2009 at 8:24 PM, Andrew Morton wrote: > > > On Wed, 28 Oct 2009 20:09:14 +0100 > > Manuel Lauss wrote: > > > > > UART autodetection breaks on the Au1300 but the IP blocks are > > > identical, at least in the datasheets. > > > > > > Pass uart type on to the 8250 driver via platform data, and move > > > the MSR quirk to another place sind autoconf() is now no longer > > > called on init. > > > > > > Signed-off-by: Manuel Lauss > > > --- > > > Tested on DB1200 and DB1300. > > > The mips parts apply on top of Ralf's mips-queue tree. > > > > > > arch/mips/alchemy/common/platform.c | 4 +++- > > > drivers/serial/8250.c | 13 +++++++------ > > > 2 files changed, 10 insertions(+), 7 deletions(-) > > > > > > diff --git a/arch/mips/alchemy/common/platform.c > > b/arch/mips/alchemy/common/platform.c > > > index 195e5b3..3be14b0 100644 > > > --- a/arch/mips/alchemy/common/platform.c > > > +++ b/arch/mips/alchemy/common/platform.c > > > @@ -26,7 +26,9 @@ > > > .irq = _irq, \ > > > .regshift = 2, \ > > > .iotype = UPIO_AU, \ > > > - .flags = UPF_SKIP_TEST | UPF_IOREMAP \ > > > + .flags = UPF_SKIP_TEST | UPF_IOREMAP | \ > > > + UPF_FIXED_TYPE, \ > > > + .type = PORT_16550A, \ > > > } > > > > The kernel which you patched differs from current mainline here. > > > I know, that's why I added "The mips parts apply on top of Ralf's > mips-queue tree" below > the patch description. If that's the case then Ralf's mips-queue tree isn't in linux-next :( > If it makes it easier to apply, I could split this one in a mips and in a > 8250 patch? That's a hard call without knowing what's going on in mipsworld. If these patches applied to current mainline we could do it all as one patch and, with suitable acks, slap it into 2.6.32. Are these fixes also appropriate to 2.6.31.x and earlier? If so, that's another reason to prepare the patches against current mainline and just trample over the mips devel queue.