Download raw body.
em(4): simplify mac type tests
On Wed, Aug 05, 2026 at 12:19:29PM +1000, Jonathan Gray wrote:
> reduce the number of places to change when adding mac types
Makes sense to me. The em_clear_hw_cntrs() bit looks weird, but
I think we've been getting that wrong since pch2lan was added, and
this change fixes it. ok jmatthew@
>
> diff --git sys/dev/pci/if_em.c sys/dev/pci/if_em.c
> index db3aeab21ac..5121269ef22 100644
> --- sys/dev/pci/if_em.c
> +++ sys/dev/pci/if_em.c
> @@ -1684,8 +1684,7 @@ em_legacy_irq_quirk_spt(struct em_softc *sc)
> uint32_t reg;
>
> /* Legacy interrupt: SPT needs a quirk. */
> - if (sc->hw.mac_type != em_pch_spt && sc->hw.mac_type != em_pch_cnp &&
> - sc->hw.mac_type != em_pch_tgp && sc->hw.mac_type != em_pch_adp)
> + if (sc->hw.mac_type < em_pch_spt)
> return;
> if (sc->legacy_irq == 0)
> return;
> diff --git sys/dev/pci/if_em_hw.c sys/dev/pci/if_em_hw.c
> index 541ea943d86..0d2c0869df2 100644
> --- sys/dev/pci/if_em_hw.c
> +++ sys/dev/pci/if_em_hw.c
> @@ -1582,13 +1582,7 @@ em_init_hw(struct em_softc *sc)
> E1000_WRITE_REG(hw, STATUS, reg_data);
> }
>
> - if (hw->mac_type == em_pchlan ||
> - hw->mac_type == em_pch2lan ||
> - hw->mac_type == em_pch_lpt ||
> - hw->mac_type == em_pch_spt ||
> - hw->mac_type == em_pch_cnp ||
> - hw->mac_type == em_pch_tgp ||
> - hw->mac_type == em_pch_adp) {
> + if (hw->mac_type >= em_pchlan) {
> /*
> * The MAC-PHY interconnect may still be in SMBus mode
> * after Sx->S0. Toggle the LANPHYPC Value bit to force
> @@ -2428,13 +2422,7 @@ em_copper_link_igp_setup(struct em_hw *hw)
> }
> }
> /* disable lplu d0 during driver init */
> - if (hw->mac_type == em_pchlan ||
> - hw->mac_type == em_pch2lan ||
> - hw->mac_type == em_pch_lpt ||
> - hw->mac_type == em_pch_spt ||
> - hw->mac_type == em_pch_cnp ||
> - hw->mac_type == em_pch_tgp ||
> - hw->mac_type == em_pch_adp)
> + if (hw->mac_type >= em_pchlan)
> ret_val = em_set_lplu_state_pchlan(hw, FALSE);
> else
> ret_val = em_set_d0_lplu_state(hw, FALSE);
> @@ -2704,13 +2692,7 @@ em_copper_link_mgp_setup(struct em_hw *hw)
> return E1000_SUCCESS;
>
> /* disable lplu d0 during driver init */
> - if (hw->mac_type == em_pchlan ||
> - hw->mac_type == em_pch2lan ||
> - hw->mac_type == em_pch_lpt ||
> - hw->mac_type == em_pch_spt ||
> - hw->mac_type == em_pch_cnp ||
> - hw->mac_type == em_pch_tgp ||
> - hw->mac_type == em_pch_adp)
> + if (hw->mac_type >= em_pchlan)
> ret_val = em_set_lplu_state_pchlan(hw, FALSE);
>
> /* Enable CRS on TX. This must be set for half-duplex operation. */
> @@ -4351,12 +4333,7 @@ em_check_for_link(struct em_hw *hw)
> em_check_downshift(hw);
>
> /* Enable/Disable EEE after link up */
> - if (hw->mac_type == em_pch2lan ||
> - hw->mac_type == em_pch_lpt ||
> - hw->mac_type == em_pch_spt ||
> - hw->mac_type == em_pch_cnp ||
> - hw->mac_type == em_pch_tgp ||
> - hw->mac_type == em_pch_adp) {
> + if (hw->mac_type >= em_pch2lan) {
> ret_val = em_set_eee_pchlan(hw);
> if (ret_val)
> return ret_val;
> @@ -5139,13 +5116,7 @@ em_read_phy_reg(struct em_hw *hw, uint32_t reg_addr, uint16_t *phy_data)
> uint16_t swfw;
> DEBUGFUNC("em_read_phy_reg");
>
> - if (hw->mac_type == em_pchlan ||
> - hw->mac_type == em_pch2lan ||
> - hw->mac_type == em_pch_lpt ||
> - hw->mac_type == em_pch_spt ||
> - hw->mac_type == em_pch_cnp ||
> - hw->mac_type == em_pch_tgp ||
> - hw->mac_type == em_pch_adp)
> + if (hw->mac_type >= em_pchlan)
> return (em_access_phy_reg_hv(hw, reg_addr, phy_data, TRUE));
>
> if (((hw->mac_type == em_80003es2lan) || (hw->mac_type == em_82575) ||
> @@ -5267,9 +5238,7 @@ em_read_phy_reg_ex(struct em_hw *hw, uint32_t reg_addr, uint16_t *phy_data)
> }
> *phy_data = (uint16_t) mdic;
>
> - if (hw->mac_type == em_pch2lan || hw->mac_type == em_pch_lpt ||
> - hw->mac_type == em_pch_spt || hw->mac_type == em_pch_cnp ||
> - hw->mac_type == em_pch_tgp || hw->mac_type == em_pch_adp)
> + if (hw->mac_type >= em_pch2lan)
> usec_delay(100);
> } else {
> /*
> @@ -5318,13 +5287,7 @@ em_write_phy_reg(struct em_hw *hw, uint32_t reg_addr, uint16_t phy_data)
> uint32_t ret_val;
> DEBUGFUNC("em_write_phy_reg");
>
> - if (hw->mac_type == em_pchlan ||
> - hw->mac_type == em_pch2lan ||
> - hw->mac_type == em_pch_lpt ||
> - hw->mac_type == em_pch_spt ||
> - hw->mac_type == em_pch_cnp ||
> - hw->mac_type == em_pch_tgp ||
> - hw->mac_type == em_pch_adp)
> + if (hw->mac_type >= em_pchlan)
> return (em_access_phy_reg_hv(hw, reg_addr, &phy_data, FALSE));
>
> if (em_swfw_sync_acquire(hw, hw->swfw))
> @@ -5432,9 +5395,7 @@ em_write_phy_reg_ex(struct em_hw *hw, uint32_t reg_addr, uint16_t phy_data)
> return -E1000_ERR_PHY;
> }
>
> - if (hw->mac_type == em_pch2lan || hw->mac_type == em_pch_lpt ||
> - hw->mac_type == em_pch_spt || hw->mac_type == em_pch_cnp ||
> - hw->mac_type == em_pch_tgp || hw->mac_type == em_pch_adp)
> + if (hw->mac_type >= em_pch2lan)
> usec_delay(100);
> } else {
> /*
> @@ -7841,9 +7802,7 @@ em_init_rx_addrs(struct em_hw *hw)
> uint32_t rar_num;
> DEBUGFUNC("em_init_rx_addrs");
>
> - if (hw->mac_type == em_pch_lpt || hw->mac_type == em_pch_spt ||
> - hw->mac_type == em_pch_cnp || hw->mac_type == em_pch_tgp ||
> - hw->mac_type == em_pch_adp || hw->mac_type == em_pch2lan)
> + if (hw->mac_type >= em_pch2lan)
> if (em_phy_no_cable_workaround(hw))
> printf(" ...failed to apply em_phy_no_cable_"
> "workaround.\n");
> @@ -8359,13 +8318,7 @@ em_clear_hw_cntrs(struct em_hw *hw)
> em_read_phy_reg(hw, HV_TNCRS_LOWER, &phy_data);
> }
>
> - if (hw->mac_type == em_ich8lan ||
> - hw->mac_type == em_ich9lan ||
> - hw->mac_type == em_ich10lan ||
> - hw->mac_type == em_pchlan ||
> - (hw->mac_type != em_pch2lan && hw->mac_type != em_pch_lpt &&
> - hw->mac_type != em_pch_spt && hw->mac_type != em_pch_cnp &&
> - hw->mac_type != em_pch_tgp && hw->mac_type != em_pch_adp))
> + if (hw->mac_type >= em_ich8lan)
> return;
>
> temp = E1000_READ_REG(hw, ICRXPTC);
> @@ -11127,13 +11080,7 @@ em_init_lcd_from_nvm(struct em_hw *hw)
> /* Check if SW needs configure the PHY */
> if (hw->device_id == E1000_DEV_ID_ICH8_IGP_M_AMT ||
> hw->device_id == E1000_DEV_ID_ICH8_IGP_M ||
> - hw->mac_type == em_pchlan ||
> - hw->mac_type == em_pch2lan ||
> - hw->mac_type == em_pch_lpt ||
> - hw->mac_type == em_pch_spt ||
> - hw->mac_type == em_pch_cnp ||
> - hw->mac_type == em_pch_tgp ||
> - hw->mac_type == em_pch_adp)
> + hw->mac_type >= em_pchlan)
> sw_cfg_mask = FEXTNVM_SW_CONFIG_ICH8M;
> else
> sw_cfg_mask = FEXTNVM_SW_CONFIG;
> diff --git sys/dev/pci/if_em_hw.h sys/dev/pci/if_em_hw.h
> index 6b861c2620b..62abe8098b6 100644
> --- sys/dev/pci/if_em_hw.h
> +++ sys/dev/pci/if_em_hw.h
> @@ -88,11 +88,7 @@ typedef enum {
> em_num_macs
> } em_mac_type;
>
> -#define IS_ICH8(t) \
> - (t == em_ich8lan || t == em_ich9lan || t == em_ich10lan || \
> - t == em_pchlan || t == em_pch2lan || t == em_pch_lpt || \
> - t == em_pch_spt || t == em_pch_cnp || t == em_pch_tgp || \
> - t == em_pch_adp)
> +#define IS_ICH8(t) (t >= em_ich8lan)
>
> typedef enum {
> em_eeprom_uninitialized = 0,
>
em(4): simplify mac type tests