This patch make changes to an if else statement which simplifies the code
allowing to remove a return.
CC: Joe Perches <j...@perches.com>
Signed-off-by: Peter Senna Tschudin <peter.se...@gmail.com>
---
Subject was: [PATCH 3/9] usb/host/fotg210: Remove useless else statement
To be applied on top of V1.
Changes from V1:
Instead of just removing the else statement, remove the return inside
the if statement so the code is even more readable.
drivers/usb/host/fotg210-hcd.c | 12 +++++-------
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/drivers/usb/host/fotg210-hcd.c b/drivers/usb/host/fotg210-hcd.c
index 7f6aa99..064cd85 100644
--- a/drivers/usb/host/fotg210-hcd.c
+++ b/drivers/usb/host/fotg210-hcd.c
@@ -1396,15 +1396,13 @@ static int check_reset_complete(struct fotg210_hcd
*fotg210, int index,
return port_status;
/* if reset finished and it's still not enabled -- handoff */
- if (!(port_status & PORT_PE)) {
+ if (!(port_status & PORT_PE))
/* with integrated TT, there's nobody to hand it to! */
- fotg210_dbg(fotg210,
- "Failed to enable port %d on root hub TT\n",
+ fotg210_dbg(fotg210, "Failed to enable port %d on root hub
TT\n",
+ index + 1);
+ else
+ fotg210_dbg(fotg210, "port %d reset complete, port enabled\n",
index + 1);
- return port_status;
- }
- fotg210_dbg(fotg210, "port %d reset complete, port enabled\n",
- index + 1);
return port_status;
}
--
2.1.0
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
