On 31. juli 2017 17:01, Vivien Didelot wrote:
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.


In a later series I assume? Or is allowed to add patches to a series
under review?



Thanks,

         Vivien


Egil

Reply via email to