From: Michael Grzeschik <m.grzesc...@pengutronix.de> Date: Mon, 7 Dec 2020 13:56:24 +0100
> @@ -991,13 +1090,16 @@ static void ksz8_port_setup(struct ksz_device *dev, > int port, bool cpu_port) > static void ksz8_config_cpu_port(struct dsa_switch *ds) > { > struct ksz_device *dev = ds->priv; > + struct ksz8 *ksz8 = dev->priv; > + const u8 *regs = ksz8->regs; > + const u32 *masks = ksz8->masks; > struct ksz_port *p; > u8 remote; > int i; > Please use reverse christmas tree ordering for local variables. Thank you.