[21/21] net/ixgbe/base: update version

Message ID 20200612032410.20864-22-guinanx.sun@intel.com (mailing list archive)
State Superseded, archived
Delegated to: Qi Zhang
Headers
Series update ixgbe base code |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/travis-robot success Travis build: passed
ci/Intel-compilation success Compilation OK

Commit Message

Guinan Sun June 12, 2020, 3:24 a.m. UTC
  Update base code version in readme.

Signed-off-by: Guinan Sun <guinanx.sun@intel.com>
---
 drivers/net/ixgbe/base/README | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Ferruh Yigit June 22, 2020, 12:01 p.m. UTC | #1
On 6/12/2020 4:24 AM, Guinan Sun wrote:
> Update base code version in readme.
> 
> Signed-off-by: Guinan Sun <guinanx.sun@intel.com>
> ---
>  drivers/net/ixgbe/base/README | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ixgbe/base/README b/drivers/net/ixgbe/base/README
> index a48b14ed2..c8d6b39ac 100644
> --- a/drivers/net/ixgbe/base/README
> +++ b/drivers/net/ixgbe/base/README
> @@ -6,7 +6,7 @@ Intel® IXGBE driver
>  ===================
>  
>  This directory contains source code of FreeBSD ixgbe driver of version
> -cid-ixgbe.2018.08.28.tar.gz released by the team which develop
> +not-released-cid-ixgbe.2020.06.05.tar.gz released by the team which develop

What does 'not-released' mean?
Since this is to clarify the version of the base code, why not just
'cid-ixgbe.2020.06.05.tar.gz'?

>  basic drivers for any ixgbe NIC. The sub-directory of base/
>  contains the original source package.
>  This driver is valid for the product(s) listed below
>
  
Guinan Sun July 1, 2020, 6:27 a.m. UTC | #2
Hi Ferruh

> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Monday, June 22, 2020 8:01 PM
> To: Sun, GuinanX <guinanx.sun@intel.com>; dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH 21/21] net/ixgbe/base: update version
> 
> On 6/12/2020 4:24 AM, Guinan Sun wrote:
> > Update base code version in readme.
> >
> > Signed-off-by: Guinan Sun <guinanx.sun@intel.com>
> > ---
> >  drivers/net/ixgbe/base/README | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/ixgbe/base/README
> > b/drivers/net/ixgbe/base/README index a48b14ed2..c8d6b39ac 100644
> > --- a/drivers/net/ixgbe/base/README
> > +++ b/drivers/net/ixgbe/base/README
> > @@ -6,7 +6,7 @@ Intel® IXGBE driver
> >  ===================
> >
> >  This directory contains source code of FreeBSD ixgbe driver of
> > version -cid-ixgbe.2018.08.28.tar.gz released by the team which
> > develop
> > +not-released-cid-ixgbe.2020.06.05.tar.gz released by the team which
> > +develop
> 
> What does 'not-released' mean?

not-released means they are drops of shared code to the DPDK team.

> Since this is to clarify the version of the base code, why not just 'cid-
> ixgbe.2020.06.05.tar.gz'?

The drop package is named like this, not our modification.

> 
> >  basic drivers for any ixgbe NIC. The sub-directory of base/  contains
> > the original source package.
> >  This driver is valid for the product(s) listed below
> >
  

Patch

diff --git a/drivers/net/ixgbe/base/README b/drivers/net/ixgbe/base/README
index a48b14ed2..c8d6b39ac 100644
--- a/drivers/net/ixgbe/base/README
+++ b/drivers/net/ixgbe/base/README
@@ -6,7 +6,7 @@  Intel® IXGBE driver
 ===================
 
 This directory contains source code of FreeBSD ixgbe driver of version
-cid-ixgbe.2018.08.28.tar.gz released by the team which develop
+not-released-cid-ixgbe.2020.06.05.tar.gz released by the team which develop
 basic drivers for any ixgbe NIC. The sub-directory of base/
 contains the original source package.
 This driver is valid for the product(s) listed below