Hi Marek, On Sun, 25 Aug 2019 05:59:13 +0200, Marek Behún <marek.be...@nic.cz> wrote: > +int mv88e6341_serdes_get_lane(struct mv88e6xxx_chip *chip, int port, s8 > *lane) > +{ > + u8 cmode = chip->ports[port].cmode; > + > + *lane = -1; > + > + if (port != 5) > + return 0;
Aren't you relying on -ENODEV as well? > + > + if (cmode == MV88E6XXX_PORT_STS_CMODE_1000BASE_X || > + cmode == MV88E6XXX_PORT_STS_CMODE_SGMII || > + cmode == MV88E6XXX_PORT_STS_CMODE_2500BASEX) > + *lane = MV88E6341_PORT5_LANE; > + > + return 0; > +}