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.

Reply via email to