This patch hoists pl011_hwinit() further up within the driver. This
permits a later patch to introduce an extended .poll_init callback
that does more than pure hardware initialization.

Signed-off-by: Daniel Thompson <daniel.thomp...@linaro.org>
Cc: Russell King <li...@arm.linux.org.uk>
Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org>
Cc: Jiri Slaby <jsl...@suse.cz>
Cc: linux-ser...@vger.kernel.org
---
 drivers/tty/serial/amba-pl011.c | 78 ++++++++++++++++++++---------------------
 1 file changed, 39 insertions(+), 39 deletions(-)

diff --git a/drivers/tty/serial/amba-pl011.c b/drivers/tty/serial/amba-pl011.c
index 02016fc..0b06dcf 100644
--- a/drivers/tty/serial/amba-pl011.c
+++ b/drivers/tty/serial/amba-pl011.c
@@ -1425,6 +1425,45 @@ static void pl011_break_ctl(struct uart_port *port, int 
break_state)
        spin_unlock_irqrestore(&uap->port.lock, flags);
 }
 
+static int pl011_hwinit(struct uart_port *port)
+{
+       struct uart_amba_port *uap =
+           container_of(port, struct uart_amba_port, port);
+       int retval;
+
+       /* Optionaly enable pins to be muxed in and configured */
+       pinctrl_pm_select_default_state(port->dev);
+
+       /*
+        * Try to enable the clock producer.
+        */
+       retval = clk_prepare_enable(uap->clk);
+       if (retval)
+               return retval;
+
+       uap->port.uartclk = clk_get_rate(uap->clk);
+
+       /* Clear pending error and receive interrupts */
+       writew(UART011_OEIS | UART011_BEIS | UART011_PEIS | UART011_FEIS |
+              UART011_RTIS | UART011_RXIS, uap->port.membase + UART011_ICR);
+
+       /*
+        * Save interrupts enable mask, and enable RX interrupts in case if
+        * the interrupt is used for NMI entry.
+        */
+       uap->im = readw(uap->port.membase + UART011_IMSC);
+       writew(UART011_RTIM | UART011_RXIM, uap->port.membase + UART011_IMSC);
+
+       if (dev_get_platdata(uap->port.dev)) {
+               struct amba_pl011_data *plat;
+
+               plat = dev_get_platdata(uap->port.dev);
+               if (plat->init)
+                       plat->init();
+       }
+       return 0;
+}
+
 #ifdef CONFIG_CONSOLE_POLL
 
 static void pl011_quiesce_irqs(struct uart_port *port)
@@ -1483,45 +1522,6 @@ static void pl011_put_poll_char(struct uart_port *port,
 
 #endif /* CONFIG_CONSOLE_POLL */
 
-static int pl011_hwinit(struct uart_port *port)
-{
-       struct uart_amba_port *uap =
-           container_of(port, struct uart_amba_port, port);
-       int retval;
-
-       /* Optionaly enable pins to be muxed in and configured */
-       pinctrl_pm_select_default_state(port->dev);
-
-       /*
-        * Try to enable the clock producer.
-        */
-       retval = clk_prepare_enable(uap->clk);
-       if (retval)
-               return retval;
-
-       uap->port.uartclk = clk_get_rate(uap->clk);
-
-       /* Clear pending error and receive interrupts */
-       writew(UART011_OEIS | UART011_BEIS | UART011_PEIS | UART011_FEIS |
-              UART011_RTIS | UART011_RXIS, uap->port.membase + UART011_ICR);
-
-       /*
-        * Save interrupts enable mask, and enable RX interrupts in case if
-        * the interrupt is used for NMI entry.
-        */
-       uap->im = readw(uap->port.membase + UART011_IMSC);
-       writew(UART011_RTIM | UART011_RXIM, uap->port.membase + UART011_IMSC);
-
-       if (dev_get_platdata(uap->port.dev)) {
-               struct amba_pl011_data *plat;
-
-               plat = dev_get_platdata(uap->port.dev);
-               if (plat->init)
-                       plat->init();
-       }
-       return 0;
-}
-
 static void pl011_write_lcr_h(struct uart_amba_port *uap, unsigned int lcr_h)
 {
        writew(lcr_h, uap->port.membase + uap->lcrh_rx);
-- 
1.9.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to