From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Kroah-Hartman Subject: Re: [PATCH 26/36] serial: 8250_jz47xx: support for Ingenic jz47xx UARTs Date: Mon, 2 Feb 2015 08:32:32 -0800 Message-ID: <20150202163232.GA7055@kroah.com> References: <1421620067-23933-1-git-send-email-paul.burton@imgtec.com> <1421620881-25136-1-git-send-email-paul.burton@imgtec.com> <20150130233105.GA10564@kroah.com> <54CFA5BC.2070103@imgtec.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <54CFA5BC.2070103@imgtec.com> Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-subscribe: List-owner: List-post: List-archive: To: Zubair Lutfullah Kakakhel Cc: Paul Burton , linux-mips@linux-mips.org, Lars-Peter Clausen , linux-serial@vger.kernel.org List-Id: linux-serial@vger.kernel.org On Mon, Feb 02, 2015 at 04:28:44PM +0000, Zubair Lutfullah Kakakhel wrote: > Hi Greg, > > On 30/01/15 23:31, Greg Kroah-Hartman wrote: > > On Sun, Jan 18, 2015 at 02:41:21PM -0800, Paul Burton wrote: > >> Introduce a driver suitable for use with the UARTs present in > >> Ingenic jz47xx series SoCs. These are described as being ns16550 > >> compatible but aren't quite - they require the setting of an extra bit > >> in the FCR register to enable the UART module. The serial_out > >> implementation is the same as that in arch/mips/jz4740/serial.c - which > >> will shortly be removed. > >> > >> Signed-off-by: Paul Burton > >> Cc: Greg Kroah-Hartman > >> Cc: Lars-Peter Clausen > >> Cc: linux-serial@vger.kernel.org > >> --- > >> drivers/tty/serial/8250/8250_jz47xx.c | 228 ++++++++++++++++++++++++++++++++++ > >> drivers/tty/serial/8250/Kconfig | 8 ++ > >> drivers/tty/serial/8250/Makefile | 1 + > >> 3 files changed, 237 insertions(+) > >> create mode 100644 drivers/tty/serial/8250/8250_jz47xx.c > > > > This patch blows up on x86 systems, breaking the build :( > > Ouch. > > Did you compile after all the patches applied? Or after every single patch. After every individual patch, like is required :) > I can't manage to reproduce the build error with all of them applied. Try it with just this one patch applied. thanks, greg k-h