[dpdk-dev] i40e: fix wrong copy-paste which led to one bit missing from I40E_INSET_FLEX_PAYLOAD mask

Message ID 1449071533-9023-1-git-send-email-andrey.chilikin@intel.com (mailing list archive)
State Accepted, archived
Headers

Commit Message

Chilikin, Andrey Dec. 2, 2015, 3:52 p.m. UTC
  Fix wrong copy-paste which led to one bit missing from I40E_INSET_FLEX_PAYLOAD mask

Signed-off-by: Andrey Chilikin <andrey.chilikin@intel.com>
---
 drivers/net/i40e/i40e_ethdev.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
  

Comments

Thomas Monjalon Dec. 6, 2015, 9:30 p.m. UTC | #1
2015-12-02 15:52, Andrey Chilikin:
> Fix wrong copy-paste which led to one bit missing from I40E_INSET_FLEX_PAYLOAD mask
> 
> Signed-off-by: Andrey Chilikin <andrey.chilikin@intel.com>

Applied, thanks
  
Zhang, Helin Dec. 7, 2015, 12:55 a.m. UTC | #2
> -----Original Message-----
> From: Chilikin, Andrey
> Sent: Wednesday, December 2, 2015 11:52 PM
> To: dev@dpdk.org
> Cc: Zhang, Helin; Wu, Jingjing; Chilikin, Andrey
> Subject: [PATCH] i40e: fix wrong copy-paste which led to one bit missing
> from I40E_INSET_FLEX_PAYLOAD mask
> 
> Fix wrong copy-paste which led to one bit missing from
> I40E_INSET_FLEX_PAYLOAD mask
> 
> Signed-off-by: Andrey Chilikin <andrey.chilikin@intel.com>
Acked-by: Helin Zhang <helin.zhang@intel.com>

Though applied, I'd like to ack it here, and thanks to Andrey's good catch!
  

Patch

diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c
index 5cd6e88..7e03a1f 100644
--- a/drivers/net/i40e/i40e_ethdev.c
+++ b/drivers/net/i40e/i40e_ethdev.c
@@ -194,7 +194,7 @@ 
 #define I40E_INSET_FLEX_PAYLOAD_W8 0x8000000000000000ULL
 #define I40E_INSET_FLEX_PAYLOAD \
 	(I40E_INSET_FLEX_PAYLOAD_W1 | I40E_INSET_FLEX_PAYLOAD_W2 | \
-	I40E_INSET_FLEX_PAYLOAD_W3 | I40E_INSET_FLEX_PAYLOAD_W3 | \
+	I40E_INSET_FLEX_PAYLOAD_W3 | I40E_INSET_FLEX_PAYLOAD_W4 | \
 	I40E_INSET_FLEX_PAYLOAD_W5 | I40E_INSET_FLEX_PAYLOAD_W6 | \
 	I40E_INSET_FLEX_PAYLOAD_W7 | I40E_INSET_FLEX_PAYLOAD_W8)