patch 'net/i40e/base: fix blinking X722 with X557 PHY' has been queued to stable release 21.11.9

[Kevin Traynor]

Hi,

FYI, your patch has been queued to stable release 21.11.9

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 12/02/24. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://github.com/kevintraynor/dpdk-stable

This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable/commit/2e22461f86d824443724cb8cfcbbae2f5f6e7c71

Thanks.

Kevin

---
>From 2e22461f86d824443724cb8cfcbbae2f5f6e7c71 Mon Sep 17 00:00:00 2001
From: Eryk Rybak <eryk.roch.rybak at intel.com>
Date: Mon, 2 Sep 2024 10:54:23 +0100
Subject: [PATCH] net/i40e/base: fix blinking X722 with X557 PHY

[ upstream commit bf0183e9ab98c946e0c7e178149e4b685465b9b1 ]

On x722 with x557 PHY LEDs do not blink under certain circumstances,
because the function was attempting to avoid triggering LED activity when
it detected that LED was already active. Fix it to just always trigger
LED blinking regardless of the LED state.

Fixes: 8db9e2a1b232 ("i40e: base driver")

Signed-off-by: Eryk Rybak <eryk.roch.rybak at intel.com>
Signed-off-by: Anatoly Burakov <anatoly.burakov at intel.com>
Acked-by: Bruce Richardson <bruce.richardson at intel.com>
---
 drivers/net/i40e/base/i40e_common.c | 32 -----------------------------
 1 file changed, 32 deletions(-)

diff --git a/drivers/net/i40e/base/i40e_common.c b/drivers/net/i40e/base/i40e_common.c
index 4d8167692d..d444c4a78e 100644
--- a/drivers/net/i40e/base/i40e_common.c
+++ b/drivers/net/i40e/base/i40e_common.c
@@ -1586,5 +1586,4 @@ static u32 i40e_led_is_mine(struct i40e_hw *hw, int idx)
 u32 i40e_led_get(struct i40e_hw *hw)
 {
-	u32 current_mode = 0;
 	u32 mode = 0;
 	int i;
@@ -1599,19 +1598,4 @@ u32 i40e_led_get(struct i40e_hw *hw)
 			continue;
 
-		/* ignore gpio LED src mode entries related to the activity
-		 *  LEDs
-		 */
-		current_mode = ((gpio_val & I40E_GLGEN_GPIO_CTL_LED_MODE_MASK)
-				>> I40E_GLGEN_GPIO_CTL_LED_MODE_SHIFT);
-		switch (current_mode) {
-		case I40E_COMBINED_ACTIVITY:
-		case I40E_FILTER_ACTIVITY:
-		case I40E_MAC_ACTIVITY:
-		case I40E_LINK_ACTIVITY:
-			continue;
-		default:
-			break;
-		}
-
 		mode = (gpio_val & I40E_GLGEN_GPIO_CTL_LED_MODE_MASK) >>
 			I40E_GLGEN_GPIO_CTL_LED_MODE_SHIFT;
@@ -1633,5 +1617,4 @@ u32 i40e_led_get(struct i40e_hw *hw)
 void i40e_led_set(struct i40e_hw *hw, u32 mode, bool blink)
 {
-	u32 current_mode = 0;
 	int i;
 
@@ -1650,19 +1633,4 @@ void i40e_led_set(struct i40e_hw *hw, u32 mode, bool blink)
 			continue;
 
-		/* ignore gpio LED src mode entries related to the activity
-		 * LEDs
-		 */
-		current_mode = ((gpio_val & I40E_GLGEN_GPIO_CTL_LED_MODE_MASK)
-				>> I40E_GLGEN_GPIO_CTL_LED_MODE_SHIFT);
-		switch (current_mode) {
-		case I40E_COMBINED_ACTIVITY:
-		case I40E_FILTER_ACTIVITY:
-		case I40E_MAC_ACTIVITY:
-		case I40E_LINK_ACTIVITY:
-			continue;
-		default:
-			break;
-		}
-
 		if (I40E_IS_X710TL_DEVICE(hw->device_id)) {
 			u32 pin_func = 0;
-- 
2.47.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2024-11-27 17:17:39.363363194 +0000
+++ 0034-net-i40e-base-fix-blinking-X722-with-X557-PHY.patch	2024-11-27 17:17:38.197269228 +0000
@@ -1 +1 @@
-From bf0183e9ab98c946e0c7e178149e4b685465b9b1 Mon Sep 17 00:00:00 2001
+From 2e22461f86d824443724cb8cfcbbae2f5f6e7c71 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit bf0183e9ab98c946e0c7e178149e4b685465b9b1 ]
+
@@ -12 +13,0 @@
-Cc: stable at dpdk.org
@@ -22 +23 @@
-index be27cc9d0b..80500697ed 100644
+index 4d8167692d..d444c4a78e 100644
@@ -25 +26 @@
-@@ -1549,5 +1549,4 @@ static u32 i40e_led_is_mine(struct i40e_hw *hw, int idx)
+@@ -1586,5 +1586,4 @@ static u32 i40e_led_is_mine(struct i40e_hw *hw, int idx)
@@ -31 +32 @@
-@@ -1562,19 +1561,4 @@ u32 i40e_led_get(struct i40e_hw *hw)
+@@ -1599,19 +1598,4 @@ u32 i40e_led_get(struct i40e_hw *hw)
@@ -51 +52 @@
-@@ -1596,5 +1580,4 @@ u32 i40e_led_get(struct i40e_hw *hw)
+@@ -1633,5 +1617,4 @@ u32 i40e_led_get(struct i40e_hw *hw)
@@ -57 +58 @@
-@@ -1613,19 +1596,4 @@ void i40e_led_set(struct i40e_hw *hw, u32 mode, bool blink)
+@@ -1650,19 +1633,4 @@ void i40e_led_set(struct i40e_hw *hw, u32 mode, bool blink)