On Tue, Aug 13, 2019 at 07:12:43PM +0200, Marek BehĂșn wrote:
> @@ -598,12 +599,49 @@ int mv88e6352_port_link_state(struct mv88e6xxx_chip 
> *chip, int port,
>                             struct phylink_link_state *state)
>  {
>       int err;
> -     u16 reg;
> +     u16 reg, mac;
>  
>       err = mv88e6xxx_port_read(chip, port, MV88E6XXX_PORT_STS, &reg);
>       if (err)
>               return err;

Hi Marek

It seems a bit off putting this block of code here, after reading
MV88E6XXX_PORT_STS but before using the value. You don't need STS to
determine the interface mode.

If you keep the code together, you can then reuse reg, rather than add
mac.

Apart from that, this looks O.K.

      Andrew

Reply via email to