Re: [PATCH net-next 00/18] net: dsa: mv88e6xxx: 802.1s and 88E6390 VTU

[Vivien Didelot]

Hi David,

David Miller <davem@xxxxxxxxxxxxx> writes:

> From: Vivien Didelot <vivien.didelot@xxxxxxxxxxxxxxxxxxxx>
> Date: Mon, 01 May 2017 11:51:05 -0400
>
>> Hi David,
>> 
>> Vivien Didelot <vivien.didelot@xxxxxxxxxxxxxxxxxxxx> writes:
>> 
>>> Vivien Didelot (18):
>>>   net: dsa: mv88e6xxx: add max VID to info
>>>   net: dsa: mv88e6xxx: split VTU entry data member
>>>   net: dsa: mv88e6xxx: move VTU Operation accessors
>>>   net: dsa: mv88e6xxx: move VTU flush
>>>   net: dsa: mv88e6xxx: move VTU FID accessors
>>>   net: dsa: mv88e6xxx: move VTU SID accessors
>>>   net: dsa: mv88e6xxx: move VTU VID accessors
>>>   net: dsa: mv88e6xxx: move generic VTU GetNext
>>>   net: dsa: mv88e6xxx: move VTU Data accessors
>>>   net: dsa: mv88e6xxx: move STU GetNext operation
>>>   net: dsa: mv88e6xxx: get STU entry on VTU GetNext
>>>   net: dsa: mv88e6xxx: load STU entry with VTU entry
>>>   net: dsa: mv88e6xxx: add VTU GetNext operation
>>>   net: dsa: mv88e6xxx: add VTU Load/Purge operation
>>>   net: dsa: mv88e6xxx: make VTU helpers static
>>>   net: dsa: mv88e6xxx: simplify VTU entry getter
>>>   net: dsa: mv88e6xxx: support the VTU Page bit
>>>   net: dsa: mv88e6xxx: add VTU support for 88E6390
>>>
>>>  drivers/net/dsa/mv88e6xxx/Makefile      |   1 +
>>>  drivers/net/dsa/mv88e6xxx/chip.c        | 568 +++++++++-----------------------
>>>  drivers/net/dsa/mv88e6xxx/global1.h     |  14 +
>>>  drivers/net/dsa/mv88e6xxx/global1_vtu.c | 505 ++++++++++++++++++++++++++++
>>>  drivers/net/dsa/mv88e6xxx/mv88e6xxx.h   |  31 +-
>>>  5 files changed, 684 insertions(+), 435 deletions(-)
>>>  create mode 100644 drivers/net/dsa/mv88e6xxx/global1_vtu.c
>> 
>> I don't see the patchset in net-next. Can you pick it?
>
> This had feedback which you must address, therefore the state in
> patchwork is "changes requested" which you could have checked before
> asking me this.
>
> If the patch set is fine and you got ACKs for all of your patches
> then please resubmit.
>
> Otherwise, make the changes you were asked to make or explain to
> the person who asked for them why they are not appropriate.

We did discuss the patchset and everything is good as is. Can someone
change the state in patchwork afterwards? It feels weird to resend the
patch with no changes.

Thanks,

        Vivien