commit b049bd9de4959dd9e4b586d14b6de450a52c6f1f
author Russell King <rmk@dyn-67.arm.linux.org.uk> Mon Aug 20 10:28:15 2007 +0100
committer Russell King <rmk+kernel@arm.linux.org.uk> Fri Oct 12 21:15:07 2007 +0100
tree 11bcddc46d8af9e7f4fa43e247bc20bf5170d635
parent ebebd9b0a1463d5de89017ad59a6b9cd4044687f

[ARM] pxa: update pxa serial driver to use clk support

Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>

drivers/serial/pxa.c

diff --git a/drivers/serial/pxa.c b/drivers/serial/pxa.c
index 59889f6..af3a011 100644
--- a/drivers/serial/pxa.c
+++ b/drivers/serial/pxa.c
@@ -42,6 +42,7 @@
 #include <linux/tty.h>
 #include <linux/tty_flip.h>
 #include <linux/serial_core.h>
+#include <linux/clk.h>
 
 #include <asm/io.h>
 #include <asm/hardware.h>
@@ -55,7 +56,7 @@
 	unsigned char           lcr;
 	unsigned char           mcr;
 	unsigned int            lsr_break_flag;
-	unsigned int		cken;
+	struct clk		*clk;
 	char			*name;
 };
 
@@ -351,6 +352,8 @@
 	else
 		up->mcr = 0;
 
+	up->port.uartclk = clk_get_rate(up->clk);
+
 	/*
 	 * Allocate the IRQ
 	 */
@@ -546,9 +549,11 @@
 	      unsigned int oldstate)
 {
 	struct uart_pxa_port *up = (struct uart_pxa_port *)port;
-	pxa_set_cken(up->cken, !state);
+
 	if (!state)
-		udelay(1);
+		clk_enable(up->clk);
+	else
+		clk_disable(up->clk);
 }
 
 static void serial_pxa_release_port(struct uart_port *port)
@@ -635,6 +640,8 @@
 	struct uart_pxa_port *up = serial_pxa_ports[co->index];
 	unsigned int ier;
 
+	clk_enable(up->clk);
+
 	/*
 	 *	First save the IER then disable the interrupts
 	 */
@@ -649,6 +656,8 @@
 	 */
 	wait_for_xmitr(up);
 	serial_out(up, UART_IER, ier);
+
+	clk_disable(up->clk);
 }
 
 static int __init
@@ -752,6 +761,12 @@
 	if (!sport)
 		return -ENOMEM;
 
+	sport->clk = clk_get(&dev->dev, "UARTCLK");
+	if (IS_ERR(sport->clk)) {
+		ret = PTR_ERR(sport->clk);
+		goto err_free;
+	}
+
 	sport->port.type = PORT_PXA;
 	sport->port.iotype = UPIO_MEM;
 	sport->port.mapbase = mmres->start;
@@ -761,7 +776,7 @@
 	sport->port.line = dev->id;
 	sport->port.dev = &dev->dev;
 	sport->port.flags = UPF_IOREMAP | UPF_BOOT_AUTOCONF;
-	sport->port.uartclk = 921600 * 16;
+	sport->port.uartclk = clk_get_rate(sport->clk);
 
 	/*
 	 * Is it worth keeping this?
@@ -780,7 +795,7 @@
 	sport->port.membase = ioremap(mmres->start, mmres->end - mmres->start + 1);
 	if (!sport->port.membase) {
 		ret = -ENOMEM;
-		goto err_free;
+		goto err_clk;
 	}
 
 	serial_pxa_ports[dev->id] = sport;
@@ -790,6 +805,8 @@
 
 	return 0;
 
+ err_clk:
+	clk_put(sport->clk);
  err_free:
 	kfree(sport);
 	return ret;
@@ -802,6 +819,7 @@
 	platform_set_drvdata(dev, NULL);
 
 	uart_remove_one_port(&serial_pxa_reg, &sport->port);
+	clk_put(sport->clk);
 	kfree(sport);
 
 	return 0;