linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 2/6] USB: serial: Changes to conform with checkpatch. - spaces around '?' and ':'.
@ 2012-05-13 13:52 Ben Minerds
  2012-05-13 13:52 ` [PATCH 3/6] USB: serial: Changes to conform with checkpatch. - space around '=' Ben Minerds
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Ben Minerds @ 2012-05-13 13:52 UTC (permalink / raw)
  To: alan; +Cc: gregkh, richard.weinberger, linux-usb, linux-kernel, puzzleduck

Removed 14 checkpatch.sh errors.

Signed-off-by: Ben Minerds <puzzleduck@gmail.com>
---
 drivers/usb/serial/keyspan.c |   18 +++++++++---------
 1 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/usb/serial/keyspan.c b/drivers/usb/serial/keyspan.c
index b689990..d029a04 100644
--- a/drivers/usb/serial/keyspan.c
+++ b/drivers/usb/serial/keyspan.c
@@ -180,7 +180,7 @@ static void keyspan_set_termios(struct tty_struct *tty,
 	tty_encode_baud_rate(tty, baud_rate, baud_rate);
 	/* set CTS/RTS handshake etc. */
 	p_priv->cflag = cflag;
-	p_priv->flow_control = (cflag & CRTSCTS)? flow_cts: flow_none;
+	p_priv->flow_control = (cflag & CRTSCTS) ? flow_cts : flow_none;
 
 	/* Mark/Space not supported */
 	tty->termios->c_cflag &= ~CMSPAR;
@@ -1134,7 +1134,7 @@ static int keyspan_open(struct tty_struct *tty, struct usb_serial_port *port)
 	}
 	/* set CTS/RTS handshake etc. */
 	p_priv->cflag = cflag;
-	p_priv->flow_control = (cflag & CRTSCTS)? flow_cts: flow_none;
+	p_priv->flow_control = (cflag & CRTSCTS) ? flow_cts : flow_none;
 
 	keyspan_send_setup(port, 1);
 	/* mdelay(100); */
@@ -1744,7 +1744,7 @@ static int keyspan_usa26_send_setup(struct usb_serial *serial,
 		msg.setPrescaler = 0xff;
 	}
 
-	msg.lcr = (p_priv->cflag & CSTOPB)? STOPBITS_678_2: STOPBITS_5678_1;
+	msg.lcr = (p_priv->cflag & CSTOPB) ? STOPBITS_678_2 : STOPBITS_5678_1;
 	switch (p_priv->cflag & CSIZE) {
 	case CS5:
 		msg.lcr |= USA_DATABITS_5;
@@ -1761,7 +1761,7 @@ static int keyspan_usa26_send_setup(struct usb_serial *serial,
 	}
 	if (p_priv->cflag & PARENB) {
 		/* note USA_PARITY_NONE == 0 */
-		msg.lcr |= (p_priv->cflag & PARODD)?
+		msg.lcr |= (p_priv->cflag & PARODD) ?
 			USA_PARITY_ODD : USA_PARITY_EVEN;
 	}
 	msg.setLcr = 0xff;
@@ -2033,7 +2033,7 @@ static int keyspan_usa49_send_setup(struct usb_serial *serial,
 		/* msg.setPrescaler = 0xff; */
 	}
 
-	msg.lcr = (p_priv->cflag & CSTOPB)? STOPBITS_678_2: STOPBITS_5678_1;
+	msg.lcr = (p_priv->cflag & CSTOPB) ? STOPBITS_678_2 : STOPBITS_5678_1;
 	switch (p_priv->cflag & CSIZE) {
 	case CS5:
 		msg.lcr |= USA_DATABITS_5;
@@ -2050,7 +2050,7 @@ static int keyspan_usa49_send_setup(struct usb_serial *serial,
 	}
 	if (p_priv->cflag & PARENB) {
 		/* note USA_PARITY_NONE == 0 */
-		msg.lcr |= (p_priv->cflag & PARODD)?
+		msg.lcr |= (p_priv->cflag & PARODD) ?
 			USA_PARITY_ODD : USA_PARITY_EVEN;
 	}
 	msg.setLcr = 0xff;
@@ -2219,7 +2219,7 @@ static int keyspan_usa90_send_setup(struct usb_serial *serial,
 		msg.txMode = TXMODE_BYHAND;
 	}
 
-	msg.lcr = (p_priv->cflag & CSTOPB)? STOPBITS_678_2: STOPBITS_5678_1;
+	msg.lcr = (p_priv->cflag & CSTOPB) ? STOPBITS_678_2 : STOPBITS_5678_1;
 	switch (p_priv->cflag & CSIZE) {
 	case CS5:
 		msg.lcr |= USA_DATABITS_5;
@@ -2236,7 +2236,7 @@ static int keyspan_usa90_send_setup(struct usb_serial *serial,
 	}
 	if (p_priv->cflag & PARENB) {
 		/* note USA_PARITY_NONE == 0 */
-		msg.lcr |= (p_priv->cflag & PARODD)?
+		msg.lcr |= (p_priv->cflag & PARODD) ?
 			USA_PARITY_ODD : USA_PARITY_EVEN;
 	}
 	if (p_priv->old_cflag != p_priv->cflag) {
@@ -2365,7 +2365,7 @@ static int keyspan_usa67_send_setup(struct usb_serial *serial,
 	}
 	if (p_priv->cflag & PARENB) {
 		/* note USA_PARITY_NONE == 0 */
-		msg.lcr |= (p_priv->cflag & PARODD)?
+		msg.lcr |= (p_priv->cflag & PARODD) ?
 					USA_PARITY_ODD : USA_PARITY_EVEN;
 	}
 	msg.setLcr = 0xff;
-- 
1.7.2.5


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

* [PATCH 3/6] USB: serial: Changes to conform with checkpatch. - space around '='.
  2012-05-13 13:52 [PATCH 2/6] USB: serial: Changes to conform with checkpatch. - spaces around '?' and ':' Ben Minerds
@ 2012-05-13 13:52 ` Ben Minerds
  2012-05-13 13:52 ` [PATCH 4/6] USB: serial: Changes to conform with checkpatch. - space near open parenthesis '(' Ben Minerds
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Ben Minerds @ 2012-05-13 13:52 UTC (permalink / raw)
  To: alan; +Cc: gregkh, richard.weinberger, linux-usb, linux-kernel, puzzleduck

Removed 1 checkpatch.sh error.

Signed-off-by: Ben Minerds <puzzleduck@gmail.com>
---
 drivers/usb/serial/keyspan.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/usb/serial/keyspan.c b/drivers/usb/serial/keyspan.c
index d029a04..425e113 100644
--- a/drivers/usb/serial/keyspan.c
+++ b/drivers/usb/serial/keyspan.c
@@ -485,7 +485,7 @@ static void usa28_indat_callback(struct urb *urb)
 		p_priv = usb_get_serial_port_data(port);
 		data = urb->transfer_buffer;
 
-		tty =tty_port_tty_get(&port->port);
+		tty = tty_port_tty_get(&port->port);
 		if (tty && urb->actual_length) {
 			tty_insert_flip_string(tty, data, urb->actual_length);
 			tty_flip_buffer_push(tty);
-- 
1.7.2.5


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

* [PATCH 4/6] USB: serial: Changes to conform with checkpatch. - space near open parenthesis '('.
  2012-05-13 13:52 [PATCH 2/6] USB: serial: Changes to conform with checkpatch. - spaces around '?' and ':' Ben Minerds
  2012-05-13 13:52 ` [PATCH 3/6] USB: serial: Changes to conform with checkpatch. - space around '=' Ben Minerds
@ 2012-05-13 13:52 ` Ben Minerds
  2012-05-13 13:52 ` [PATCH 5/6] USB: serial: Changes to conform with checkpatch. - trailing whitespace Ben Minerds
  2012-05-13 13:52 ` [PATCH 6/6] USB: serial: Changes to conform with checkpatch. - parentheses not required Ben Minerds
  3 siblings, 0 replies; 5+ messages in thread
From: Ben Minerds @ 2012-05-13 13:52 UTC (permalink / raw)
  To: alan; +Cc: gregkh, richard.weinberger, linux-usb, linux-kernel, puzzleduck

Removed 2 checkpatch.sh errors.

Signed-off-by: Ben Minerds <puzzleduck@gmail.com>
---
 drivers/usb/serial/keyspan.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/usb/serial/keyspan.c b/drivers/usb/serial/keyspan.c
index 425e113..49e814c 100644
--- a/drivers/usb/serial/keyspan.c
+++ b/drivers/usb/serial/keyspan.c
@@ -569,7 +569,7 @@ static void	usa28_instat_callback(struct urb *urb)
 	p_priv->dcd_state = ((msg->dcd) ? 1 : 0);
 	p_priv->ri_state = ((msg->ri) ? 1 : 0);
 
-	if( old_dcd_state != p_priv->dcd_state && old_dcd_state) {
+	if (old_dcd_state != p_priv->dcd_state && old_dcd_state) {
 		tty = tty_port_tty_get(&port->port);
 		if (tty && !C_CLOCAL(tty)) 
 			tty_hangup(tty);
-- 
1.7.2.5


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

* [PATCH 5/6] USB: serial: Changes to conform with checkpatch. - trailing whitespace.
  2012-05-13 13:52 [PATCH 2/6] USB: serial: Changes to conform with checkpatch. - spaces around '?' and ':' Ben Minerds
  2012-05-13 13:52 ` [PATCH 3/6] USB: serial: Changes to conform with checkpatch. - space around '=' Ben Minerds
  2012-05-13 13:52 ` [PATCH 4/6] USB: serial: Changes to conform with checkpatch. - space near open parenthesis '(' Ben Minerds
@ 2012-05-13 13:52 ` Ben Minerds
  2012-05-13 13:52 ` [PATCH 6/6] USB: serial: Changes to conform with checkpatch. - parentheses not required Ben Minerds
  3 siblings, 0 replies; 5+ messages in thread
From: Ben Minerds @ 2012-05-13 13:52 UTC (permalink / raw)
  To: alan; +Cc: gregkh, richard.weinberger, linux-usb, linux-kernel, puzzleduck

Removed 1 checkpatch.sh error.

Signed-off-by: Ben Minerds <puzzleduck@gmail.com>
---
 drivers/usb/serial/keyspan.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/usb/serial/keyspan.c b/drivers/usb/serial/keyspan.c
index 49e814c..3c02c73 100644
--- a/drivers/usb/serial/keyspan.c
+++ b/drivers/usb/serial/keyspan.c
@@ -571,7 +571,7 @@ static void	usa28_instat_callback(struct urb *urb)
 
 	if (old_dcd_state != p_priv->dcd_state && old_dcd_state) {
 		tty = tty_port_tty_get(&port->port);
-		if (tty && !C_CLOCAL(tty)) 
+		if (tty && !C_CLOCAL(tty))
 			tty_hangup(tty);
 		tty_kref_put(tty);
 	}
-- 
1.7.2.5


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

* [PATCH 6/6] USB: serial: Changes to conform with checkpatch. - parentheses not required.
  2012-05-13 13:52 [PATCH 2/6] USB: serial: Changes to conform with checkpatch. - spaces around '?' and ':' Ben Minerds
                   ` (2 preceding siblings ...)
  2012-05-13 13:52 ` [PATCH 5/6] USB: serial: Changes to conform with checkpatch. - trailing whitespace Ben Minerds
@ 2012-05-13 13:52 ` Ben Minerds
  3 siblings, 0 replies; 5+ messages in thread
From: Ben Minerds @ 2012-05-13 13:52 UTC (permalink / raw)
  To: alan; +Cc: gregkh, richard.weinberger, linux-usb, linux-kernel, puzzleduck

Removed 1 checkpatch.sh error.

Signed-off-by: Ben Minerds <puzzleduck@gmail.com>
---
 drivers/usb/serial/keyspan.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/usb/serial/keyspan.c b/drivers/usb/serial/keyspan.c
index 3c02c73..8f8df13 100644
--- a/drivers/usb/serial/keyspan.c
+++ b/drivers/usb/serial/keyspan.c
@@ -1273,7 +1273,7 @@ static int keyspan_fake_startup(struct usb_serial *serial)
 
 	if (request_ihex_firmware(&fw, fw_name, &serial->dev->dev)) {
 		dev_err(&serial->dev->dev, "Required keyspan firmware image (%s) unavailable.\n", fw_name);
-		return(1);
+		return 1;
 	}
 
 	dbg("Uploading Keyspan %s firmware.", fw_name);
-- 
1.7.2.5


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

end of thread, other threads:[~2012-05-13 13:51 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-05-13 13:52 [PATCH 2/6] USB: serial: Changes to conform with checkpatch. - spaces around '?' and ':' Ben Minerds
2012-05-13 13:52 ` [PATCH 3/6] USB: serial: Changes to conform with checkpatch. - space around '=' Ben Minerds
2012-05-13 13:52 ` [PATCH 4/6] USB: serial: Changes to conform with checkpatch. - space near open parenthesis '(' Ben Minerds
2012-05-13 13:52 ` [PATCH 5/6] USB: serial: Changes to conform with checkpatch. - trailing whitespace Ben Minerds
2012-05-13 13:52 ` [PATCH 6/6] USB: serial: Changes to conform with checkpatch. - parentheses not required Ben Minerds

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).