diff mbox

[dpdk-dev,v2,8/8] net/qede: update PMD version to 2.5.1.1

Message ID 1498937402-25547-9-git-send-email-rasesh.mody@cavium.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Mody, Rasesh July 1, 2017, 7:30 p.m. UTC 
  Update the PMD version and add 32bit support in documentation.

Signed-off-by: Rasesh Mody <rasesh.mody@cavium.com>
---
 doc/guides/nics/features/qede.ini |    1 +
 drivers/net/qede/qede_ethdev.h    |    2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

Comments

Ferruh Yigit July 3, 2017, 10:06 a.m. UTC | #1 
On 7/1/2017 8:30 PM, Rasesh Mody wrote:
> Update the PMD version and add 32bit support in documentation.
> 
> Signed-off-by: Rasesh Mody <rasesh.mody@cavium.com>
> ---
>  doc/guides/nics/features/qede.ini |    1 +
>  drivers/net/qede/qede_ethdev.h    |    2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/doc/guides/nics/features/qede.ini b/doc/guides/nics/features/qede.ini
> index fba5dc3..43caf94 100644
> --- a/doc/guides/nics/features/qede.ini
> +++ b/doc/guides/nics/features/qede.ini
> @@ -32,6 +32,7 @@ Extended stats       = Y
>  Stats per queue      = Y
>  Multiprocess aware   = Y
>  Linux UIO            = Y
> +x86-32               = Y

Can you please extract this into separate patch?
And since you are updating supported architectures, can you please add
others too, I assume qede supports arm?

>  x86-64               = Y
>  Usage doc            = Y
>  N-tuple filter       = Y
Ferruh Yigit July 5, 2017, 9:57 a.m. UTC | #2 
On 7/3/2017 11:06 AM, Ferruh Yigit wrote:
> On 7/1/2017 8:30 PM, Rasesh Mody wrote:
>> Update the PMD version and add 32bit support in documentation.
>>
>> Signed-off-by: Rasesh Mody <rasesh.mody@cavium.com>
>> ---
>>  doc/guides/nics/features/qede.ini |    1 +
>>  drivers/net/qede/qede_ethdev.h    |    2 +-
>>  2 files changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/doc/guides/nics/features/qede.ini b/doc/guides/nics/features/qede.ini
>> index fba5dc3..43caf94 100644
>> --- a/doc/guides/nics/features/qede.ini
>> +++ b/doc/guides/nics/features/qede.ini
>> @@ -32,6 +32,7 @@ Extended stats       = Y
>>  Stats per queue      = Y
>>  Multiprocess aware   = Y
>>  Linux UIO            = Y
>> +x86-32               = Y
> 
> Can you please extract this into separate patch?
> And since you are updating supported architectures, can you please add
> others too, I assume qede supports arm?

Rasesh seems on vacation, to be able to include patchset into RC1, I
will drop this part from patch and will get the set.

Rasesh, Harish,

Can you please send the requested document patch before RC2?

Thanks,
ferruh

> 
>>  x86-64               = Y
>>  Usage doc            = Y
>>  N-tuple filter       = Y
>
diff mbox

Patch

diff --git a/doc/guides/nics/features/qede.ini b/doc/guides/nics/features/qede.ini
index fba5dc3..43caf94 100644
--- a/doc/guides/nics/features/qede.ini
+++ b/doc/guides/nics/features/qede.ini
@@ -32,6 +32,7 @@  Extended stats       = Y
 Stats per queue      = Y
 Multiprocess aware   = Y
 Linux UIO            = Y
+x86-32               = Y
 x86-64               = Y
 Usage doc            = Y
 N-tuple filter       = Y
diff --git a/drivers/net/qede/qede_ethdev.h b/drivers/net/qede/qede_ethdev.h
index 9a93286..510b6e2 100644
--- a/drivers/net/qede/qede_ethdev.h
+++ b/drivers/net/qede/qede_ethdev.h
@@ -50,7 +50,7 @@ 
 #define QEDE_PMD_VER_PREFIX		"QEDE PMD"
 #define QEDE_PMD_VERSION_MAJOR		2
 #define QEDE_PMD_VERSION_MINOR	        5
-#define QEDE_PMD_VERSION_REVISION       0
+#define QEDE_PMD_VERSION_REVISION       1
 #define QEDE_PMD_VERSION_PATCH	        1
 
 #define QEDE_PMD_VERSION qede_stringify(QEDE_PMD_VERSION_MAJOR) "."     \