> -----Original Message----- > From: Ferruh Yigit <[email protected]> > Sent: Friday, April 14, 2023 23:40 > To: Guo, Junfeng <[email protected]>; Rushil Gupta > <[email protected]>; Zhang, Qi Z <[email protected]>; Wu, Jingjing > <[email protected]> > Cc: [email protected]; [email protected]; Jeroen de Borst > <[email protected]>; Aaron Conole <[email protected]>; > dpdklab <[email protected]> > Subject: Re: [PATCH] net/gve: Check whether the driver is compatible > with the device presented. > > 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.
Sure, we will try this in the coming version patches. Really a good solution for the patch dependencies! Thanks for the enhancement!
