On Wed, Apr 15, 2015 at 10:12:42PM -0700, Guenter Roeck wrote: > Return correct error code if _mv88e6xxx_reg_read returns an error. > > Fixes: facd95b2e0ec0 ("net: dsa: mv88e6xxx: Add Hardware bridging support") > Signed-off-by: Guenter Roeck <li...@roeck-us.net>
Hi Guenter Good catch. I'm surprised there is no compiler warning, possibly used before set. Reviewed-by: Andrew Lunn <and...@lunn.ch> Thanks Andrew > --- > drivers/net/dsa/mv88e6xxx.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/dsa/mv88e6xxx.c b/drivers/net/dsa/mv88e6xxx.c > index fc8d3b6ffe8e..f64186a5f634 100644 > --- a/drivers/net/dsa/mv88e6xxx.c > +++ b/drivers/net/dsa/mv88e6xxx.c > @@ -908,8 +908,10 @@ static int mv88e6xxx_set_port_state(struct dsa_switch > *ds, int port, u8 state) > mutex_lock(&ps->smi_mutex); > > reg = _mv88e6xxx_reg_read(ds, REG_PORT(port), PORT_CONTROL); > - if (reg < 0) > + if (reg < 0) { > + ret = reg; > goto abort; > + } > > oldstate = reg & PORT_CONTROL_STATE_MASK; > if (oldstate != state) { > -- > 2.1.0 > > -- > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html