> -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 <and...@lunn.ch>

    Andrew

Reply via email to