Commit b25c2e7d authored by Mahesh Bandewar's avatar Mahesh Bandewar Committed by David S. Miller

bonding: Simplify __get_duplex function.

Eliminate 'else' clause by simply initializing variable
Signed-off-by: default avatarMahesh Bandewar <maheshb@google.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 12d43096
...@@ -327,14 +327,12 @@ static u16 __get_link_speed(struct port *port) ...@@ -327,14 +327,12 @@ static u16 __get_link_speed(struct port *port)
static u8 __get_duplex(struct port *port) static u8 __get_duplex(struct port *port)
{ {
struct slave *slave = port->slave; struct slave *slave = port->slave;
u8 retval; u8 retval = 0x0;
/* handling a special case: when the configuration starts with /* handling a special case: when the configuration starts with
* link down, it sets the duplex to 0. * link down, it sets the duplex to 0.
*/ */
if (slave->link != BOND_LINK_UP) { if (slave->link == BOND_LINK_UP) {
retval = 0x0;
} else {
switch (slave->duplex) { switch (slave->duplex) {
case DUPLEX_FULL: case DUPLEX_FULL:
retval = 0x1; retval = 0x1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment