From: Marek Behún <marek.be...@nic.cz> Date: Thu, 22 Aug 2019 01:27:15 +0200
> diff --git a/drivers/net/dsa/mv88e6xxx/serdes.c > b/drivers/net/dsa/mv88e6xxx/serdes.c > index 20c526c2a9ee..0f3d7cbb696b 100644 > --- a/drivers/net/dsa/mv88e6xxx/serdes.c > +++ b/drivers/net/dsa/mv88e6xxx/serdes.c > @@ -506,9 +506,11 @@ static void mv88e6390_serdes_irq_link_sgmii(struct > mv88e6xxx_chip *chip, > int port, int lane) > { > struct dsa_switch *ds = chip->ds; > + u8 cmode = chip->ports[port].cmode; > int duplex = DUPLEX_UNKNOWN; > int speed = SPEED_UNKNOWN; > int link, err; > + phy_interface_t mode; > u16 status; Please retain the reverse christmas tree ordering of local variables here. Thank you.