> - mdiobus_write(phydev->mdio.bus, priv->addr, XILINX_GMII2RGMII_REG, val); > + err = mdiobus_write(phydev->mdio.bus, priv->addr, XILINX_GMII2RGMII_REG, > + val); > + if (err < 0) > + return err; > > return 0; Do you need to assign err? Why not just return mdiobus_write(phydev->mdio.bus, priv->addr, XILINX_GMII2RGMII_REG, val);