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

Reply via email to