While the qed_lm_maps is closely tied with the QED_LM_* defines,
when iterating over the array use actual size instead of the qed
define to prevent future possible issues.

Signed-off-by: Yuval Mintz <yuval.mi...@cavium.com>
---
 drivers/net/ethernet/qlogic/qede/qede_ethtool.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/qlogic/qede/qede_ethtool.c 
b/drivers/net/ethernet/qlogic/qede/qede_ethtool.c
index 42d9739..d230742 100644
--- a/drivers/net/ethernet/qlogic/qede/qede_ethtool.c
+++ b/drivers/net/ethernet/qlogic/qede/qede_ethtool.c
@@ -320,7 +320,7 @@ struct qede_link_mode_mapping {
 {                                                              \
        int i;                                                  \
                                                                \
-       for (i = 0; i < QED_LM_COUNT; i++) {                    \
+       for (i = 0; i < ARRAY_SIZE(qed_lm_map); i++) {          \
                if ((caps) & (qed_lm_map[i].qed_link_mode))     \
                        __set_bit(qed_lm_map[i].ethtool_link_mode,\
                                  lk_ksettings->link_modes.name); \
@@ -331,7 +331,7 @@ struct qede_link_mode_mapping {
 {                                                              \
        int i;                                                  \
                                                                \
-       for (i = 0; i < QED_LM_COUNT; i++) {                    \
+       for (i = 0; i < ARRAY_SIZE(qed_lm_map); i++) {          \
                if (test_bit(qed_lm_map[i].ethtool_link_mode,   \
                             lk_ksettings->link_modes.name))    \
                        caps |= qed_lm_map[i].qed_link_mode;    \
-- 
1.9.3

Reply via email to