Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933402AbbLSSqm (ORCPT ); Sat, 19 Dec 2015 13:46:42 -0500 Received: from mail-lb0-f181.google.com ([209.85.217.181]:36765 "EHLO mail-lb0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933270AbbLSSqj (ORCPT ); Sat, 19 Dec 2015 13:46:39 -0500 Message-ID: <1450550792.15911.5.camel@gmail.com> Subject: Re: [PATCH v4 1/2] serial: rewrite pxa2xx-uart to use 8250_core From: Sergei Ianovich To: Robert Jarzmik Cc: linux-kernel@vger.kernel.org, Kevin Hilman , Arnd Bergmann , "moderated list:ARM PORT" , "open list:SERIAL DRIVERS" Date: Sat, 19 Dec 2015 21:46:32 +0300 In-Reply-To: <87y4cqfu2j.fsf@belgarion.home> References: <1387309071-22382-1-git-send-email-ynvich@gmail.com> <1449700088-28076-1-git-send-email-ynvich@gmail.com> <1449700088-28076-2-git-send-email-ynvich@gmail.com> <8737uyhaji.fsf@belgarion.home> <87y4cqfu2j.fsf@belgarion.home> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.18.2-1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1862 Lines: 46 On Sat, 2015-12-19 at 14:26 +0100, Robert Jarzmik wrote: > Robert Jarzmik writes: > > > Sergei Ianovich writes: > > > > > pxa2xx-uart was a separate uart platform driver. It was declaring > > > the same device names and numbers as 8250 driver. As a result, > > > it was impossible to use 8250 driver on PXA SoCs. > > > > > > Upon closer examination pxa2xx-uart turned out to be a clone of > > > 8250_core driver. > > > > > > Workaround for Erratum #19 according to Marvel(R) PXA270M > > > Processor > > > Specification Update (April 19, 2010) is dropped. 8250_core reads > > > from FIFO immediately after checking DR bit in LSR. > > > > > > Signed-off-by: Sergei Ianovich > > > Reviewed-by: Heikki Krogerus > > > Reviewed-by: James Cameron > > > Acked-by: Greg Kroah-Hartman > > Tested-by: Robert Jarzmik > > Shrunk the list, the following is rather mach-pxa specific. > > Actually there is a small glitch ... > Have a look at arch/arm/mach-pxa/viper.c, line 490: > #ifndef CONFIG_SERIAL_PXA > ... Thanks for spotting this. This is caused by a change in the latest version of the patch (SERIAL_8250_PXA instead of SERIAL_PXA). This change could be reverted. > But that can be handled in an subsequent patch to keep your acks and > reviews. > I will respin the patch. Please comment on the acks and reviews. They were made at an earlier version of the patch. That version no longer applies. Can the updated version carry on the flags? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/