Re: [PATCH] net/gve: Check whether the driver is compatible with the device presented.

[Ferruh Yigit <ferruh.yigit@amd.com>]

On 4/14/2023 10:05 AM, Guo, Junfeng wrote:
> Hi Ferruh,
> 
> Here is the summary of the dependency of related patches:
> 
> P1: necessary headers update
>     https://patchwork.dpdk.org/project/dpdk/list/?series=27647&state=*
> P2: minor update for base code
>     https://patchwork.dpdk.org/project/dpdk/list/?series=27653&state=*
> P3: gve PMD enhancement
>     https://patchwork.dpdk.org/project/dpdk/list/?series=27687&state=*
> P4: add new AdminQ cmd to verify drv cap
>     https://patchwork.dpdk.org/project/dpdk/list/?series=27703&state=*
> 
> The merging order could be: P1 > P2 > P3 > P4, due to the dependency.
> Just to inform you about this here. Thanks for the review!
> 

Thanks Junfeng, this is useful.

Btw, we have a defined syntax for this [1], having it in commit log may
help automated CI to parse it and apply in correct order (when it is
fully supported in CI :) )


[1]
https://doc.dpdk.org/guides/contributing/patches.html#patch-dependencies

Depends-on: series-NNNNN ("Title of the series")
or
Depends-on: patch-NNNNN ("Title of the patch")

And I just recognized that syntax is not clear for multiple dependency
case, we should define it better.