Browse Source

brcmfmac: remove regs parameter from sdio probe functions

The chip recognition requires a base address that was provided
to it during the probe. However, the address is a fixed define
value so it is unnecessary to pass through the probe functions.

Reviewed-by: Franky Lin <frankyl@broadcom.com>
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: John W. Linville <linville@tuxdriver.com>
Arend van Spriel 11 years ago
parent
commit
4744d16402

+ 1 - 4
drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c

@@ -992,17 +992,14 @@ static int brcmf_sdio_remove(struct brcmf_sdio_dev *sdiodev)
 
 
 static int brcmf_sdio_probe(struct brcmf_sdio_dev *sdiodev)
 static int brcmf_sdio_probe(struct brcmf_sdio_dev *sdiodev)
 {
 {
-	u32 regs = 0;
 	int ret = 0;
 	int ret = 0;
 
 
 	ret = brcmf_sdioh_attach(sdiodev);
 	ret = brcmf_sdioh_attach(sdiodev);
 	if (ret)
 	if (ret)
 		goto out;
 		goto out;
 
 
-	regs = SI_ENUM_BASE;
-
 	/* try to attach to the target device */
 	/* try to attach to the target device */
-	sdiodev->bus = brcmf_sdbrcm_probe(regs, sdiodev);
+	sdiodev->bus = brcmf_sdbrcm_probe(sdiodev);
 	if (!sdiodev->bus) {
 	if (!sdiodev->bus) {
 		brcmf_err("device attach failed\n");
 		brcmf_err("device attach failed\n");
 		ret = -ENODEV;
 		ret = -ENODEV;

+ 4 - 7
drivers/net/wireless/brcm80211/brcmfmac/dhd_sdio.c

@@ -3803,7 +3803,7 @@ static bool brcmf_sdbrcm_probe_malloc(struct brcmf_sdio *bus)
 }
 }
 
 
 static bool
 static bool
-brcmf_sdbrcm_probe_attach(struct brcmf_sdio *bus, u32 regsva)
+brcmf_sdbrcm_probe_attach(struct brcmf_sdio *bus)
 {
 {
 	u8 clkctl = 0;
 	u8 clkctl = 0;
 	int err = 0;
 	int err = 0;
@@ -3835,7 +3835,7 @@ brcmf_sdbrcm_probe_attach(struct brcmf_sdio *bus, u32 regsva)
 		goto fail;
 		goto fail;
 	}
 	}
 
 
-	if (brcmf_sdio_chip_attach(bus->sdiodev, &bus->ci, regsva)) {
+	if (brcmf_sdio_chip_attach(bus->sdiodev, &bus->ci)) {
 		brcmf_err("brcmf_sdio_chip_attach failed!\n");
 		brcmf_err("brcmf_sdio_chip_attach failed!\n");
 		goto fail;
 		goto fail;
 	}
 	}
@@ -4037,16 +4037,13 @@ static struct brcmf_bus_ops brcmf_sdio_bus_ops = {
 	.gettxq = brcmf_sdbrcm_bus_gettxq,
 	.gettxq = brcmf_sdbrcm_bus_gettxq,
 };
 };
 
 
-void *brcmf_sdbrcm_probe(u32 regsva, struct brcmf_sdio_dev *sdiodev)
+void *brcmf_sdbrcm_probe(struct brcmf_sdio_dev *sdiodev)
 {
 {
 	int ret;
 	int ret;
 	struct brcmf_sdio *bus;
 	struct brcmf_sdio *bus;
 
 
 	brcmf_dbg(TRACE, "Enter\n");
 	brcmf_dbg(TRACE, "Enter\n");
 
 
-	/* We make an assumption about address window mappings:
-	 * regsva == SI_ENUM_BASE*/
-
 	/* Allocate private bus interface state */
 	/* Allocate private bus interface state */
 	bus = kzalloc(sizeof(struct brcmf_sdio), GFP_ATOMIC);
 	bus = kzalloc(sizeof(struct brcmf_sdio), GFP_ATOMIC);
 	if (!bus)
 	if (!bus)
@@ -4080,7 +4077,7 @@ void *brcmf_sdbrcm_probe(u32 regsva, struct brcmf_sdio_dev *sdiodev)
 	}
 	}
 
 
 	/* attempt to attach to the dongle */
 	/* attempt to attach to the dongle */
-	if (!(brcmf_sdbrcm_probe_attach(bus, regsva))) {
+	if (!(brcmf_sdbrcm_probe_attach(bus))) {
 		brcmf_err("brcmf_sdbrcm_probe_attach failed\n");
 		brcmf_err("brcmf_sdbrcm_probe_attach failed\n");
 		goto fail;
 		goto fail;
 	}
 	}

+ 4 - 4
drivers/net/wireless/brcm80211/brcmfmac/sdio_chip.c

@@ -438,7 +438,7 @@ static inline int brcmf_sdio_chip_cichk(struct chip_info *ci)
 #endif
 #endif
 
 
 static int brcmf_sdio_chip_recognition(struct brcmf_sdio_dev *sdiodev,
 static int brcmf_sdio_chip_recognition(struct brcmf_sdio_dev *sdiodev,
-				       struct chip_info *ci, u32 regs)
+				       struct chip_info *ci)
 {
 {
 	u32 regdata;
 	u32 regdata;
 	int ret;
 	int ret;
@@ -449,7 +449,7 @@ static int brcmf_sdio_chip_recognition(struct brcmf_sdio_dev *sdiodev,
 	 * other ways of recognition should be added here.
 	 * other ways of recognition should be added here.
 	 */
 	 */
 	ci->c_inf[0].id = BCMA_CORE_CHIPCOMMON;
 	ci->c_inf[0].id = BCMA_CORE_CHIPCOMMON;
-	ci->c_inf[0].base = regs;
+	ci->c_inf[0].base = SI_ENUM_BASE;
 	regdata = brcmf_sdio_regrl(sdiodev,
 	regdata = brcmf_sdio_regrl(sdiodev,
 				   CORE_CC_REG(ci->c_inf[0].base, chipid),
 				   CORE_CC_REG(ci->c_inf[0].base, chipid),
 				   NULL);
 				   NULL);
@@ -681,7 +681,7 @@ brcmf_sdio_chip_buscoresetup(struct brcmf_sdio_dev *sdiodev,
 }
 }
 
 
 int brcmf_sdio_chip_attach(struct brcmf_sdio_dev *sdiodev,
 int brcmf_sdio_chip_attach(struct brcmf_sdio_dev *sdiodev,
-			   struct chip_info **ci_ptr, u32 regs)
+			   struct chip_info **ci_ptr)
 {
 {
 	int ret;
 	int ret;
 	struct chip_info *ci;
 	struct chip_info *ci;
@@ -697,7 +697,7 @@ int brcmf_sdio_chip_attach(struct brcmf_sdio_dev *sdiodev,
 	if (ret != 0)
 	if (ret != 0)
 		goto err;
 		goto err;
 
 
-	ret = brcmf_sdio_chip_recognition(sdiodev, ci, regs);
+	ret = brcmf_sdio_chip_recognition(sdiodev, ci);
 	if (ret != 0)
 	if (ret != 0)
 		goto err;
 		goto err;
 
 

+ 1 - 1
drivers/net/wireless/brcm80211/brcmfmac/sdio_chip.h

@@ -224,7 +224,7 @@ struct sdpcmd_regs {
 };
 };
 
 
 int brcmf_sdio_chip_attach(struct brcmf_sdio_dev *sdiodev,
 int brcmf_sdio_chip_attach(struct brcmf_sdio_dev *sdiodev,
-			   struct chip_info **ci_ptr, u32 regs);
+			   struct chip_info **ci_ptr);
 void brcmf_sdio_chip_detach(struct chip_info **ci_ptr);
 void brcmf_sdio_chip_detach(struct chip_info **ci_ptr);
 void brcmf_sdio_chip_drivestrengthinit(struct brcmf_sdio_dev *sdiodev,
 void brcmf_sdio_chip_drivestrengthinit(struct brcmf_sdio_dev *sdiodev,
 				       struct chip_info *ci, u32 drivestrength);
 				       struct chip_info *ci, u32 drivestrength);

+ 1 - 1
drivers/net/wireless/brcm80211/brcmfmac/sdio_host.h

@@ -239,7 +239,7 @@ int brcmf_sdio_ramrw(struct brcmf_sdio_dev *sdiodev, bool write, u32 address,
 /* Issue an abort to the specified function */
 /* Issue an abort to the specified function */
 int brcmf_sdcard_abort(struct brcmf_sdio_dev *sdiodev, uint fn);
 int brcmf_sdcard_abort(struct brcmf_sdio_dev *sdiodev, uint fn);
 
 
-void *brcmf_sdbrcm_probe(u32 regsva, struct brcmf_sdio_dev *sdiodev);
+void *brcmf_sdbrcm_probe(struct brcmf_sdio_dev *sdiodev);
 void brcmf_sdbrcm_disconnect(void *ptr);
 void brcmf_sdbrcm_disconnect(void *ptr);
 void brcmf_sdbrcm_isr(void *arg);
 void brcmf_sdbrcm_isr(void *arg);