[dpdk-dev] DEV Ask For Help about patch sumbit

[Ferruh Yigit]

On 3/19/2021 3:27 AM, caowenbo at mucse.com wrote:
> Hi Ferruh
> 
> I’am a developer of MuChuang IIC. Welcome to reach us on website 
> http://www.mucse.com.//
> 
> Now we want to support our 10/40 Ethernet Nic and then continue the development 
> and maintenance in community.
> 
> But I don’t know  the first thingthat I need to do for the submit patch.
> 
> Can you give me some advice? 😊
> 
> ,
> 
> And I read the mail list ,that you had mentioned the below things.
> 
> Last as few upstreamed PMDs as example:
> 
> ionic:
> 
> https://patches.dpdk.org/project/dpdk/list/?series=8206&state=%2A&archive=both 
> <https://patches.dpdk.org/project/dpdk/list/?series=8206&state=%2A&archive=both>
> 
> pfe:
> 
> https://patches.dpdk.org/project/dpdk/list/?series=6777&state=%2A&archive=both 
> <https://patches.dpdk.org/project/dpdk/list/?series=6777&state=%2A&archive=both>
> 
> octeontx2 (one of the good samples I suggest checking):
> 
> https://patches.dpdk.org/project/dpdk/list/?series=5302&state=%2A&archive=both 
> <https://patches.dpdk.org/project/dpdk/list/?series=5302&state=%2A&archive=both>
> 
> If I just follow this step by step
> 
> is that the right way?
> 
> Is there anything else I need to do ?
> 
> Hope your kindness back.:）
> 
> Regards Wenbo
> 

Hi Wenbo,

It is good to hear that you decide to upstream your driver, welcome to the 
community.

Overall I suggest reading following documentation:
- Contributing Code to DPDK
   https://doc.dpdk.org/guides/contributing/patches.html

- DPDK Coding Style
   https://doc.dpdk.org/guides/contributing/coding_style.html


Splitting the driver into more logical and easy consumable patches can help all, 
as samples given in above drivers.

And please try to upstream regularly and in small chunks, instead of 
accumulating many changes and sending them in one go as a big patch.

There are multiple automated tests are running on the sent patches, please check 
status of your patches after you send them, and if some warnings/errors detected 
you can send an updated version until all issues fixed.

It is possible to manage your patches from patchwork, update their status as new 
patches arrived etc..
https://patches.dpdk.org/project/dpdk/
Please be sure you are registered to mail list and patchwork.

And please be sure you have adequate documentation with your driver, including 
the product links etc..

These are all I can think of as of now, cc'ed a few more folks for the things I 
missed, welcome again :)