diff options
author | Arend van Spriel <arend@broadcom.com> | 2015-01-06 17:02:54 -0500 |
---|---|---|
committer | Kalle Valo <kvalo@codeaurora.org> | 2015-01-09 08:48:06 -0500 |
commit | 8bd61f8d8790c5502f7fdbf49861bbea1339054f (patch) | |
tree | 75965f1403d70c4fd85366427f3d8c19303d1895 /drivers/net/wireless | |
parent | 86f2db86d410f5dc3e89cc7d9c31607f0c7763bd (diff) |
brcmfmac: get rid of duplicate SDIO device identifiers
Instead of defining SDIO device identifier in brcm80211 code use
the defintions in linux/mmc/sdio_ids.h directly.
Reviewed-by: Hante Meuleman <meuleman@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Diffstat (limited to 'drivers/net/wireless')
-rw-r--r-- | drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c | 22 | ||||
-rw-r--r-- | drivers/net/wireless/brcm80211/brcmfmac/sdio.c | 2 | ||||
-rw-r--r-- | drivers/net/wireless/brcm80211/include/brcm_hw_ids.h | 13 |
3 files changed, 12 insertions, 25 deletions
diff --git a/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c b/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c index dffd9e44f5b6..00ba90b89455 100644 --- a/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c +++ b/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c | |||
@@ -996,20 +996,20 @@ out: | |||
996 | } | 996 | } |
997 | 997 | ||
998 | #define BRCMF_SDIO_DEVICE(dev_id) \ | 998 | #define BRCMF_SDIO_DEVICE(dev_id) \ |
999 | {SDIO_DEVICE(BRCM_SDIO_VENDOR_ID_BROADCOM, dev_id)} | 999 | {SDIO_DEVICE(SDIO_VENDOR_ID_BROADCOM, dev_id)} |
1000 | 1000 | ||
1001 | /* devices we support, null terminated */ | 1001 | /* devices we support, null terminated */ |
1002 | static const struct sdio_device_id brcmf_sdmmc_ids[] = { | 1002 | static const struct sdio_device_id brcmf_sdmmc_ids[] = { |
1003 | BRCMF_SDIO_DEVICE(BRCM_SDIO_43143_DEVICE_ID), | 1003 | BRCMF_SDIO_DEVICE(SDIO_DEVICE_ID_BROADCOM_43143), |
1004 | BRCMF_SDIO_DEVICE(BRCM_SDIO_43241_DEVICE_ID), | 1004 | BRCMF_SDIO_DEVICE(SDIO_DEVICE_ID_BROADCOM_43241), |
1005 | BRCMF_SDIO_DEVICE(BRCM_SDIO_4329_DEVICE_ID), | 1005 | BRCMF_SDIO_DEVICE(SDIO_DEVICE_ID_BROADCOM_4329), |
1006 | BRCMF_SDIO_DEVICE(BRCM_SDIO_4330_DEVICE_ID), | 1006 | BRCMF_SDIO_DEVICE(SDIO_DEVICE_ID_BROADCOM_4330), |
1007 | BRCMF_SDIO_DEVICE(BRCM_SDIO_4334_DEVICE_ID), | 1007 | BRCMF_SDIO_DEVICE(SDIO_DEVICE_ID_BROADCOM_4334), |
1008 | BRCMF_SDIO_DEVICE(BRCM_SDIO_43340_DEVICE_ID), | 1008 | BRCMF_SDIO_DEVICE(SDIO_DEVICE_ID_BROADCOM_43340), |
1009 | BRCMF_SDIO_DEVICE(BRCM_SDIO_43341_DEVICE_ID), | 1009 | BRCMF_SDIO_DEVICE(SDIO_DEVICE_ID_BROADCOM_43341), |
1010 | BRCMF_SDIO_DEVICE(BRCM_SDIO_43362_DEVICE_ID), | 1010 | BRCMF_SDIO_DEVICE(SDIO_DEVICE_ID_BROADCOM_43362), |
1011 | BRCMF_SDIO_DEVICE(BRCM_SDIO_4335_4339_DEVICE_ID), | 1011 | BRCMF_SDIO_DEVICE(SDIO_DEVICE_ID_BROADCOM_4335_4339), |
1012 | BRCMF_SDIO_DEVICE(BRCM_SDIO_4354_DEVICE_ID), | 1012 | BRCMF_SDIO_DEVICE(SDIO_DEVICE_ID_BROADCOM_4354), |
1013 | { /* end: all zeroes */ } | 1013 | { /* end: all zeroes */ } |
1014 | }; | 1014 | }; |
1015 | MODULE_DEVICE_TABLE(sdio, brcmf_sdmmc_ids); | 1015 | MODULE_DEVICE_TABLE(sdio, brcmf_sdmmc_ids); |
diff --git a/drivers/net/wireless/brcm80211/brcmfmac/sdio.c b/drivers/net/wireless/brcm80211/brcmfmac/sdio.c index 551da356a5bd..99a37765888d 100644 --- a/drivers/net/wireless/brcm80211/brcmfmac/sdio.c +++ b/drivers/net/wireless/brcm80211/brcmfmac/sdio.c | |||
@@ -3816,7 +3816,7 @@ static u32 brcmf_sdio_buscore_read32(void *ctx, u32 addr) | |||
3816 | u32 val, rev; | 3816 | u32 val, rev; |
3817 | 3817 | ||
3818 | val = brcmf_sdiod_regrl(sdiodev, addr, NULL); | 3818 | val = brcmf_sdiod_regrl(sdiodev, addr, NULL); |
3819 | if (sdiodev->func[0]->device == BRCM_SDIO_4335_4339_DEVICE_ID && | 3819 | if (sdiodev->func[0]->device == SDIO_DEVICE_ID_BROADCOM_4335_4339 && |
3820 | addr == CORE_CC_REG(SI_ENUM_BASE, chipid)) { | 3820 | addr == CORE_CC_REG(SI_ENUM_BASE, chipid)) { |
3821 | rev = (val & CID_REV_MASK) >> CID_REV_SHIFT; | 3821 | rev = (val & CID_REV_MASK) >> CID_REV_SHIFT; |
3822 | if (rev >= 2) { | 3822 | if (rev >= 2) { |
diff --git a/drivers/net/wireless/brcm80211/include/brcm_hw_ids.h b/drivers/net/wireless/brcm80211/include/brcm_hw_ids.h index 00215efbc13b..2124a17d0bfd 100644 --- a/drivers/net/wireless/brcm80211/include/brcm_hw_ids.h +++ b/drivers/net/wireless/brcm80211/include/brcm_hw_ids.h | |||
@@ -22,7 +22,6 @@ | |||
22 | 22 | ||
23 | #define BRCM_USB_VENDOR_ID_BROADCOM 0x0a5c | 23 | #define BRCM_USB_VENDOR_ID_BROADCOM 0x0a5c |
24 | #define BRCM_PCIE_VENDOR_ID_BROADCOM PCI_VENDOR_ID_BROADCOM | 24 | #define BRCM_PCIE_VENDOR_ID_BROADCOM PCI_VENDOR_ID_BROADCOM |
25 | #define BRCM_SDIO_VENDOR_ID_BROADCOM SDIO_VENDOR_ID_BROADCOM | ||
26 | 25 | ||
27 | /* Chipcommon Core Chip IDs */ | 26 | /* Chipcommon Core Chip IDs */ |
28 | #define BRCM_CC_43143_CHIP_ID 43143 | 27 | #define BRCM_CC_43143_CHIP_ID 43143 |
@@ -46,18 +45,6 @@ | |||
46 | #define BRCM_CC_43570_CHIP_ID 43570 | 45 | #define BRCM_CC_43570_CHIP_ID 43570 |
47 | #define BRCM_CC_43602_CHIP_ID 43602 | 46 | #define BRCM_CC_43602_CHIP_ID 43602 |
48 | 47 | ||
49 | /* SDIO Device IDs */ | ||
50 | #define BRCM_SDIO_43143_DEVICE_ID BRCM_CC_43143_CHIP_ID | ||
51 | #define BRCM_SDIO_43241_DEVICE_ID BRCM_CC_43241_CHIP_ID | ||
52 | #define BRCM_SDIO_4329_DEVICE_ID BRCM_CC_4329_CHIP_ID | ||
53 | #define BRCM_SDIO_4330_DEVICE_ID BRCM_CC_4330_CHIP_ID | ||
54 | #define BRCM_SDIO_4334_DEVICE_ID BRCM_CC_4334_CHIP_ID | ||
55 | #define BRCM_SDIO_43340_DEVICE_ID BRCM_CC_43340_CHIP_ID | ||
56 | #define BRCM_SDIO_43341_DEVICE_ID 43341 | ||
57 | #define BRCM_SDIO_43362_DEVICE_ID BRCM_CC_43362_CHIP_ID | ||
58 | #define BRCM_SDIO_4335_4339_DEVICE_ID BRCM_CC_4335_CHIP_ID | ||
59 | #define BRCM_SDIO_4354_DEVICE_ID BRCM_CC_4354_CHIP_ID | ||
60 | |||
61 | /* USB Device IDs */ | 48 | /* USB Device IDs */ |
62 | #define BRCM_USB_43143_DEVICE_ID 0xbd1e | 49 | #define BRCM_USB_43143_DEVICE_ID 0xbd1e |
63 | #define BRCM_USB_43236_DEVICE_ID 0xbd17 | 50 | #define BRCM_USB_43236_DEVICE_ID 0xbd17 |