bnx2x: Re-arrange the link structures for better alignment
Change ieee_fc to u16 instead of u32 and re-arrange the link parameters structures Signed-off-by: Yitchak Gertner <gertner@broadcom.com> Signed-off-by: Eilon Greenstein <eilong@broadcom.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
committed by
David S. Miller
parent
8924665a7d
commit
1ef70b9c12
@@ -1347,7 +1347,7 @@ static void bnx2x_set_brcm_cl37_advertisment(struct link_params *params)
|
|||||||
MDIO_OVER_1G_UP3, 0x400);
|
MDIO_OVER_1G_UP3, 0x400);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void bnx2x_calc_ieee_aneg_adv(struct link_params *params, u32 *ieee_fc)
|
static void bnx2x_calc_ieee_aneg_adv(struct link_params *params, u16 *ieee_fc)
|
||||||
{
|
{
|
||||||
*ieee_fc = MDIO_COMBO_IEEE0_AUTO_NEG_ADV_FULL_DUPLEX;
|
*ieee_fc = MDIO_COMBO_IEEE0_AUTO_NEG_ADV_FULL_DUPLEX;
|
||||||
/* resolve pause mode and advertisement
|
/* resolve pause mode and advertisement
|
||||||
@@ -1381,7 +1381,7 @@ static void bnx2x_calc_ieee_aneg_adv(struct link_params *params, u32 *ieee_fc)
|
|||||||
}
|
}
|
||||||
|
|
||||||
static void bnx2x_set_ieee_aneg_advertisment(struct link_params *params,
|
static void bnx2x_set_ieee_aneg_advertisment(struct link_params *params,
|
||||||
u32 ieee_fc)
|
u16 ieee_fc)
|
||||||
{
|
{
|
||||||
struct bnx2x *bp = params->bp;
|
struct bnx2x *bp = params->bp;
|
||||||
/* for AN, we are always publishing full duplex */
|
/* for AN, we are always publishing full duplex */
|
||||||
@@ -1389,7 +1389,7 @@ static void bnx2x_set_ieee_aneg_advertisment(struct link_params *params,
|
|||||||
CL45_WR_OVER_CL22(bp, params->port,
|
CL45_WR_OVER_CL22(bp, params->port,
|
||||||
params->phy_addr,
|
params->phy_addr,
|
||||||
MDIO_REG_BANK_COMBO_IEEE0,
|
MDIO_REG_BANK_COMBO_IEEE0,
|
||||||
MDIO_COMBO_IEEE0_AUTO_NEG_ADV, (u16)ieee_fc);
|
MDIO_COMBO_IEEE0_AUTO_NEG_ADV, ieee_fc);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void bnx2x_restart_autoneg(struct link_params *params, u8 enable_cl73)
|
static void bnx2x_restart_autoneg(struct link_params *params, u8 enable_cl73)
|
||||||
|
@@ -81,6 +81,11 @@ struct link_params {
|
|||||||
#define SWITCH_CFG_AUTO_DETECT PORT_FEATURE_CON_SWITCH_AUTO_DETECT
|
#define SWITCH_CFG_AUTO_DETECT PORT_FEATURE_CON_SWITCH_AUTO_DETECT
|
||||||
|
|
||||||
u16 hw_led_mode; /* part of the hw_config read from the shmem */
|
u16 hw_led_mode; /* part of the hw_config read from the shmem */
|
||||||
|
|
||||||
|
/* phy_addr populated by the phy_init function */
|
||||||
|
u8 phy_addr;
|
||||||
|
/*u8 reserved1;*/
|
||||||
|
|
||||||
u32 lane_config;
|
u32 lane_config;
|
||||||
u32 ext_phy_config;
|
u32 ext_phy_config;
|
||||||
#define XGXS_EXT_PHY_TYPE(ext_phy_config) (ext_phy_config & \
|
#define XGXS_EXT_PHY_TYPE(ext_phy_config) (ext_phy_config & \
|
||||||
@@ -90,39 +95,41 @@ struct link_params {
|
|||||||
/* Phy register parameter */
|
/* Phy register parameter */
|
||||||
u32 chip_id;
|
u32 chip_id;
|
||||||
|
|
||||||
/* phy_addr populated by the CLC */
|
|
||||||
u8 phy_addr;
|
|
||||||
u16 xgxs_config_rx[4]; /* preemphasis values for the rx side */
|
u16 xgxs_config_rx[4]; /* preemphasis values for the rx side */
|
||||||
|
|
||||||
u16 xgxs_config_tx[4]; /* preemphasis values for the tx side */
|
u16 xgxs_config_tx[4]; /* preemphasis values for the tx side */
|
||||||
|
|
||||||
u32 feature_config_flags;
|
u32 feature_config_flags;
|
||||||
#define FEATURE_CONFIG_OVERRIDE_PREEMPHASIS_ENABLED (1<<0)
|
#define FEATURE_CONFIG_OVERRIDE_PREEMPHASIS_ENABLED (1<<0)
|
||||||
#define FEATURE_CONFIG_BC_SUPPORTS_OPT_MDL_VRFY (1<<2)
|
#define FEATURE_CONFIG_BC_SUPPORTS_OPT_MDL_VRFY (1<<2)
|
||||||
#define FEATURE_CONFIG_BCM8727_NOC (1<<3)
|
#define FEATURE_CONFIG_BCM8727_NOC (1<<3)
|
||||||
|
|
||||||
/* Device pointer passed to all callback functions */
|
/* Device pointer passed to all callback functions */
|
||||||
struct bnx2x *bp;
|
struct bnx2x *bp;
|
||||||
};
|
};
|
||||||
|
|
||||||
/* Output parameters */
|
/* Output parameters */
|
||||||
struct link_vars {
|
struct link_vars {
|
||||||
|
u8 phy_flags;
|
||||||
|
|
||||||
|
u8 mac_type;
|
||||||
|
#define MAC_TYPE_NONE 0
|
||||||
|
#define MAC_TYPE_EMAC 1
|
||||||
|
#define MAC_TYPE_BMAC 2
|
||||||
|
|
||||||
u8 phy_link_up; /* internal phy link indication */
|
u8 phy_link_up; /* internal phy link indication */
|
||||||
u8 link_up;
|
u8 link_up;
|
||||||
u16 duplex;
|
|
||||||
u16 flow_ctrl;
|
|
||||||
u32 ieee_fc;
|
|
||||||
u8 mac_type;
|
|
||||||
|
|
||||||
#define MAC_TYPE_NONE 0
|
|
||||||
#define MAC_TYPE_EMAC 1
|
|
||||||
#define MAC_TYPE_BMAC 2
|
|
||||||
u16 line_speed;
|
u16 line_speed;
|
||||||
|
u16 duplex;
|
||||||
|
|
||||||
|
u16 flow_ctrl;
|
||||||
|
u16 ieee_fc;
|
||||||
|
|
||||||
u32 autoneg;
|
u32 autoneg;
|
||||||
#define AUTO_NEG_DISABLED 0x0
|
#define AUTO_NEG_DISABLED 0x0
|
||||||
#define AUTO_NEG_ENABLED 0x1
|
#define AUTO_NEG_ENABLED 0x1
|
||||||
#define AUTO_NEG_COMPLETE 0x2
|
#define AUTO_NEG_COMPLETE 0x2
|
||||||
#define AUTO_NEG_PARALLEL_DETECTION_USED 0x3
|
#define AUTO_NEG_PARALLEL_DETECTION_USED 0x3
|
||||||
|
|
||||||
u8 phy_flags;
|
|
||||||
|
|
||||||
/* The same definitions as the shmem parameter */
|
/* The same definitions as the shmem parameter */
|
||||||
u32 link_status;
|
u32 link_status;
|
||||||
|
Reference in New Issue
Block a user