[dpdk-dev] Patchworks Re: [PATCH v5 0/9] net/ionic: minor updates and documentation

Ferruh Yigit ferruh.yigit at intel.com
Thu Dec 10 16:06:23 CET 2020


On 12/10/2020 2:44 PM, Andrew Boyer wrote:
> Ferruh,
> 'After sending a new version of a patch, developers should set the previous 
> patch as “Superseded”.'
> 
> Did this happen automatically for v4 & friends or did you do it? (I did not). I 
> will do it in future if it is not automatic.
> 
> 'When a patch is applied, it is set to “Accepted”.'
> 
> Does that happen automatically, or do you want to do it, or can I help by doing 
> it myself?
> (Or do you wait until it reaches main?)
> 

Patchwork updates are not automatic, all done manually.

When a new version of patch(set) is sent, author should update the status of the 
previous version as 'Superseded'.

When patch is merged, patchwork status set as 'Accepted' by committer, author 
doesn't need to do that.

If you think existing documentation is not clear enough, would you mind sending 
a web patch [1] to clarify it, this can help others too.

[1] https://git.dpdk.org/tools/dpdk-web/


> -Andrew
> 
>> On Dec 10, 2020, at 7:31 AM, Ferruh Yigit <ferruh.yigit at intel.com 
>> <mailto:ferruh.yigit at intel.com>> wrote:
>>
>> On 12/10/2020 2:57 AM, Andrew Boyer wrote:
>>> These patches make some minor changes to the ionic PMD.
>>> Signed-off-by: Andrew Boyer <aboyer at pensando.io <mailto:aboyer at pensando.io>>
>>> ---
>>> v5:
>>> * Fix syntax error in .rst file
>>> v4:
>>> * Address additional reviewer comments
>>> v3:
>>> * Fixed up false-positives from codespell and gcc 8.3/9.3
>>> v2:
>>> * Address Ferruh's comments (thank you!)
>>> * Switch to net/ionic: prefix
>>> * Leave UNMAINTAINED flag for now
>>> * Split patch 8 into patch 8 & 9; drop whitespace changes
>>> Andrew Boyer (9):
>>>   net/ionic: connect ionic to the build system
>>>   net/ionic: update interface file to the latest version
>>>   net/ionic: update documentation and MAINTAINERS
>>>   net/ionic: check for cmd completion more frequently
>>>   net/ionic: remove some unused fields
>>>   net/ionic: convert 'deferred' boolean to a flag bit
>>>   net/ionic: warn if loopback mode is requested
>>>   net/ionic: minor refactorings and helper variables
>>>   net/ionic: minor logging fixups
>>
>> For series,
>> Reviewed-by: Ferruh Yigit <ferruh.yigit at intel.com <mailto:ferruh.yigit at intel.com>>
>>
>> Series applied to dpdk-next-net/main, thanks.
> 



More information about the dev mailing list