> -int mv88e6390x_serdes_get_lane(struct mv88e6xxx_chip *chip, int port)
> +int mv88e6390x_serdes_get_lane(struct mv88e6xxx_chip *chip, int port, s8
> *lane)
> {
> u8 cmode_port9, cmode_port10, cmode_port;
>
> @@ -323,76 +320,80 @@ int mv88e6390x_serdes_get_lane(struct mv88e6xxx_chip
> *chip, int port)
> cmode_port10 = chip->ports[10].cmode;
> cmode_port = chip->ports[port].cmode;
>
> + *lane = -1;
> +
You could move that into mv88e8xxx_serdes_get_lane().
Otherwise
Reviewed-by: Andrew Lunn <[email protected]>
Andrew