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 <paul.burton@xxxxxxxxxx> >> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> >> Cc: Lars-Peter Clausen <lars@xxxxxxxxxx> >> Cc: linux-serial@xxxxxxxxxxxxxxx >> --- >> 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. I can't manage to reproduce the build error with all of them applied. Used x86_64_defconfig and enabled CONFIG_SERIAL_8250_JZ47XX=y Thanks, ZubairLK > > Sorry, I can't take it. > > greg k-h >