Commit f9c058b6 authored by Dmitry Kravkov's avatar Dmitry Kravkov Committed by David S. Miller

bnx2x: DCBX: use #define instead of magic

Signed-off-by: default avatarDmitry Kravkov <dmitry@broadcom.com>
Signed-off-by: default avatarEilon Greenstein <eilong@broadcom.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 00253a8c
...@@ -874,7 +874,7 @@ static void bnx2x_dcbx_admin_mib_updated_params(struct bnx2x *bp, ...@@ -874,7 +874,7 @@ static void bnx2x_dcbx_admin_mib_updated_params(struct bnx2x *bp,
/*For IEEE admin_recommendation_bw_precentage /*For IEEE admin_recommendation_bw_precentage
*For IEEE admin_recommendation_ets_pg */ *For IEEE admin_recommendation_ets_pg */
af->pfc.pri_en_bitmap = (u8)dp->admin_pfc_bitmap; af->pfc.pri_en_bitmap = (u8)dp->admin_pfc_bitmap;
for (i = 0; i < 4; i++) { for (i = 0; i < DCBX_CONFIG_MAX_APP_PROTOCOL; i++) {
if (dp->admin_priority_app_table[i].valid) { if (dp->admin_priority_app_table[i].valid) {
struct bnx2x_admin_priority_app_table *table = struct bnx2x_admin_priority_app_table *table =
dp->admin_priority_app_table; dp->admin_priority_app_table;
...@@ -2249,7 +2249,7 @@ static int bnx2x_set_admin_app_up(struct bnx2x *bp, u8 idtype, u16 idval, u8 up) ...@@ -2249,7 +2249,7 @@ static int bnx2x_set_admin_app_up(struct bnx2x *bp, u8 idtype, u16 idval, u8 up)
int i, ff; int i, ff;
/* iterate over the app entries looking for idtype and idval */ /* iterate over the app entries looking for idtype and idval */
for (i = 0, ff = -1; i < 4; i++) { for (i = 0, ff = -1; i < DCBX_CONFIG_MAX_APP_PROTOCOL; i++) {
struct bnx2x_admin_priority_app_table *app_ent = struct bnx2x_admin_priority_app_table *app_ent =
&bp->dcbx_config_params.admin_priority_app_table[i]; &bp->dcbx_config_params.admin_priority_app_table[i];
if (bnx2x_admin_app_is_equal(app_ent, idtype, idval)) if (bnx2x_admin_app_is_equal(app_ent, idtype, idval))
...@@ -2258,7 +2258,7 @@ static int bnx2x_set_admin_app_up(struct bnx2x *bp, u8 idtype, u16 idval, u8 up) ...@@ -2258,7 +2258,7 @@ static int bnx2x_set_admin_app_up(struct bnx2x *bp, u8 idtype, u16 idval, u8 up)
if (ff < 0 && !app_ent->valid) if (ff < 0 && !app_ent->valid)
ff = i; ff = i;
} }
if (i < 4) if (i < DCBX_CONFIG_MAX_APP_PROTOCOL)
/* if found overwrite up */ /* if found overwrite up */
bp->dcbx_config_params. bp->dcbx_config_params.
admin_priority_app_table[i].priority = up; admin_priority_app_table[i].priority = up;
......
...@@ -90,6 +90,7 @@ struct bnx2x_admin_priority_app_table { ...@@ -90,6 +90,7 @@ struct bnx2x_admin_priority_app_table {
u32 app_id; u32 app_id;
}; };
#define DCBX_CONFIG_MAX_APP_PROTOCOL 4
struct bnx2x_config_dcbx_params { struct bnx2x_config_dcbx_params {
u32 overwrite_settings; u32 overwrite_settings;
u32 admin_dcbx_version; u32 admin_dcbx_version;
...@@ -109,7 +110,8 @@ struct bnx2x_config_dcbx_params { ...@@ -109,7 +110,8 @@ struct bnx2x_config_dcbx_params {
u32 admin_recommendation_bw_precentage[8]; u32 admin_recommendation_bw_precentage[8];
u32 admin_recommendation_ets_pg[8]; u32 admin_recommendation_ets_pg[8];
u32 admin_pfc_bitmap; u32 admin_pfc_bitmap;
struct bnx2x_admin_priority_app_table admin_priority_app_table[4]; struct bnx2x_admin_priority_app_table
admin_priority_app_table[DCBX_CONFIG_MAX_APP_PROTOCOL];
u32 admin_default_priority; u32 admin_default_priority;
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment