Re: [PATCH net-next] net: dsa: mv88e6xxx: check for mode change in port_setup_mac
From: Andrew Lunn <andrew@lunn.ch>
Date: 2019-08-13 20:32:38
On Tue, Aug 13, 2019 at 07:12:43PM +0200, Marek Behún wrote:
quoted hunk ↗ jump to hunk
@@ -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, ®); 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