* [PATCH 0/5] tty: serial: msm: Fix checkpatch errors
@ 2014-08-05 7:51 Kiran Padwal
2014-08-05 7:51 ` [PATCH 1/5] tty: serial: msm: remove braces {} in msm_serial.c Kiran Padwal
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: Kiran Padwal @ 2014-08-05 7:51 UTC (permalink / raw)
To: davidb
Cc: dwalker, bryanh, sboyd, gregkh, jslaby, kiran.padwal,
linux-arm-msm, linux-serial, linux-kernel
Kiran Padwal (5):
tty: serial: msm: remove braces {} in msm_serial.c
tty: serial: msm: Fix 'Missing a blank line after declarations'
warning
tty: serial: msm: Fix style warnings relating to printk()
tty: serial: msm: Fix 'else is not generally useful after a break or
return' warning
tty: serial: msm: Fix 'void function return statements are not
generally useful' warning
drivers/tty/serial/msm_serial.c | 26 +++++++++++---------------
1 file changed, 11 insertions(+), 15 deletions(-)
--
1.7.9.5
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/5] tty: serial: msm: remove braces {} in msm_serial.c
2014-08-05 7:51 [PATCH 0/5] tty: serial: msm: Fix checkpatch errors Kiran Padwal
@ 2014-08-05 7:51 ` Kiran Padwal
2014-08-05 7:52 ` [PATCH 2/5] tty: serial: msm: Fix 'Missing a blank line after declarations' warning Kiran Padwal
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Kiran Padwal @ 2014-08-05 7:51 UTC (permalink / raw)
To: davidb
Cc: dwalker, bryanh, sboyd, gregkh, jslaby, kiran.padwal,
linux-arm-msm, linux-serial, linux-kernel
fixed below checkpatch.pl warning:
WARNING: braces {} are not necessary for any arm of this statement
Signed-off-by: Kiran Padwal <kiran.padwal@smartplayin.com>
---
drivers/tty/serial/msm_serial.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/tty/serial/msm_serial.c b/drivers/tty/serial/msm_serial.c
index 0da0b54..6c7300b 100644
--- a/drivers/tty/serial/msm_serial.c
+++ b/drivers/tty/serial/msm_serial.c
@@ -190,11 +190,10 @@ static void handle_rx(struct uart_port *port)
/* Mask conditions we're ignorning. */
sr &= port->read_status_mask;
- if (sr & UART_SR_RX_BREAK) {
+ if (sr & UART_SR_RX_BREAK)
flag = TTY_BREAK;
- } else if (sr & UART_SR_PAR_FRAME_ERR) {
+ else if (sr & UART_SR_PAR_FRAME_ERR)
flag = TTY_FRAME;
- }
if (!uart_handle_sysrq_char(port, c))
tty_insert_flip_char(tport, c, flag);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/5] tty: serial: msm: Fix 'Missing a blank line after declarations' warning
2014-08-05 7:51 [PATCH 0/5] tty: serial: msm: Fix checkpatch errors Kiran Padwal
2014-08-05 7:51 ` [PATCH 1/5] tty: serial: msm: remove braces {} in msm_serial.c Kiran Padwal
@ 2014-08-05 7:52 ` Kiran Padwal
2014-08-05 7:52 ` [PATCH 3/5] tty: serial: msm: Fix style warnings relating to printk() Kiran Padwal
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Kiran Padwal @ 2014-08-05 7:52 UTC (permalink / raw)
To: davidb
Cc: dwalker, bryanh, sboyd, gregkh, jslaby, kiran.padwal,
linux-arm-msm, linux-serial, linux-kernel
This patch fixes below checkpatch.pl warning and it remove extra blank lines:
WARNING: Missing a blank line after declarations
Signed-off-by: Kiran Padwal <kiran.padwal@smartplayin.com>
---
drivers/tty/serial/msm_serial.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/tty/serial/msm_serial.c b/drivers/tty/serial/msm_serial.c
index 6c7300b..0295493 100644
--- a/drivers/tty/serial/msm_serial.c
+++ b/drivers/tty/serial/msm_serial.c
@@ -314,7 +314,6 @@ static unsigned int msm_get_mctrl(struct uart_port *port)
return TIOCM_CAR | TIOCM_CTS | TIOCM_DSR | TIOCM_RTS;
}
-
static void msm_reset(struct uart_port *port)
{
struct msm_port *msm_port = UART_TO_MSM(port);
@@ -335,6 +334,7 @@ static void msm_reset(struct uart_port *port)
static void msm_set_mctrl(struct uart_port *port, unsigned int mctrl)
{
unsigned int mr;
+
mr = msm_read(port, UART_MR1);
if (!(mctrl & TIOCM_RTS)) {
@@ -430,7 +430,6 @@ static int msm_set_baud_rate(struct uart_port *port, unsigned int baud)
return baud;
}
-
static void msm_init_clock(struct uart_port *port)
{
struct msm_port *msm_port = UART_TO_MSM(port);
@@ -645,6 +644,7 @@ fail_release_port:
static void msm_config_port(struct uart_port *port, int flags)
{
int ret;
+
if (flags & UART_CONFIG_TYPE) {
port->type = PORT_MSM;
ret = msm_request_port(port);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 3/5] tty: serial: msm: Fix style warnings relating to printk()
2014-08-05 7:51 [PATCH 0/5] tty: serial: msm: Fix checkpatch errors Kiran Padwal
2014-08-05 7:51 ` [PATCH 1/5] tty: serial: msm: remove braces {} in msm_serial.c Kiran Padwal
2014-08-05 7:52 ` [PATCH 2/5] tty: serial: msm: Fix 'Missing a blank line after declarations' warning Kiran Padwal
@ 2014-08-05 7:52 ` Kiran Padwal
2014-08-05 7:52 ` [PATCH 4/5] tty: serial: msm: Fix 'else is not generally useful after a break or return' warning Kiran Padwal
2014-08-05 7:52 ` [PATCH 5/5] tty: serial: msm: Fix 'void function return statements are not generally useful' warning Kiran Padwal
4 siblings, 0 replies; 6+ messages in thread
From: Kiran Padwal @ 2014-08-05 7:52 UTC (permalink / raw)
To: davidb
Cc: dwalker, bryanh, sboyd, gregkh, jslaby, kiran.padwal,
linux-arm-msm, linux-serial, linux-kernel
fixed below checkpatch.pl warning:
WARNING: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(... to printk(KERN_ERR ...
WARNING: Prefer [subsystem eg: netdev]_info([subsystem]dev, ... then dev_info(dev, ... then pr_info(... to printk(KERN_INFO ...
Signed-off-by: Kiran Padwal <kiran.padwal@smartplayin.com>
---
drivers/tty/serial/msm_serial.c | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/drivers/tty/serial/msm_serial.c b/drivers/tty/serial/msm_serial.c
index 0295493..e70d4e8 100644
--- a/drivers/tty/serial/msm_serial.c
+++ b/drivers/tty/serial/msm_serial.c
@@ -677,7 +677,7 @@ static void msm_power(struct uart_port *port, unsigned int state,
clk_disable_unprepare(msm_port->pclk);
break;
default:
- printk(KERN_ERR "msm_serial: Unknown PM state %d\n", state);
+ pr_err("msm_serial: Unknown PM state %d\n", state);
}
}
@@ -957,7 +957,7 @@ static int __init msm_console_setup(struct console *co, char *options)
msm_write(port, UART_CR_TX_ENABLE, UART_CR);
}
- printk(KERN_INFO "msm_serial: console setup on port #%d\n", port->line);
+ pr_info("msm_serial: console setup on port #%d\n", port->line);
return uart_set_options(port, co, baud, parity, bits, flow);
}
@@ -1012,7 +1012,7 @@ static int msm_serial_probe(struct platform_device *pdev)
if (unlikely(pdev->id < 0 || pdev->id >= UART_NR))
return -ENXIO;
- printk(KERN_INFO "msm_serial: detected port #%d\n", pdev->id);
+ dev_info(&pdev->dev, "msm_serial: detected port #%d\n", pdev->id);
port = get_port_from_line(pdev->id);
port->dev = &pdev->dev;
@@ -1037,8 +1037,7 @@ static int msm_serial_probe(struct platform_device *pdev)
}
port->uartclk = clk_get_rate(msm_port->clk);
- printk(KERN_INFO "uartclk = %d\n", port->uartclk);
-
+ dev_info(&pdev->dev, "uartclk = %d\n", port->uartclk);
resource = platform_get_resource(pdev, IORESOURCE_MEM, 0);
if (unlikely(!resource))
@@ -1092,7 +1091,7 @@ static int __init msm_serial_init(void)
if (unlikely(ret))
uart_unregister_driver(&msm_uart_driver);
- printk(KERN_INFO "msm_serial: driver initialized\n");
+ pr_info("msm_serial: driver initialized\n");
return ret;
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 4/5] tty: serial: msm: Fix 'else is not generally useful after a break or return' warning
2014-08-05 7:51 [PATCH 0/5] tty: serial: msm: Fix checkpatch errors Kiran Padwal
` (2 preceding siblings ...)
2014-08-05 7:52 ` [PATCH 3/5] tty: serial: msm: Fix style warnings relating to printk() Kiran Padwal
@ 2014-08-05 7:52 ` Kiran Padwal
2014-08-05 7:52 ` [PATCH 5/5] tty: serial: msm: Fix 'void function return statements are not generally useful' warning Kiran Padwal
4 siblings, 0 replies; 6+ messages in thread
From: Kiran Padwal @ 2014-08-05 7:52 UTC (permalink / raw)
To: davidb
Cc: dwalker, bryanh, sboyd, gregkh, jslaby, kiran.padwal,
linux-arm-msm, linux-serial, linux-kernel
fixed below checkpatch.pl warning:
WARNING: else is not generally useful after a break or return
Signed-off-by: Kiran Padwal <kiran.padwal@smartplayin.com>
---
drivers/tty/serial/msm_serial.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/tty/serial/msm_serial.c b/drivers/tty/serial/msm_serial.c
index e70d4e8..dfebbaf 100644
--- a/drivers/tty/serial/msm_serial.c
+++ b/drivers/tty/serial/msm_serial.c
@@ -700,8 +700,8 @@ static int msm_poll_get_char_single(struct uart_port *port)
if (!(msm_read(port, UART_SR) & UART_SR_RX_READY))
return NO_POLL_CHAR;
- else
- return msm_read(port, rf_reg) & 0xff;
+
+ return msm_read(port, rf_reg) & 0xff;
}
static int msm_poll_get_char_dm_1p3(struct uart_port *port)
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 5/5] tty: serial: msm: Fix 'void function return statements are not generally useful' warning
2014-08-05 7:51 [PATCH 0/5] tty: serial: msm: Fix checkpatch errors Kiran Padwal
` (3 preceding siblings ...)
2014-08-05 7:52 ` [PATCH 4/5] tty: serial: msm: Fix 'else is not generally useful after a break or return' warning Kiran Padwal
@ 2014-08-05 7:52 ` Kiran Padwal
4 siblings, 0 replies; 6+ messages in thread
From: Kiran Padwal @ 2014-08-05 7:52 UTC (permalink / raw)
To: davidb
Cc: dwalker, bryanh, sboyd, gregkh, jslaby, kiran.padwal,
linux-arm-msm, linux-serial, linux-kernel
fixed below checkpatch.pl warning:
WARNING: void function return statements are not generally useful
Signed-off-by: Kiran Padwal <kiran.padwal@smartplayin.com>
---
drivers/tty/serial/msm_serial.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/tty/serial/msm_serial.c b/drivers/tty/serial/msm_serial.c
index dfebbaf..26df038 100644
--- a/drivers/tty/serial/msm_serial.c
+++ b/drivers/tty/serial/msm_serial.c
@@ -787,8 +787,6 @@ static void msm_poll_put_char(struct uart_port *port, unsigned char c)
/* Enable interrupts */
msm_write(port, imr, UART_IMR);
-
- return;
}
#endif
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-08-05 7:56 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-05 7:51 [PATCH 0/5] tty: serial: msm: Fix checkpatch errors Kiran Padwal
2014-08-05 7:51 ` [PATCH 1/5] tty: serial: msm: remove braces {} in msm_serial.c Kiran Padwal
2014-08-05 7:52 ` [PATCH 2/5] tty: serial: msm: Fix 'Missing a blank line after declarations' warning Kiran Padwal
2014-08-05 7:52 ` [PATCH 3/5] tty: serial: msm: Fix style warnings relating to printk() Kiran Padwal
2014-08-05 7:52 ` [PATCH 4/5] tty: serial: msm: Fix 'else is not generally useful after a break or return' warning Kiran Padwal
2014-08-05 7:52 ` [PATCH 5/5] tty: serial: msm: Fix 'void function return statements are not generally useful' warning Kiran Padwal
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).