From: Andrew Lunn <and...@lunn.ch> Date: Wed, 27 Sep 2017 00:26:04 +0200
> + > + for (port = 0; port < mv88e6xxx_num_ports(chip); ++port) { Just a nit pick since you are going to respin this anyways: for (x; cond(x); x++) seems to be the canonical way to express this in the kernel, so please make "++port" into "port++" Thank you.