[PATCHv2] ARM: sa1100: refactor sa1100 serial driver
Jochen Friedrich
jochen at scram.de
Tue Nov 22 07:55:18 EST 2011
- move pin initialization into board files
- make use of resources in platform device
rather than hardcoding them in the driver
- add DT support
Signed-off-by: Jochen Friedrich <jochen at scram.de>
---
V2: fixed off-by-one error
.../devicetree/bindings/tty/serial/sa1100-uart.txt | 20 ++
.../devicetree/bindings/vendor-prefixes.txt | 1 +
arch/arm/include/asm/mach/serial_sa1100.h | 5 +-
arch/arm/mach-sa1100/badge4.c | 3 +
arch/arm/mach-sa1100/cerf.c | 3 +
arch/arm/mach-sa1100/collie.c | 2 +
arch/arm/mach-sa1100/generic.c | 85 +++++++-
arch/arm/mach-sa1100/generic.h | 2 +
arch/arm/mach-sa1100/h3xxx.c | 3 +-
arch/arm/mach-sa1100/hackkit.c | 3 +
arch/arm/mach-sa1100/jornada720.c | 3 +
arch/arm/mach-sa1100/lart.c | 3 +
arch/arm/mach-sa1100/pleb.c | 3 +
arch/arm/mach-sa1100/shannon.c | 3 +
arch/arm/mach-sa1100/simpad.c | 5 +-
drivers/tty/serial/sa1100.c | 240 +++++++++++---------
16 files changed, 261 insertions(+), 123 deletions(-)
create mode 100644 Documentation/devicetree/bindings/tty/serial/sa1100-uart.txt
diff --git a/Documentation/devicetree/bindings/tty/serial/sa1100-uart.txt b/Documentation/devicetree/bindings/tty/serial/sa1100-uart.txt
new file mode 100644
index 0000000..3dc76ca
--- /dev/null
+++ b/Documentation/devicetree/bindings/tty/serial/sa1100-uart.txt
@@ -0,0 +1,20 @@
+* SA11x0 Universal Asynchronous Receiver/Transmitter (UART)
+
+Required properties:
+- compatible: Should be "intel,<chip>-uart"
+ The compatible <chip> indicated will be the first SoC to support an
+ additional mode or an UART new feature.
+- reg: Should contain registers location and length
+- interrupts: Should contain interrupt
+
+<chip> compatible description:
+- sa1100: generic UART implementation for SA11x0 SoCs
+
+Example:
+
+ uart0: serial at 0x80010000 {
+ compatible = "intel,sa1100-uart";
+ reg = <0x80010000 0x24>;
+ interrupts = <15>;
+ };
+
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index e855278..5b14518 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -19,6 +19,7 @@ gef GE Fanuc Intelligent Platforms Embedded Systems, Inc.
hp Hewlett Packard
ibm International Business Machines (IBM)
idt Integrated Device Technologies, Inc.
+intel Intel
intercontrol Inter Control Group
linux Linux-specific binding
marvell Marvell Technology Group Ltd.
diff --git a/arch/arm/include/asm/mach/serial_sa1100.h b/arch/arm/include/asm/mach/serial_sa1100.h
index d09064b..8305ab5 100644
--- a/arch/arm/include/asm/mach/serial_sa1100.h
+++ b/arch/arm/include/asm/mach/serial_sa1100.h
@@ -9,7 +9,6 @@
*/
struct uart_port;
-struct uart_info;
/*
* This is a temporary structure for registering these
@@ -22,10 +21,10 @@ struct sa1100_port_fns {
int (*set_wake)(struct uart_port *, u_int);
};
+struct platform_device *sa1100_get_uart(int id);
+
#ifdef CONFIG_SERIAL_SA1100
void sa1100_register_uart_fns(struct sa1100_port_fns *fns);
-void sa1100_register_uart(int idx, int port);
#else
#define sa1100_register_uart_fns(fns) do { } while (0)
-#define sa1100_register_uart(idx,port) do { } while (0)
#endif
diff --git a/arch/arm/mach-sa1100/badge4.c b/arch/arm/mach-sa1100/badge4.c
index bda83e1..505645e 100644
--- a/arch/arm/mach-sa1100/badge4.c
+++ b/arch/arm/mach-sa1100/badge4.c
@@ -296,6 +296,9 @@ static void __init badge4_map_io(void)
sa1100_map_io();
iotable_init(badge4_io_desc, ARRAY_SIZE(badge4_io_desc));
+ PPDR |= PPC_TXD1 | PPC_TXD3;
+ PPSR |= PPC_TXD1 | PPC_TXD3;
+
sa1100_register_uart_fns(&badge4_port_fns);
sa1100_register_uart(0, 3);
sa1100_register_uart(1, 1);
diff --git a/arch/arm/mach-sa1100/cerf.c b/arch/arm/mach-sa1100/cerf.c
index 7f3da4b..a626790 100644
--- a/arch/arm/mach-sa1100/cerf.c
+++ b/arch/arm/mach-sa1100/cerf.c
@@ -113,6 +113,9 @@ static void __init cerf_map_io(void)
sa1100_map_io();
iotable_init(cerf_io_desc, ARRAY_SIZE(cerf_io_desc));
+ PPDR |= PPC_TXD1 | PPC_TXD2 | PPC_TXD3;
+ PPSR |= PPC_TXD1 | PPC_TXD2 | PPC_TXD3;
+
sa1100_register_uart(0, 3);
sa1100_register_uart(1, 2); /* disable this and the uart2 device for sa1100_fir */
sa1100_register_uart(2, 1);
diff --git a/arch/arm/mach-sa1100/collie.c b/arch/arm/mach-sa1100/collie.c
index 2965cc9..7a17703 100644
--- a/arch/arm/mach-sa1100/collie.c
+++ b/arch/arm/mach-sa1100/collie.c
@@ -330,6 +330,8 @@ static void __init collie_init(void)
PPC_LDD6 | PPC_LDD7 | PPC_L_PCLK | PPC_L_LCLK | PPC_L_FCLK | PPC_L_BIAS |
PPC_TXD1 | PPC_TXD2 | PPC_TXD3 | PPC_TXD4 | PPC_SCLK | PPC_SFRM;
+ PPSR |= PPC_TXD1 | PPC_TXD3;
+
PWER = _COLLIE_GPIO_AC_IN | _COLLIE_GPIO_CO | _COLLIE_GPIO_ON_KEY |
_COLLIE_GPIO_WAKEUP | _COLLIE_GPIO_nREMOCON_INT | PWER_RTC;
diff --git a/arch/arm/mach-sa1100/generic.c b/arch/arm/mach-sa1100/generic.c
index 5fa5ae1..884ab63 100644
--- a/arch/arm/mach-sa1100/generic.c
+++ b/arch/arm/mach-sa1100/generic.c
@@ -166,7 +166,7 @@ static struct platform_device sa11x0udc_device = {
static struct resource sa11x0uart1_resources[] = {
[0] = {
.start = __PREG(Ser1UTCR0),
- .end = __PREG(Ser1UTCR0) + 0xffff,
+ .end = __PREG(Ser1UTCR0) + 0x24 - 1,
.flags = IORESOURCE_MEM,
},
[1] = {
@@ -178,15 +178,35 @@ static struct resource sa11x0uart1_resources[] = {
static struct platform_device sa11x0uart1_device = {
.name = "sa11x0-uart",
- .id = 1,
+ .id = -1,
.num_resources = ARRAY_SIZE(sa11x0uart1_resources),
.resource = sa11x0uart1_resources,
};
+static struct resource sa11x0uart2_resources[] = {
+ [0] = {
+ .start = __PREG(Ser2UTCR0),
+ .end = __PREG(Ser2UTCR0) + 0x24 - 1,
+ .flags = IORESOURCE_MEM,
+ },
+ [1] = {
+ .start = IRQ_Ser2ICP,
+ .end = IRQ_Ser2ICP,
+ .flags = IORESOURCE_IRQ,
+ },
+};
+
+static struct platform_device sa11x0uart2_device = {
+ .name = "sa11x0-uart",
+ .id = -1,
+ .num_resources = ARRAY_SIZE(sa11x0uart2_resources),
+ .resource = sa11x0uart2_resources,
+};
+
static struct resource sa11x0uart3_resources[] = {
[0] = {
.start = __PREG(Ser3UTCR0),
- .end = __PREG(Ser3UTCR0) + 0xffff,
+ .end = __PREG(Ser3UTCR0) + 0x24 - 1,
.flags = IORESOURCE_MEM,
},
[1] = {
@@ -198,7 +218,7 @@ static struct resource sa11x0uart3_resources[] = {
static struct platform_device sa11x0uart3_device = {
.name = "sa11x0-uart",
- .id = 3,
+ .id = -1,
.num_resources = ARRAY_SIZE(sa11x0uart3_resources),
.resource = sa11x0uart3_resources,
};
@@ -341,8 +361,6 @@ static struct platform_device sa11x0rtc_device = {
static struct platform_device *sa11x0_devices[] __initdata = {
&sa11x0udc_device,
- &sa11x0uart1_device,
- &sa11x0uart3_device,
&sa11x0ssp_device,
&sa11x0pcmcia_device,
&sa11x0fb_device,
@@ -351,12 +369,65 @@ static struct platform_device *sa11x0_devices[] __initdata = {
static int __init sa1100_init(void)
{
+ int ret = 0;
pm_power_off = sa1100_power_off;
- return platform_add_devices(sa11x0_devices, ARRAY_SIZE(sa11x0_devices));
+ if (sa11x0uart1_device.id >= 0)
+ ret = platform_device_register(&sa11x0uart1_device);
+ if (ret)
+ return ret;
+ if (sa11x0uart2_device.id >= 0)
+ ret = platform_device_register(&sa11x0uart2_device);
+ if (ret)
+ goto err_uart1;
+ if (sa11x0uart3_device.id >= 0)
+ ret = platform_device_register(&sa11x0uart3_device);
+ if (ret)
+ goto err_uart2;
+ ret = platform_add_devices(sa11x0_devices, ARRAY_SIZE(sa11x0_devices));
+ if (ret)
+ goto err_uart3;
+ return 0;
+
+err_uart3:
+ platform_device_unregister(&sa11x0uart3_device);
+err_uart2:
+ platform_device_unregister(&sa11x0uart2_device);
+err_uart1:
+ platform_device_unregister(&sa11x0uart1_device);
+ return ret;
}
arch_initcall(sa1100_init);
+void __init sa1100_register_uart(int id, int hw)
+{
+ switch (hw) {
+ case 1:
+ sa11x0uart1_device.id = id;
+ break;
+
+ case 2:
+ sa11x0uart2_device.id = id;
+ break;
+
+ case 3:
+ sa11x0uart3_device.id = id;
+ break;
+ }
+}
+
+struct platform_device *sa1100_get_uart(int id)
+{
+ if (sa11x0uart1_device.id == id)
+ return &sa11x0uart1_device;
+ if (sa11x0uart2_device.id == id)
+ return &sa11x0uart2_device;
+ if (sa11x0uart3_device.id == id)
+ return &sa11x0uart3_device;
+ return NULL;
+}
+EXPORT_SYMBOL(sa1100_get_uart);
+
void (*sa1100fb_backlight_power)(int on);
void (*sa1100fb_lcd_power)(int on);
diff --git a/arch/arm/mach-sa1100/generic.h b/arch/arm/mach-sa1100/generic.h
index b7a9a60..d5a51d0 100644
--- a/arch/arm/mach-sa1100/generic.h
+++ b/arch/arm/mach-sa1100/generic.h
@@ -39,3 +39,5 @@ void sa11x0_register_irda(struct irda_platform_data *irda);
struct mcp_plat_data;
void sa11x0_register_mcp(struct mcp_plat_data *data);
+
+extern void sa1100_register_uart(int hw, int id);
diff --git a/arch/arm/mach-sa1100/h3xxx.c b/arch/arm/mach-sa1100/h3xxx.c
index b0784c9..f3c3ba9 100644
--- a/arch/arm/mach-sa1100/h3xxx.c
+++ b/arch/arm/mach-sa1100/h3xxx.c
@@ -299,7 +299,8 @@ void __init h3xxx_map_io(void)
// sa1100_register_uart(1, 1); /* Microcontroller on 3100/3600 */
/* Ensure those pins are outputs and driving low */
- PPDR |= PPC_TXD4 | PPC_SCLK | PPC_SFRM;
+ PPDR |= PPC_TXD3 | PPC_TXD4 | PPC_SCLK | PPC_SFRM;
+ PPSR |= PPC_TXD3;
PPSR &= ~(PPC_TXD4 | PPC_SCLK | PPC_SFRM);
/* Configure suspend conditions */
diff --git a/arch/arm/mach-sa1100/hackkit.c b/arch/arm/mach-sa1100/hackkit.c
index 30f4a55..e852a87 100644
--- a/arch/arm/mach-sa1100/hackkit.c
+++ b/arch/arm/mach-sa1100/hackkit.c
@@ -80,6 +80,9 @@ static void __init hackkit_map_io(void)
sa1100_map_io();
iotable_init(hackkit_io_desc, ARRAY_SIZE(hackkit_io_desc));
+ PPDR |= PPC_TXD1 | PPC_TXD2 | PPC_TXD3;
+ PPSR |= PPC_TXD1 | PPC_TXD2 | PPC_TXD3;
+
sa1100_register_uart_fns(&hackkit_port_fns);
sa1100_register_uart(0, 1); /* com port */
sa1100_register_uart(1, 2);
diff --git a/arch/arm/mach-sa1100/jornada720.c b/arch/arm/mach-sa1100/jornada720.c
index 77198fe..d99e00a 100644
--- a/arch/arm/mach-sa1100/jornada720.c
+++ b/arch/arm/mach-sa1100/jornada720.c
@@ -297,6 +297,9 @@ static void __init jornada720_map_io(void)
sa1100_map_io();
iotable_init(jornada720_io_desc, ARRAY_SIZE(jornada720_io_desc));
+ PPDR |= PPC_TXD1 | PPC_TXD3;
+ PPSR |= PPC_TXD1 | PPC_TXD3;
+
sa1100_register_uart(0, 3);
sa1100_register_uart(1, 1);
}
diff --git a/arch/arm/mach-sa1100/lart.c b/arch/arm/mach-sa1100/lart.c
index 5bc59d0..a8fae5b 100644
--- a/arch/arm/mach-sa1100/lart.c
+++ b/arch/arm/mach-sa1100/lart.c
@@ -50,6 +50,9 @@ static void __init lart_map_io(void)
sa1100_map_io();
iotable_init(lart_io_desc, ARRAY_SIZE(lart_io_desc));
+ PPDR |= PPC_TXD1 | PPC_TXD2 | PPC_TXD3;
+ PPSR |= PPC_TXD1 | PPC_TXD2 | PPC_TXD3;
+
sa1100_register_uart(0, 3);
sa1100_register_uart(1, 1);
sa1100_register_uart(2, 2);
diff --git a/arch/arm/mach-sa1100/pleb.c b/arch/arm/mach-sa1100/pleb.c
index 65161f2..b8fced0 100644
--- a/arch/arm/mach-sa1100/pleb.c
+++ b/arch/arm/mach-sa1100/pleb.c
@@ -121,6 +121,9 @@ static void __init pleb_map_io(void)
{
sa1100_map_io();
+ PPDR |= PPC_TXD1 | PPC_TXD3;
+ PPSR |= PPC_TXD1 | PPC_TXD3;
+
sa1100_register_uart(0, 3);
sa1100_register_uart(1, 1);
diff --git a/arch/arm/mach-sa1100/shannon.c b/arch/arm/mach-sa1100/shannon.c
index 1cccbf5..84f5b14 100644
--- a/arch/arm/mach-sa1100/shannon.c
+++ b/arch/arm/mach-sa1100/shannon.c
@@ -67,6 +67,9 @@ static void __init shannon_map_io(void)
{
sa1100_map_io();
+ PPDR |= PPC_TXD1 | PPC_TXD3;
+ PPSR |= PPC_TXD1 | PPC_TXD3;
+
sa1100_register_uart(0, 3);
sa1100_register_uart(1, 1);
diff --git a/arch/arm/mach-sa1100/simpad.c b/arch/arm/mach-sa1100/simpad.c
index 4790f3f..d3c9d2b 100644
--- a/arch/arm/mach-sa1100/simpad.c
+++ b/arch/arm/mach-sa1100/simpad.c
@@ -206,7 +206,10 @@ static void __init simpad_map_io(void)
RS232_ON | ENABLE_5V | RESET_SIMCARD | DECT_POWER_ON);
__simpad_write_cs3(); /* Spinlocks not yet initialized */
- sa1100_register_uart_fns(&simpad_port_fns);
+ PPDR |= PPC_TXD1 | PPC_TXD3;
+ PPSR |= PPC_TXD1 | PPC_TXD3;
+
+ sa1100_register_uart_fns(&simpad_port_fns);
sa1100_register_uart(0, 3); /* serial interface */
sa1100_register_uart(1, 1); /* DECT */
diff --git a/drivers/tty/serial/sa1100.c b/drivers/tty/serial/sa1100.c
index ef7a21a..a084930 100644
--- a/drivers/tty/serial/sa1100.c
+++ b/drivers/tty/serial/sa1100.c
@@ -35,6 +35,7 @@
#include <linux/serial_core.h>
#include <linux/serial.h>
#include <linux/io.h>
+#include <linux/of.h>
#include <asm/irq.h>
#include <mach/hardware.h>
@@ -43,6 +44,7 @@
/* We've been assigned a range on the "Low-density serial ports" major */
#define SERIAL_SA1100_MAJOR 204
#define MINOR_START 5
+#define SA1100_DEVICENAME "ttySA"
#define NR_PORTS 3
@@ -71,11 +73,14 @@
#define UART_PUT_UTSR0(sport,v) __raw_writel((v),(sport)->port.membase + UTSR0)
#define UART_PUT_UTSR1(sport,v) __raw_writel((v),(sport)->port.membase + UTSR1)
#define UART_PUT_CHAR(sport,v) __raw_writel((v),(sport)->port.membase + UTDR)
+#if defined(CONFIG_OF)
+static const struct of_device_id sa1100_serial_dt_ids[] = {
+ { .compatible = "intel,sa1100-uart" },
+ { /* sentinel */ }
+};
-/*
- * This is the size of our serial port register set.
- */
-#define UART_PORT_SIZE 0x24
+MODULE_DEVICE_TABLE(of, sa1100_serial_dt_ids);
+#endif
/*
* This determines how often we check the modem status signals
@@ -89,6 +94,7 @@ struct sa1100_port {
struct uart_port port;
struct timer_list timer;
unsigned int old_status;
+ unsigned int size;
};
/*
@@ -521,7 +527,9 @@ static void sa1100_release_port(struct uart_port *port)
{
struct sa1100_port *sport = (struct sa1100_port *)port;
- release_mem_region(sport->port.mapbase, UART_PORT_SIZE);
+ release_mem_region(port->mapbase, sport->size);
+ iounmap(port->membase);
+ port->membase = NULL;
}
/*
@@ -531,8 +539,20 @@ static int sa1100_request_port(struct uart_port *port)
{
struct sa1100_port *sport = (struct sa1100_port *)port;
- return request_mem_region(sport->port.mapbase, UART_PORT_SIZE,
- "sa11x0-uart") != NULL ? 0 : -EBUSY;
+ if (!request_mem_region(port->mapbase, sport->size,
+ "sa11x0-uart")) {
+ dev_err(port->dev, "Memory region busy\n");
+ return -EBUSY;
+ }
+
+ if (!port->membase)
+ port->membase = ioremap(port->mapbase, sport->size);
+ if (!port->membase) {
+ dev_err(port->dev, "Unable to map registers\n");
+ release_mem_region(port->mapbase, sport->size);
+ return -EBUSY;
+ }
+ return 0;
}
/*
@@ -555,24 +575,21 @@ static void sa1100_config_port(struct uart_port *port, int flags)
static int
sa1100_verify_port(struct uart_port *port, struct serial_struct *ser)
{
- struct sa1100_port *sport = (struct sa1100_port *)port;
- int ret = 0;
-
- if (ser->type != PORT_UNKNOWN && ser->type != PORT_SA1100)
- ret = -EINVAL;
- if (sport->port.irq != ser->irq)
- ret = -EINVAL;
+ if (ser->type != PORT_SA1100)
+ return -EINVAL;
+ if (port->irq != ser->irq)
+ return -EINVAL;
if (ser->io_type != SERIAL_IO_MEM)
- ret = -EINVAL;
- if (sport->port.uartclk / 16 != ser->baud_base)
- ret = -EINVAL;
- if ((void *)sport->port.mapbase != ser->iomem_base)
- ret = -EINVAL;
- if (sport->port.iobase != ser->port)
- ret = -EINVAL;
+ return -EINVAL;
+ if (port->uartclk / 16 != ser->baud_base)
+ return -EINVAL;
+ if ((void *)port->mapbase != ser->iomem_base)
+ return -EINVAL;
+ if (port->iobase != ser->port)
+ return -EINVAL;
if (ser->hub6 != 0)
- ret = -EINVAL;
- return ret;
+ return -EINVAL;
+ return 0;
}
static struct uart_ops sa1100_pops = {
@@ -595,6 +612,7 @@ static struct uart_ops sa1100_pops = {
};
static struct sa1100_port sa1100_ports[NR_PORTS];
+static unsigned long sa1100_ports_in_use;
/*
* Setup the SA1100 serial ports. Note that we don't include the IrDA
@@ -608,33 +626,6 @@ static struct sa1100_port sa1100_ports[NR_PORTS];
* Note that NanoEngine UART3 becomes UART2, and UART2 is no longer
* used here.
*/
-static void __init sa1100_init_ports(void)
-{
- static int first = 1;
- int i;
-
- if (!first)
- return;
- first = 0;
-
- for (i = 0; i < NR_PORTS; i++) {
- sa1100_ports[i].port.uartclk = 3686400;
- sa1100_ports[i].port.ops = &sa1100_pops;
- sa1100_ports[i].port.fifosize = 8;
- sa1100_ports[i].port.line = i;
- sa1100_ports[i].port.iotype = UPIO_MEM;
- init_timer(&sa1100_ports[i].timer);
- sa1100_ports[i].timer.function = sa1100_timeout;
- sa1100_ports[i].timer.data = (unsigned long)&sa1100_ports[i];
- }
-
- /*
- * make transmit lines outputs, so that when the port
- * is closed, the output is in the MARK state.
- */
- PPDR |= PPC_TXD1 | PPC_TXD3;
- PPSR |= PPC_TXD1 | PPC_TXD3;
-}
void __devinit sa1100_register_uart_fns(struct sa1100_port_fns *fns)
{
@@ -647,41 +638,6 @@ void __devinit sa1100_register_uart_fns(struct sa1100_port_fns *fns)
sa1100_pops.set_wake = fns->set_wake;
}
-void __init sa1100_register_uart(int idx, int port)
-{
- if (idx >= NR_PORTS) {
- printk(KERN_ERR "%s: bad index number %d\n", __func__, idx);
- return;
- }
-
- switch (port) {
- case 1:
- sa1100_ports[idx].port.membase = (void __iomem *)&Ser1UTCR0;
- sa1100_ports[idx].port.mapbase = _Ser1UTCR0;
- sa1100_ports[idx].port.irq = IRQ_Ser1UART;
- sa1100_ports[idx].port.flags = UPF_BOOT_AUTOCONF;
- break;
-
- case 2:
- sa1100_ports[idx].port.membase = (void __iomem *)&Ser2UTCR0;
- sa1100_ports[idx].port.mapbase = _Ser2UTCR0;
- sa1100_ports[idx].port.irq = IRQ_Ser2ICP;
- sa1100_ports[idx].port.flags = UPF_BOOT_AUTOCONF;
- break;
-
- case 3:
- sa1100_ports[idx].port.membase = (void __iomem *)&Ser3UTCR0;
- sa1100_ports[idx].port.mapbase = _Ser3UTCR0;
- sa1100_ports[idx].port.irq = IRQ_Ser3UART;
- sa1100_ports[idx].port.flags = UPF_BOOT_AUTOCONF;
- break;
-
- default:
- printk(KERN_ERR "%s: bad port number %d\n", __func__, port);
- }
-}
-
-
#ifdef CONFIG_SERIAL_SA1100_CONSOLE
static void sa1100_console_putchar(struct uart_port *port, int ch)
{
@@ -701,6 +657,9 @@ sa1100_console_write(struct console *co, const char *s, unsigned int count)
struct sa1100_port *sport = &sa1100_ports[co->index];
unsigned int old_utcr3, status;
+ if (co->index < 0 || co->index >= NR_PORTS)
+ return;
+
/*
* First, save UTCR3 and then disable interrupts
*/
@@ -764,6 +723,8 @@ sa1100_console_setup(struct console *co, char *options)
int bits = 8;
int parity = 'n';
int flow = 'n';
+ struct resource *res_mem, *res_irq;
+ struct platform_device *pdev;
/*
* Check whether an invalid uart number has been specified, and
@@ -774,6 +735,30 @@ sa1100_console_setup(struct console *co, char *options)
co->index = 0;
sport = &sa1100_ports[co->index];
+ sport->port.uartclk = get_clock_tick_rate();
+ sport->port.ops = &sa1100_pops;
+ sport->port.fifosize = 8;
+ sport->port.line = co->index;
+ sport->port.iotype = UPIO_MEM;
+
+ pdev = sa1100_get_uart(co->index);
+ if (!pdev)
+ return -ENODEV;
+
+ res_mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+ if (!res_mem)
+ return -ENODEV;
+
+ res_irq = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
+ if (!res_irq)
+ return -ENODEV;
+
+ sport->port.mapbase = res_mem->start;
+ sport->port.irq = res_irq->start;
+ sport->size = res_mem->end - res_mem->start + 1;
+
+ sport->port.membase = ioremap(sport->port.mapbase, sport->size);
+
if (options)
uart_parse_options(options, &baud, &parity, &bits, &flow);
else
@@ -784,7 +769,7 @@ sa1100_console_setup(struct console *co, char *options)
static struct uart_driver sa1100_reg;
static struct console sa1100_console = {
- .name = "ttySA",
+ .name = SA1100_DEVICENAME,
.write = sa1100_console_write,
.device = uart_console_device,
.setup = sa1100_console_setup,
@@ -795,7 +780,6 @@ static struct console sa1100_console = {
static int __init sa1100_rs_console_init(void)
{
- sa1100_init_ports();
register_console(&sa1100_console);
return 0;
}
@@ -808,8 +792,8 @@ console_initcall(sa1100_rs_console_init);
static struct uart_driver sa1100_reg = {
.owner = THIS_MODULE,
- .driver_name = "ttySA",
- .dev_name = "ttySA",
+ .driver_name = "sa11x0-uart",
+ .dev_name = SA1100_DEVICENAME,
.major = SERIAL_SA1100_MAJOR,
.minor = MINOR_START,
.nr = NR_PORTS,
@@ -836,28 +820,62 @@ static int sa1100_serial_resume(struct platform_device *dev)
return 0;
}
-static int sa1100_serial_probe(struct platform_device *dev)
+static int __devinit sa1100_serial_probe(struct platform_device *pdev)
{
- struct resource *res = dev->resource;
- int i;
+ struct resource *res_mem, *res_irq;
+ struct device_node *np = pdev->dev.of_node;
+ struct sa1100_port *sport;
+ int ret, id;
- for (i = 0; i < dev->num_resources; i++, res++)
- if (res->flags & IORESOURCE_MEM)
- break;
+ if (np)
+ id = of_alias_get_id(np, "serial");
+ else
+ id = pdev->id;
- if (i < dev->num_resources) {
- for (i = 0; i < NR_PORTS; i++) {
- if (sa1100_ports[i].port.mapbase != res->start)
- continue;
+ if (id < 0)
+ id = find_first_zero_bit(&sa1100_ports_in_use,
+ sizeof(sa1100_ports_in_use));
- sa1100_ports[i].port.dev = &dev->dev;
- uart_add_one_port(&sa1100_reg, &sa1100_ports[i].port);
- platform_set_drvdata(dev, &sa1100_ports[i]);
- break;
- }
+ if (id < 0 || id >= NR_PORTS)
+ return -EINVAL;
+
+ if (test_and_set_bit(id, &sa1100_ports_in_use))
+ return -EBUSY;
+
+ res_mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+ if (!res_mem) {
+ clear_bit(id, &sa1100_ports_in_use);
+ printk(KERN_INFO "SA11x0 no mem for %d\n", id);
+ return -ENODEV;
+ }
+ res_irq = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
+ if (!res_irq) {
+ clear_bit(id, &sa1100_ports_in_use);
+ printk(KERN_INFO "SA11x0 no irq for %d\n", id);
+ return -ENODEV;
}
- return 0;
+ sport = &sa1100_ports[id];
+
+ sport->port.uartclk = get_clock_tick_rate();
+ sport->port.ops = &sa1100_pops;
+ sport->port.fifosize = 8;
+ sport->port.line = id;
+ sport->port.iotype = UPIO_MEM;
+ sport->port.mapbase = res_mem->start;
+ sport->port.irq = res_irq->start;
+ sport->port.flags = UPF_BOOT_AUTOCONF;
+ sport->timer.function = sa1100_timeout;
+ sport->timer.data = (unsigned long)sport;
+ sport->size = res_mem->end - res_mem->start + 1;
+
+ init_timer(&sport->timer);
+ platform_set_drvdata(pdev, sport);
+ sport->port.dev = &pdev->dev;
+ ret = uart_add_one_port(&sa1100_reg, &sport->port);
+ if (ret)
+ clear_bit(id, &sa1100_ports_in_use);
+ return ret;
}
static int sa1100_serial_remove(struct platform_device *pdev)
@@ -866,20 +884,22 @@ static int sa1100_serial_remove(struct platform_device *pdev)
platform_set_drvdata(pdev, NULL);
- if (sport)
+ if (sport) {
uart_remove_one_port(&sa1100_reg, &sport->port);
-
+ clear_bit(sport->port.line, &sa1100_ports_in_use);
+ }
return 0;
}
static struct platform_driver sa11x0_serial_driver = {
.probe = sa1100_serial_probe,
- .remove = sa1100_serial_remove,
+ .remove = __devexit_p(sa1100_serial_remove),
.suspend = sa1100_serial_suspend,
.resume = sa1100_serial_resume,
.driver = {
.name = "sa11x0-uart",
.owner = THIS_MODULE,
+ .of_match_table = of_match_ptr(sa1100_serial_dt_ids),
},
};
@@ -889,8 +909,6 @@ static int __init sa1100_serial_init(void)
printk(KERN_INFO "Serial: SA11x0 driver\n");
- sa1100_init_ports();
-
ret = uart_register_driver(&sa1100_reg);
if (ret == 0) {
ret = platform_driver_register(&sa11x0_serial_driver);
@@ -912,5 +930,5 @@ module_exit(sa1100_serial_exit);
MODULE_AUTHOR("Deep Blue Solutions Ltd");
MODULE_DESCRIPTION("SA1100 generic serial port driver");
MODULE_LICENSE("GPL");
-MODULE_ALIAS_CHARDEV_MAJOR(SERIAL_SA1100_MAJOR);
+MODULE_ALIAS_CHARDEV(SERIAL_SA1100_MAJOR, MINOR_START);
MODULE_ALIAS("platform:sa11x0-uart");
--
1.7.7.3
More information about the devicetree-discuss
mailing list