diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-06-23 18:17:04 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-06-25 11:42:33 -0400 |
| commit | 64b2f72671a9a8cbea1aa575d3ea3998987e8c53 (patch) | |
| tree | e62dd809b59f1aa4f7c6c4c4160a2cc876e247e9 /drivers | |
| parent | 74811a710e18c22eca1ffbf97ebd6201efea7d91 (diff) | |
net: dsa: mv88e6xxx: fix error code in mv88e6390_serdes_power()
We're accidentally returning the wrong variable. "cmode" is
uninitialized at this point so it causes a static checker warning.
Fixes: 6335e9f2446b ("net: dsa: mv88e6xxx: mv88e6390X SERDES support")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/dsa/mv88e6xxx/serdes.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/dsa/mv88e6xxx/serdes.c b/drivers/net/dsa/mv88e6xxx/serdes.c index 411b4f522792..f3c01119b3d1 100644 --- a/drivers/net/dsa/mv88e6xxx/serdes.c +++ b/drivers/net/dsa/mv88e6xxx/serdes.c @@ -192,7 +192,7 @@ int mv88e6390_serdes_power(struct mv88e6xxx_chip *chip, int port, bool on) err = mv88e6xxx_port_get_cmode(chip, port, &cmode); if (err) - return cmode; + return err; switch (port) { case 2: |