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

[Guo, Junfeng]

> -----Original Message-----
> From: Ferruh Yigit <ferruh.yigit at amd.com>
> Sent: Friday, April 14, 2023 23:40
> To: Guo, Junfeng <junfeng.guo at intel.com>; Rushil Gupta
> <rushilg at google.com>; Zhang, Qi Z <qi.z.zhang at intel.com>; Wu, Jingjing
> <jingjing.wu at intel.com>
> Cc: joshwash at google.com; dev at dpdk.org; Jeroen de Borst
> <jeroendb at google.com>; Aaron Conole <aconole at redhat.com>;
> dpdklab <dpdklab at 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!