| Date | Fri, 2 Oct 2015 12:30:16 -0500 | From | Felipe Balbi <> | Subject | Re: [PATCH 03/14] RFC: usb/host/fotg210: Remove useless else statement |
| |
On Mon, Sep 21, 2015 at 05:01:07PM +0200, Peter Senna Tschudin wrote: > This patch remove an else statement after a return to make the code > easier to understand. > > Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
looks good to me.
> --- > drivers/usb/host/fotg210-hcd.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/usb/host/fotg210-hcd.c b/drivers/usb/host/fotg210-hcd.c > index 36413b2..f574143 100644 > --- a/drivers/usb/host/fotg210-hcd.c > +++ b/drivers/usb/host/fotg210-hcd.c > @@ -1410,10 +1410,9 @@ static int check_reset_complete(struct fotg210_hcd *fotg210, int index, > "Failed to enable port %d on root hub TT\n", > index+1); > return port_status; > - } else { > - fotg210_dbg(fotg210, "port %d reset complete, port enabled\n", > - index + 1); > } > + fotg210_dbg(fotg210, "port %d reset complete, port enabled\n", > + index + 1); > > return port_status; > } > -- > 2.1.0 >
-- balbi [unhandled content-type:application/pgp-signature] |