Thread (8 messages) 8 messages, 3 authors, 2012-09-24

Re: [PATCH 1/2] serial: New serial driver SCCNXP

From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Date: 2012-09-05 20:27:07

On Sat, Aug 25, 2012 at 07:24:19PM +0400, Alexander Shiyan wrote:
quoted hunk ↗ jump to hunk
This driver is a replacement for a SC26XX driver with a lot of
improvements and new features.
The main differences from the SC26XX driver:
- Removed dependency on MIPS. Driver can be used on any platform.
- Added support for SCC2681, SCC2691, SCC2692, SC28L91, SC28L92,
  SC28L202, SCC68681 and SCC68692 ICs.
- Using devm_-related functions.
- Improved error handling of serial port, improved FIFO handling.
- Ability to load multiple instances of drivers.

To avoid the possibility of regression, driver SC26XX left in the
system to confirm the stability of the driver on platforms where
it is being used.

Signed-off-by: Alexander Shiyan <redacted>
Acked-by: Alan Cox <redacted>
---
 drivers/tty/serial/Kconfig           |   18 +
 drivers/tty/serial/Makefile          |    1 +
 drivers/tty/serial/sccnxp.c          |  985 ++++++++++++++++++++++++++++++++++
 include/linux/platform_data/sccnxp.h |   93 ++++
 4 files changed, 1097 insertions(+), 0 deletions(-)
 create mode 100644 drivers/tty/serial/sccnxp.c
 create mode 100644 include/linux/platform_data/sccnxp.h
diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
index 4720b4b..750671d 100644
--- a/drivers/tty/serial/Kconfig
+++ b/drivers/tty/serial/Kconfig
@@ -1104,6 +1104,24 @@ config SERIAL_SC26XX_CONSOLE
 	help
 	  Support for Console on SC2681/SC2692 serial ports.
 
+config SERIAL_SCCNXP
+	bool "SCCNXP serial port support"
+	depends on !SERIAL_SC26XX
+	select SERIAL_CORE
+	default n
+	help
+	  This selects support for an advanced UART from NXP (Philips).
+	  Supported ICs are SCC2681, SCC2691, SCC2692, SC28L91, SC28L92,
+	  SC28L202, SCC68681 and SCC68692.
+	  Positioned as a replacement for the driver SC26XX.
This driver can be built as a module, right?  If so, please change this
to a tristate variable, not a "bool".  I don't feel like always building
it into my kernels :)

Care to send a follow-on patch fixing this up?

thanks,

greg k-h
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help