All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH V2 1/2] serial: MIPS: lantiq: add clk_enable() call to driver
@ 2013-08-09 18:54 John Crispin
  2013-08-09 18:54 ` [PATCH V2 2/2] serial: MIPS: lantiq: fix clock error check John Crispin
  0 siblings, 1 reply; 2+ messages in thread
From: John Crispin @ 2013-08-09 18:54 UTC (permalink / raw)
  To: Greg Kroah-Hartman; +Cc: linux-serial, linux-mips, Thomas Langer

From: Thomas Langer <thomas.langer@lantiq.com>

Enable the clock if one is present when setting up the console.

Signed-off-by: Thomas Langer <thomas.langer@lantiq.com>
Acked-by: John Crispin <blogic@openwrt.org>
---
Changes in V2:
* make use of IS_ERR()

 drivers/tty/serial/lantiq.c |    3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/tty/serial/lantiq.c b/drivers/tty/serial/lantiq.c
index 15733da..93ac046 100644
--- a/drivers/tty/serial/lantiq.c
+++ b/drivers/tty/serial/lantiq.c
@@ -636,6 +636,9 @@ lqasc_console_setup(struct console *co, char *options)
 
 	port = &ltq_port->port;
 
+	if (!IS_ERR(ltq_port->clk))
+		clk_enable(ltq_port->clk);
+
 	port->uartclk = clk_get_rate(ltq_port->fpiclk);
 
 	if (options)
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [PATCH V2 2/2] serial: MIPS: lantiq: fix clock error check
  2013-08-09 18:54 [PATCH V2 1/2] serial: MIPS: lantiq: add clk_enable() call to driver John Crispin
@ 2013-08-09 18:54 ` John Crispin
  0 siblings, 0 replies; 2+ messages in thread
From: John Crispin @ 2013-08-09 18:54 UTC (permalink / raw)
  To: Greg Kroah-Hartman; +Cc: linux-serial, linux-mips, John Crispin

The clock should be checked with the proper IS_ERR() api before using it.

Signed-off-by: John Crispin <blogic@openwrt.org>
---
 drivers/tty/serial/lantiq.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/tty/serial/lantiq.c b/drivers/tty/serial/lantiq.c
index 93ac046..88d01e0 100644
--- a/drivers/tty/serial/lantiq.c
+++ b/drivers/tty/serial/lantiq.c
@@ -318,7 +318,7 @@ lqasc_startup(struct uart_port *port)
 	struct ltq_uart_port *ltq_port = to_ltq_uart_port(port);
 	int retval;
 
-	if (ltq_port->clk)
+	if (!IS_ERR(ltq_port->clk))
 		clk_enable(ltq_port->clk);
 	port->uartclk = clk_get_rate(ltq_port->fpiclk);
 
@@ -386,7 +386,7 @@ lqasc_shutdown(struct uart_port *port)
 		port->membase + LTQ_ASC_RXFCON);
 	ltq_w32_mask(ASCTXFCON_TXFEN, ASCTXFCON_TXFFLU,
 		port->membase + LTQ_ASC_TXFCON);
-	if (ltq_port->clk)
+	if (!IS_ERR(ltq_port->clk))
 		clk_disable(ltq_port->clk);
 }
 
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-08-09 19:02 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-09 18:54 [PATCH V2 1/2] serial: MIPS: lantiq: add clk_enable() call to driver John Crispin
2013-08-09 18:54 ` [PATCH V2 2/2] serial: MIPS: lantiq: fix clock error check John Crispin

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.