[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <0F1B54C89D5F954D8535DB252AF412FA06C359EF@chinexm1.ad.analog.com>
Date: Tue, 24 Aug 2010 10:37:02 +0800
From: "Zhang, Sonic" <Sonic.Zhang@...log.com>
To: "Joe Perches" <joe@...ches.com>, <linux-kernel@...r.kernel.org>
CC: <uclinux-dist-devel@...ckfin.uclinux.org>
Subject: RE: [PATCH 1/2] drivers/serial/bfin_5xx.c: Fix logging messages continued with \
>-----Original Message-----
>From: Joe Perches [mailto:joe@...ches.com]
>Sent: Tuesday, August 24, 2010 3:02 AM
>To: linux-kernel@...r.kernel.org
>Cc: Zhang, Sonic; uclinux-dist-devel@...ckfin.uclinux.org
>Subject: [PATCH 1/2] drivers/serial/bfin_5xx.c: Fix logging
>messages continued with \
>
>Using continuation lines often introduces undesired whitespace.
>Fix the misuses.
>
Acked-by: Sonic Zhang <sonic.zhang@...log.com>
>Signed-off-by: Joe Perches <joe@...ches.com>
>---
> drivers/serial/bfin_5xx.c | 8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/drivers/serial/bfin_5xx.c
>b/drivers/serial/bfin_5xx.c index a9eff2b..4e38f99 100644
>--- a/drivers/serial/bfin_5xx.c
>+++ b/drivers/serial/bfin_5xx.c
>@@ -734,8 +734,8 @@ static int bfin_serial_startup(struct
>uart_port *port)
> IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING |
> IRQF_DISABLED, "BFIN_UART_CTS", uart)) {
> uart->cts_pin = -1;
>- pr_info("Unable to attach BlackFin UART
>CTS interrupt.\
>- So, disable it.\n");
>+ pr_info("Unable to attach BlackFin UART
>CTS interrupt. "
>+ "So, disable it.\n");
> }
> }
> if (uart->rts_pin >= 0) {
>@@ -747,8 +747,8 @@ static int bfin_serial_startup(struct
>uart_port *port)
> if (request_irq(uart->status_irq,
> bfin_serial_mctrl_cts_int,
> IRQF_DISABLED, "BFIN_UART_MODEM_STATUS", uart)) {
>- pr_info("Unable to attach BlackFin UART Modem \
>- Status interrupt.\n");
>+ pr_info("Unable to attach BlackFin UART Modem "
>+ "Status interrupt.\n");
> }
>
> /* CTS RTS PINs are negative assertive. */
>--
>1.7.2.19.g9a302
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists