staging: brcm80211: replaced void *btparam into struct pci_dev *btparam
authorRoland Vossen <rvossen@broadcom.com>
Mon, 8 Aug 2011 13:58:59 +0000 (15:58 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 23 Aug 2011 20:00:10 +0000 (13:00 -0700)
Code cleanup. Since the softmac driver only supports a PCI(e) bus, the void *
could be replaced with a less generic type.

Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/brcm80211/brcmsmac/aiutils.c
drivers/staging/brcm80211/brcmsmac/aiutils.h
drivers/staging/brcm80211/brcmsmac/mac80211_if.c
drivers/staging/brcm80211/brcmsmac/main.c
drivers/staging/brcm80211/brcmsmac/pub.h

index 37edd1ce312440d3905a43a87b663942b8d1b29e..9cfb492d7c34342202ed7f848b85185bddf23aa7 100644 (file)
@@ -861,7 +861,8 @@ u32 ai_core_sflags(struct si_pub *sih, u32 mask, u32 val)
 /* *************** from siutils.c ************** */
 /* local prototypes */
 static struct si_info *ai_doattach(struct si_info *sii, void *regs,
-                                  void *sdh, char **vars, uint *varsz);
+                                  struct pci_dev *sdh,
+                                  char **vars, uint *varsz);
 static bool ai_buscore_prep(struct si_info *sii);
 static bool ai_buscore_setup(struct si_info *sii, struct chipcregs *cc,
                             u32 savewin, uint *origidx, void *regs);
@@ -881,7 +882,8 @@ static bool ai_ispcie(struct si_info *sii);
  * vars - pointer to a pointer area for "environment" variables
  * varsz - pointer to int to return the size of the vars
  */
-struct si_pub *ai_attach(void *regs, void *sdh, char **vars, uint *varsz)
+struct si_pub *
+ai_attach(void *regs, struct pci_dev *sdh, char **vars, uint *varsz)
 {
        struct si_info *sii;
 
@@ -1052,7 +1054,7 @@ static __used void ai_nvram_process(struct si_info *sii, char *pvars)
 }
 
 static struct si_info *ai_doattach(struct si_info *sii,
-                                  void *regs, void *pbus,
+                                  void *regs, struct pci_dev *pbus,
                                   char **vars, uint *varsz)
 {
        struct si_pub *sih = &sii->pub;
index 076b4c889fcb1966be7e856aa9f0220837a80ffb..101ab92f2ea69942b47c53a7df1622a440aaae46 100644 (file)
@@ -446,6 +446,8 @@ struct si_pub {
 #define        IS_SIM(chippkg) \
        ((chippkg == HDLSIM_PKG_ID) || (chippkg == HWSIM_PKG_ID))
 
+struct pci_dev;
+
 struct gpioh_item {
        void *arg;
        bool level;
@@ -515,7 +517,7 @@ extern u32 ai_addrspacesize(struct si_pub *sih, uint asidx);
 extern void ai_write_wrap_reg(struct si_pub *sih, u32 offset, u32 val);
 
 /* === exported functions === */
-extern struct si_pub *ai_attach(void *regs, void *sdh, char **vars,
+extern struct si_pub *ai_attach(void *regs, struct pci_dev *sdh, char **vars,
                                uint *varsz);
 extern void ai_detach(struct si_pub *sih);
 extern bool ai_pci_war16165(struct si_pub *sih);
index 6bcb7ea1b10b4cf6b49752fa269e388b2ed0b670..7f6a490310fe7522a52262ab4275913c41bc38c7 100644 (file)
@@ -755,7 +755,7 @@ static int brcms_set_hint(struct brcms_info *wl, char *abbrev)
  */
 static struct brcms_info *brcms_attach(u16 vendor, u16 device,
                                       unsigned long regs,
-                                      void *btparam, uint irq)
+                                      struct pci_dev *btparam, uint irq)
 {
        struct brcms_info *wl = NULL;
        int unit, err;
@@ -795,11 +795,11 @@ static struct brcms_info *brcms_attach(u16 vendor, u16 device,
        spin_lock_init(&wl->isr_lock);
 
        /* prepare ucode */
-       if (brcms_request_fw(wl, (struct pci_dev *)btparam) < 0) {
+       if (brcms_request_fw(wl, btparam) < 0) {
                wiphy_err(wl->wiphy, "%s: Failed to find firmware usually in "
                          "%s\n", KBUILD_MODNAME, "/lib/firmware/brcm");
                brcms_release_fw(wl);
-               brcms_remove((struct pci_dev *)btparam);
+               brcms_remove(btparam);
                return NULL;
        }
 
index fb9f13fe5fc3e080dad2845caf3ec8350c69ccd6..d011ed68c712bb2e1c44595dea075214a8d09dff 100644 (file)
@@ -313,7 +313,7 @@ struct brcms_b_state;
 
 static int brcms_b_attach(struct brcms_c_info *wlc, u16 vendor, u16 device,
                          uint unit, bool piomode, void *regsva,
-                         void *btparam);
+                         struct pci_dev *btparam);
 
 /* up/down, reset, clk */
 static void brcms_b_reset(struct brcms_hardware *wlc_hw);
@@ -4301,7 +4301,7 @@ struct brcms_pub *brcms_c_pub(void *wlc)
  *    put the whole chip in reset(driver down state), no clock
  */
 int brcms_b_attach(struct brcms_c_info *wlc, u16 vendor, u16 device, uint unit,
-                  bool piomode, void *regsva, void *btparam)
+                  bool piomode, void *regsva, struct pci_dev *btparam)
 {
        struct brcms_hardware *wlc_hw;
        struct d11regs *regs;
@@ -4644,8 +4644,9 @@ int brcms_b_attach(struct brcms_c_info *wlc, u16 vendor, u16 device, uint unit,
 /*
  * The common driver entry routine. Error codes should be unique
  */
-void *brcms_c_attach(struct brcms_info *wl, u16 vendor, u16 device, uint unit,
-                bool piomode, void *regsva, void *btparam, uint *perr)
+void *
+brcms_c_attach(struct brcms_info *wl, u16 vendor, u16 device, uint unit,
+              bool piomode, void *regsva, struct pci_dev *btparam, uint *perr)
 {
        struct brcms_c_info *wlc;
        uint err = 0;
index 5e15c6753f658741fcf30d893f9829197e8e754c..bfbb2615bed4e78757721509803869fa401c0bb6 100644 (file)
@@ -551,7 +551,7 @@ struct brcms_antselcfg {
 /* common functions for every port */
 extern void *brcms_c_attach(struct brcms_info *wl, u16 vendor, u16 device,
                        uint unit, bool piomode, void *regsva,
-                       void *btparam, uint *perr);
+                       struct pci_dev *btparam, uint *perr);
 extern uint brcms_c_detach(struct brcms_c_info *wlc);
 extern int brcms_c_up(struct brcms_c_info *wlc);
 extern uint brcms_c_down(struct brcms_c_info *wlc);