On Sat, Jun 20, 2015 at 02:47:20PM -0300, Fabio Estevam wrote: > On Sat, Jun 20, 2015 at 1:15 PM, Andrew Lunn <and...@lunn.ch> wrote: > > > @@ -1764,6 +1765,11 @@ static int fec_enet_mdio_read(struct mii_bus *bus, > > int mii_id, int regnum) > > { > > struct fec_enet_private *fep = bus->priv; > > unsigned long time_left; > > + int ret; > > + > > + ret = clk_prepare_enable(fep->clk_ipg); > > + if (ret) > > + return 0xffff; > > Why don`t you return ret instead?
ret would also work. v3 to follow soon. Andrew -- To unsubscribe from this list: send the line "unsubscribe netdev" in