| Message ID | 20210505091928.22010-30-jslaby@suse.cz |
|---|---|
| State | New |
| Headers | show |
| Series | None | expand |
On Wed, May 05, 2021 at 11:19:22AM +0200, Jiri Slaby wrote: > "if"'s true branch in digi_chars_in_buffer returns. So there is no need > for "else" and indented code. Remove this else and shift the code to the > left. > > Signed-off-by: Jiri Slaby <jslaby@suse.cz> > Cc: Johan Hovold <johan@kernel.org> > Cc: linux-usb@vger.kernel.org > --- > drivers/usb/serial/digi_acceleport.c | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/drivers/usb/serial/digi_acceleport.c b/drivers/usb/serial/digi_acceleport.c > index 19ee8191647c..0c4d611621c2 100644 > --- a/drivers/usb/serial/digi_acceleport.c > +++ b/drivers/usb/serial/digi_acceleport.c > @@ -1050,12 +1050,11 @@ static unsigned int digi_chars_in_buffer(struct tty_struct *tty) > priv->dp_port_num, port->bulk_out_size - 2); > /* return(port->bulk_out_size - 2); */ > return 256; > - } else { > - dev_dbg(&port->dev, "digi_chars_in_buffer: port=%d, chars=%d\n", > - priv->dp_port_num, priv->dp_out_buf_len); > - return priv->dp_out_buf_len; > } > > + dev_dbg(&port->dev, "digi_chars_in_buffer: port=%d, chars=%d\n", > + priv->dp_port_num, priv->dp_out_buf_len); > + return priv->dp_out_buf_len; > } This doesn't look like much of an improvement so I'm dropping this one. If we want to clean this up we should use a common exit path for both branches. I'll send a couple of patches to address this and a related issue. Johan
diff --git a/drivers/usb/serial/digi_acceleport.c b/drivers/usb/serial/digi_acceleport.c index 19ee8191647c..0c4d611621c2 100644 --- a/drivers/usb/serial/digi_acceleport.c +++ b/drivers/usb/serial/digi_acceleport.c @@ -1050,12 +1050,11 @@ static unsigned int digi_chars_in_buffer(struct tty_struct *tty) priv->dp_port_num, port->bulk_out_size - 2); /* return(port->bulk_out_size - 2); */ return 256; - } else { - dev_dbg(&port->dev, "digi_chars_in_buffer: port=%d, chars=%d\n", - priv->dp_port_num, priv->dp_out_buf_len); - return priv->dp_out_buf_len; } + dev_dbg(&port->dev, "digi_chars_in_buffer: port=%d, chars=%d\n", + priv->dp_port_num, priv->dp_out_buf_len); + return priv->dp_out_buf_len; } static void digi_dtr_rts(struct usb_serial_port *port, int on)
"if"'s true branch in digi_chars_in_buffer returns. So there is no need for "else" and indented code. Remove this else and shift the code to the left. Signed-off-by: Jiri Slaby <jslaby@suse.cz> Cc: Johan Hovold <johan@kernel.org> Cc: linux-usb@vger.kernel.org --- drivers/usb/serial/digi_acceleport.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-)