Re: [PATCH net-next 2/2] net: dsa: lan9303: Simplify lan9303_xxx_packet_processing() usage

From: Egil Hjelmeland
Date: Mon Jul 31 2017 - 11:08:31 EST


On 31. juli 2017 17:01, Vivien Didelot wrote:
Hi Egil,

Egil Hjelmeland <privat@xxxxxxxxxxxxxxxxxx> 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