Hi Egil, Egil Hjelmeland <pri...@egil-hjelmeland.no> writes:
> Would doing > > - chip->ds = dsa_switch_alloc(chip->dev, DSA_MAX_PORTS); > + chip->ds = dsa_switch_alloc(chip->dev, LAN9303_NUM_PORTS); > > at the same time be good, or breaking the scope of the patch? It is indeed out of scope. You may want to add a first commit "net: dsa: lan9303: introduce LAN9303_NUM_PORTS" for instance. Thanks, Vivien