#define        CLKD_OTP                0x000f0000
 #define        CLKD_OTP_SHIFT          16
 
-/* Package IDs */
-#define        BCM4717_PKG_ID          9       /* 4717 package id */
-#define        BCM4718_PKG_ID          10      /* 4718 package id */
-#define BCM43224_FAB_SMIC      0xa     /* the chip is manufactured by SMIC */
-
-/* these are router chips */
-#define        BCM4716_CHIP_ID         0x4716  /* 4716 chipcommon chipid */
-#define        BCM47162_CHIP_ID        47162   /* 47162 chipcommon chipid */
-#define        BCM4748_CHIP_ID         0x4748  /* 4716 chipcommon chipid (OTP, RBBU) */
-
 /* dynamic clock control defines */
 #define        LPOMINFREQ              25000   /* low power oscillator min */
 #define        LPOMAXFREQ              43000   /* low power oscillator max */
 
 {
        struct bcma_device *core = wlc_hw->d11core;
 
-       if ((ai_get_chip_id(wlc_hw->sih) == BCM43224_CHIP_ID) ||
-           (ai_get_chip_id(wlc_hw->sih) == BCM43225_CHIP_ID)) {
+       if ((ai_get_chip_id(wlc_hw->sih) == BCMA_CHIP_ID_BCM43224) ||
+           (ai_get_chip_id(wlc_hw->sih) == BCMA_CHIP_ID_BCM43225)) {
                if (spurmode == WL_SPURAVOID_ON2) {     /* 126Mhz */
                        bcma_write16(core, D11REGOFFS(tsf_clk_frac_l), 0x2082);
                        bcma_write16(core, D11REGOFFS(tsf_clk_frac_h), 0x8);
        tmp = 0;
 
        if (on) {
-               if ((ai_get_chip_id(wlc_hw->sih) == BCM4313_CHIP_ID)) {
+               if ((ai_get_chip_id(wlc_hw->sih) == BCMA_CHIP_ID_BCM4313)) {
                        bcma_set32(core, D11REGOFFS(clk_ctl_st),
                                   CCS_ERSRC_REQ_HT |
                                   CCS_ERSRC_REQ_D11PLL |
        else
                wlc_hw->_nbands = 1;
 
-       if ((ai_get_chip_id(wlc_hw->sih) == BCM43225_CHIP_ID))
+       if ((ai_get_chip_id(wlc_hw->sih) == BCMA_CHIP_ID_BCM43225))
                wlc_hw->_nbands = 1;
 
        /* BMAC_NOTE: remove init of pub values when brcms_c_attach()
        wlc_hw->wlc->pub->hw_up = true;
 
        if ((wlc_hw->boardflags & BFL_FEM)
-           && (ai_get_chip_id(wlc_hw->sih) == BCM4313_CHIP_ID)) {
+           && (ai_get_chip_id(wlc_hw->sih) == BCMA_CHIP_ID_BCM4313)) {
                if (!
                    (wlc_hw->boardrev >= 0x1250
                     && (wlc_hw->boardflags & BFL_FEM_BT)))
        }
 
        if ((wlc->pub->boardflags & BFL_FEM)
-           && (ai_get_chip_id(wlc->hw->sih) == BCM4313_CHIP_ID)) {
+           && (ai_get_chip_id(wlc->hw->sih) == BCMA_CHIP_ID_BCM4313)) {
                if (wlc->pub->boardrev >= 0x1250
                    && (wlc->pub->boardflags & BFL_FEM_BT))
                        brcms_b_mhf(wlc->hw, MHF5, MHF5_4313_GPIOCTRL,
 
        pi->tbl_data_hi = tblDataHi;
        pi->tbl_data_lo = tblDataLo;
 
-       if (pi->sh->chip == BCM43224_CHIP_ID &&
+       if (pi->sh->chip == BCMA_CHIP_ID_BCM43224 &&
            pi->sh->chiprev == 1) {
                pi->tbl_addr = tblAddr;
                pi->tbl_save_id = tbl_id;
 
 void wlc_phy_table_data_write(struct brcms_phy *pi, uint width, u32 val)
 {
-       if ((pi->sh->chip == BCM43224_CHIP_ID) &&
+       if ((pi->sh->chip == BCMA_CHIP_ID_BCM43224) &&
            (pi->sh->chiprev == 1) &&
            (pi->tbl_save_id == NPHY_TBL_ID_ANTSWCTRLLUT)) {
                read_phy_reg(pi, pi->tbl_data_lo);
 
        for (idx = 0; idx < ptbl_info->tbl_len; idx++) {
 
-               if ((pi->sh->chip == BCM43224_CHIP_ID) &&
+               if ((pi->sh->chip == BCMA_CHIP_ID_BCM43224) &&
                    (pi->sh->chiprev == 1) &&
                    (tbl_id == NPHY_TBL_ID_ANTSWCTRLLUT)) {
                        read_phy_reg(pi, tblDataLo);
 
        for (idx = 0; idx < ptbl_info->tbl_len; idx++) {
 
-               if ((pi->sh->chip == BCM43224_CHIP_ID) &&
+               if ((pi->sh->chip == BCMA_CHIP_ID_BCM43224) &&
                    (pi->sh->chiprev == 1)) {
                        (void)read_phy_reg(pi, tblDataLo);
 
 
 void wlc_lcnphy_epa_switch(struct brcms_phy *pi, bool mode)
 {
-       if ((pi->sh->chip == BCM4313_CHIP_ID) &&
+       if ((pi->sh->chip == BCMA_CHIP_ID_BCM4313) &&
            (pi->sh->boardflags & BFL_FEM)) {
                if (mode) {
                        u16 txant = 0;
 
                pi->measure_hold |= PHY_HOLD_FOR_NOT_ASSOC;
 
        if ((ISNPHY(pi)) && (NREV_GE(pi->pubpi.phy_rev, 5)) &&
-           ((pi->sh->chippkg == BCM4717_PKG_ID) ||
-            (pi->sh->chippkg == BCM4718_PKG_ID))) {
+           ((pi->sh->chippkg == BCMA_PKG_ID_BCM4717) ||
+            (pi->sh->chippkg == BCMA_PKG_ID_BCM4718))) {
                if ((pi->sh->boardflags & BFL_EXTLNA) &&
                    (CHSPEC_IS2G(pi->radio_chanspec)))
                        ai_cc_reg(pi->sh->sih,
                                cascbias = 0x20;
 
                                if ((pi->sh->chip ==
-                                    BCM43224_CHIP_ID)
+                                    BCMA_CHIP_ID_BCM43224)
                                    || (pi->sh->chip ==
-                                       BCM43225_CHIP_ID)) {
+                                       BCMA_CHIP_ID_BCM43225)) {
                                        if (pi->sh->chippkg ==
-                                           BCM43224_FAB_SMIC) {
+                                           BCMA_PKG_ID_BCM43224_FAB_SMIC) {
                                                bias = 0x2a;
                                                cascbias = 0x38;
                                        }
 
                        cascbias = 0x30;
 
-                       if ((pi->sh->chip == BCM43224_CHIP_ID) ||
-                           (pi->sh->chip == BCM43225_CHIP_ID)) {
-                               if (pi->sh->chippkg == BCM43224_FAB_SMIC)
+                       if ((pi->sh->chip == BCMA_CHIP_ID_BCM43224) ||
+                           (pi->sh->chip == BCMA_CHIP_ID_BCM43225)) {
+                               if (pi->sh->chippkg == BCMA_PKG_ID_BCM43224_FAB_SMIC)
                                        cascbias = 0x35;
                        }
 
                bcma_pmu_spuravoid_pllupdate(&sii->icbus->drv_cc, spuravoid);
                wlapi_bmac_core_phypll_ctl(pi->sh->physhim, true);
 
-               if ((pi->sh->chip == BCM43224_CHIP_ID) ||
-                   (pi->sh->chip == BCM43225_CHIP_ID)) {
+               if ((pi->sh->chip == BCMA_CHIP_ID_BCM43224) ||
+                   (pi->sh->chip == BCMA_CHIP_ID_BCM43225)) {
                        if (spuravoid == 1) {
                                bcma_write16(pi->d11core,
                                             D11REGOFFS(tsf_clk_frac_l),
 
        uint delay = PMU_MAX_TRANSITION_DLY;
 
        switch (ai_get_chip_id(sih)) {
-       case BCM43224_CHIP_ID:
-       case BCM43225_CHIP_ID:
-       case BCM4313_CHIP_ID:
+       case BCMA_CHIP_ID_BCM43224:
+       case BCMA_CHIP_ID_BCM43225:
+       case BCMA_CHIP_ID_BCM4313:
                delay = 3700;
                break;
        default:
                return clock;
 
        switch (ai_get_chip_id(sih)) {
-       case BCM43224_CHIP_ID:
-       case BCM43225_CHIP_ID:
-       case BCM4313_CHIP_ID:
+       case BCMA_CHIP_ID_BCM43224:
+       case BCMA_CHIP_ID_BCM43225:
+       case BCMA_CHIP_ID_BCM4313:
                /* always 20Mhz */
                clock = 20000 * 1000;
                break;