> -----Original Message----- > From: Ferruh Yigit <ferruh.yi...@amd.com> > Sent: Friday, April 14, 2023 23:40 > To: Guo, Junfeng <junfeng....@intel.com>; Rushil Gupta > <rush...@google.com>; Zhang, Qi Z <qi.z.zh...@intel.com>; Wu, Jingjing > <jingjing...@intel.com> > Cc: joshw...@google.com; dev@dpdk.org; Jeroen de Borst > <jeroe...@google.com>; Aaron Conole <acon...@redhat.com>; > dpdklab <dpdk...@iol.unh.edu> > 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!
