"Govindraj.R" <govindraj.raja@xxxxxx> writes:
> From 4756e3743c7acd2de1030b2bd432c1b19f0b9ff5 Mon Sep 17 00:00:00 2001
> From: Govindraj R <govindraj.raja@xxxxxx>
> Date: Fri, 13 Nov 2009 12:01:54 +0530
> Subject: [PATCH] OMAP UART: Add omap-serial driver support.
>
> This patch adds support for OMAP3430-HIGH SPEED UART Controller.
>
> It adds support for the following features:
> 1. It supports Interrupt mode and DMA mode of operation.
> 2. Supports Hardware flow control and sofware flow control.
> 3. Debug Console support on all UARTs.
>
> Signed-off-by: Govindraj R <govindraj.raja@xxxxxx>
Govindraj,
FYI... to use this version of the driver with the UART omap_device
conversion I just posted, I used this additional patch on top of your
driver.
These are various issues I pointed out in the review as well, but
wanted to share this with V3 + this patch, I was able to use it
with the omap_device conversion.
Kevin
commit 79285bfc6c8711f580895256a82e7dc127156824
Author: Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx>
Date: Mon Nov 23 15:10:13 2009 -0800
OMAP: HS-UART: misc. fixes
diff --git a/arch/arm/plat-omap/include/plat/omap-serial.h b/arch/arm/plat-omap/include/plat/omap-serial.h
index 4341fae..86bec82 100644
--- a/arch/arm/plat-omap/include/plat/omap-serial.h
+++ b/arch/arm/plat-omap/include/plat/omap-serial.h
@@ -28,7 +28,7 @@
* in bootargs we specify as console=ttyO0 if uart1
* is used as console uart.
*/
-#define DEVICE_NAME "ttyO"
+#define DEVICE_NAME "ttyS"
/* *
* We default to IRQ0 for the "no irq" hack. Some
diff --git a/drivers/serial/omap-serial.c b/drivers/serial/omap-serial.c
index 6dd8bb4..65d05f8 100644
--- a/drivers/serial/omap-serial.c
+++ b/drivers/serial/omap-serial.c
@@ -993,7 +993,7 @@ static struct console serial_omap_console = {
static void serial_omap_add_console_port(struct uart_omap_port *up)
{
- serial_omap_console_ports[up->pdev->id - 1] = up;
+ serial_omap_console_ports[up->pdev->id] = up;
}
#define OMAP_CONSOLE (&serial_omap_console)
@@ -1237,7 +1237,7 @@ static int serial_omap_probe(struct platform_device *pdev)
up->port.regshift = 2;
up->port.fifosize = 64;
up->port.ops = &serial_omap_pops;
- up->port.line = pdev->id - 1;
+ up->port.line = pdev->id;
up->port.membase = ioremap_nocache(up->port.mapbase,
0x16 << up->port.regshift);
up->port.flags = UPF_BOOT_AUTOCONF;
@@ -1263,7 +1263,7 @@ static int serial_omap_probe(struct platform_device *pdev)
pr_err("\n %s: UART Driver Init Failed!\n", __func__);
return -EPERM;
}
- ui[pdev->id - 1] = up;
+ ui[pdev->id] = up;
serial_omap_add_console_port(up);
ret = uart_add_one_port(&serial_omap_reg, &up->port);
@@ -1298,7 +1298,7 @@ static struct platform_driver serial_omap_driver = {
.suspend = serial_omap_suspend,
.resume = serial_omap_resume,
.driver = {
- .name = "omap-uart",
+ .name = DRIVER_NAME,
},
};
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
