* [PATCH v2] staging: serqt_usb2: Fix coding style issues.
@ 2014-03-31 6:51 Thomas Vegas
2014-03-31 15:55 ` Greg KH
0 siblings, 1 reply; 2+ messages in thread
From: Thomas Vegas @ 2014-03-31 6:51 UTC (permalink / raw)
To: gregkh; +Cc: joe, linux-kernel, devel
Add blank lines after declarations and cleanup initializations.
Signed-off-by: Thomas Vegas <thomas_75@safe-mail.net>
---
drivers/staging/serqt_usb2/serqt_usb2.c | 22 +++++++---------------
1 file changed, 7 insertions(+), 15 deletions(-)
diff --git a/drivers/staging/serqt_usb2/serqt_usb2.c b/drivers/staging/serqt_usb2/serqt_usb2.c
index 0267dd8..10a7240 100644
--- a/drivers/staging/serqt_usb2/serqt_usb2.c
+++ b/drivers/staging/serqt_usb2/serqt_usb2.c
@@ -990,18 +990,11 @@ static void qt_block_until_empty(struct tty_struct *tty,
static void qt_close(struct usb_serial_port *port)
{
struct usb_serial *serial = port->serial;
- struct quatech_port *qt_port;
- struct quatech_port *port0;
- struct tty_struct *tty;
+ struct quatech_port *qt_port = qt_get_port_private(port);
+ struct quatech_port *port0 = qt_get_port_private(serial->port[0]);
+ struct tty_struct *tty = tty_port_tty_get(&port->port);
int status;
- unsigned int index;
- status = 0;
-
- tty = tty_port_tty_get(&port->port);
- index = port->port_number;
-
- qt_port = qt_get_port_private(port);
- port0 = qt_get_port_private(serial->port[0]);
+ unsigned int index = port->port_number;
/* shutdown any bulk reads that might be going on */
if (serial->num_bulk_out)
@@ -1030,7 +1023,6 @@ static void qt_close(struct usb_serial_port *port)
dev_dbg(&port->dev, "Shutdown interrupt_in_urb\n");
usb_kill_urb(serial->port[0]->interrupt_in_urb);
}
-
}
if (qt_port->write_urb) {
@@ -1268,9 +1260,9 @@ static void qt_set_termios(struct tty_struct *tty,
if (I_IXOFF(tty) || I_IXON(tty)) {
unsigned char stop_char = STOP_CHAR(tty);
unsigned char start_char = START_CHAR(tty);
- status =
- box_set_sw_flow_ctrl(port->serial, index, stop_char,
- start_char);
+
+ status = box_set_sw_flow_ctrl(port->serial, index, stop_char,
+ start_char);
if (status < 0)
dev_dbg(&port->dev,
"box_set_sw_flow_ctrl (enabled) failed\n");
--
1.8.3.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] staging: serqt_usb2: Fix coding style issues.
2014-03-31 6:51 [PATCH v2] staging: serqt_usb2: Fix coding style issues Thomas Vegas
@ 2014-03-31 15:55 ` Greg KH
0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2014-03-31 15:55 UTC (permalink / raw)
To: Thomas Vegas; +Cc: joe, devel, linux-kernel
On Mon, Mar 31, 2014 at 08:51:04AM +0200, Thomas Vegas wrote:
> Add blank lines after declarations and cleanup initializations.
Patches should do only 1 thing, not multiple things if at all possible.
So this should be 2 patches, not one. Care to break it up like that and
resend 2 patches instead?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-03-31 15:53 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-03-31 6:51 [PATCH v2] staging: serqt_usb2: Fix coding style issues Thomas Vegas
2014-03-31 15:55 ` Greg KH
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.