On 06. nov. 2017 15:19, Egil Hjelmeland wrote:
Now that ds->num_ports is 3, there is no need to check range of "port"
parameter.

+       lan9303_disable_processing_port(chip, port);
+       lan9303_phy_write(ds, chip->phy_addr_sel_strap + port,
+                         MII_BMCR, BMCR_PDOWN);
  }

I see now that line breaking could be adjusted here. But I am going to touch that line in a upcomming patch about phy_addr_sel_strap. I
promise fix it then, making it to a one-liner.

static int lan9303_port_bridge_join(struct dsa_switch *ds, int port,


Egil

Reply via email to