staging: brcm80211: remove dependency on pci core difinitions from aiutils.c
authorArend van Spriel <arend@broadcom.com>
Wed, 1 Jun 2011 11:45:27 +0000 (13:45 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 7 Jun 2011 19:37:00 +0000 (12:37 -0700)
The file aiutils.c included the register definition includes for the
PCI and PCIe core. This was for two functions which have been partly
moved to nicpci.c. This means that nicpci.h is the only include file
to provide interface to aiutils.c for PCI core related functions.

Signed-off-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/brcm80211/brcmsmac/aiutils.c
drivers/staging/brcm80211/brcmsmac/nicpci.c
drivers/staging/brcm80211/brcmsmac/nicpci.h

index bae40fe26d9002f1d6e00ef4c733163eb407dda2..43320b480e4246b36b4bb062c7d23f62eb5e3584 100644 (file)
@@ -27,8 +27,6 @@
 #include <bcmdevs.h>
 
 /* ********** from siutils.c *********** */
-#include <pci_core.h>
-#include <pcie_core.h>
 #include <nicpci.h>
 #include <bcmnvram.h>
 #include <bcmsrom.h>
@@ -1915,7 +1913,7 @@ void ai_pci_down(si_t *sih)
 void ai_pci_setup(si_t *sih, uint coremask)
 {
        si_info_t *sii;
-       struct sbpciregs *pciregs = NULL;
+       void *regs = NULL;
        u32 siflag = 0, w;
        uint idx = 0;
 
@@ -1932,7 +1930,7 @@ void ai_pci_setup(si_t *sih, uint coremask)
                siflag = ai_flag(sih);
 
                /* switch over to pci core */
-               pciregs = ai_setcoreidx(sih, sii->pub.buscoreidx);
+               regs = ai_setcoreidx(sih, sii->pub.buscoreidx);
        }
 
        /*
@@ -1950,16 +1948,7 @@ void ai_pci_setup(si_t *sih, uint coremask)
        }
 
        if (PCI(sii)) {
-               OR_REG(&pciregs->sbtopci2,
-                      (SBTOPCI_PREF | SBTOPCI_BURST));
-               if (sii->pub.buscorerev >= 11) {
-                       OR_REG(&pciregs->sbtopci2,
-                              SBTOPCI_RC_READMULTI);
-                       w = R_REG(&pciregs->clkrun);
-                       W_REG(&pciregs->clkrun,
-                             (w | PCI_CLKRUN_DSBL));
-                       w = R_REG(&pciregs->clkrun);
-               }
+               pcicore_pci_setup(sii->pch, regs);
 
                /* switch back to previous core */
                ai_setcoreidx(sih, idx);
@@ -1972,11 +1961,8 @@ void ai_pci_setup(si_t *sih, uint coremask)
  */
 int ai_pci_fixcfg(si_t *sih)
 {
-       uint origidx, pciidx;
-       struct sbpciregs *pciregs = NULL;
-       sbpcieregs_t *pcieregs = NULL;
+       uint origidx;
        void *regs = NULL;
-       u16 val16, *reg16 = NULL;
 
        si_info_t *sii = SI_INFO(sih);
 
@@ -1985,23 +1971,8 @@ int ai_pci_fixcfg(si_t *sih)
        origidx = ai_coreidx(&sii->pub);
 
        /* check 'pi' is correct and fix it if not */
-       if (sii->pub.buscoretype == PCIE_CORE_ID) {
-               pcieregs = ai_setcore(&sii->pub, PCIE_CORE_ID, 0);
-               regs = pcieregs;
-               reg16 = &pcieregs->sprom[SRSH_PI_OFFSET];
-       } else if (sii->pub.buscoretype == PCI_CORE_ID) {
-               pciregs = ai_setcore(&sii->pub, PCI_CORE_ID, 0);
-               regs = pciregs;
-               reg16 = &pciregs->sprom[SRSH_PI_OFFSET];
-       }
-       pciidx = ai_coreidx(&sii->pub);
-       val16 = R_REG(reg16);
-       if (((val16 & SRSH_PI_MASK) >> SRSH_PI_SHIFT) != (u16) pciidx) {
-               val16 =
-                   (u16) (pciidx << SRSH_PI_SHIFT) | (val16 &
-                                                         ~SRSH_PI_MASK);
-               W_REG(reg16, val16);
-       }
+       regs = ai_setcore(&sii->pub, sii->pub.buscoretype, 0);
+       pcicore_fixcfg(sii->pch, regs);
 
        /* restore the original index */
        ai_setcoreidx(&sii->pub, origidx);
index 196cdf168180645d3753eb2571b4caf6eecce61e..c509c9a19d91326eb5d4d33929c5f6aaa6c7083c 100644 (file)
@@ -654,3 +654,53 @@ void pcicore_down(void *pch, int state)
        /* Reduce L1 timer for better power savings */
        pcie_extendL1timer(pi, false);
 }
+
+/*
+ * precondition: current core is sii->buscoretype
+ */
+void pcicore_fixcfg(void *pch, void *regs)
+{
+       pcicore_info_t *pi = (pcicore_info_t *) pch;
+       struct si_info *sii = SI_INFO(pi->sih);
+       struct sbpciregs *pciregs = regs;
+       sbpcieregs_t *pcieregs = regs;
+       u16 val16, *reg16 = NULL;
+       uint pciidx;
+
+       /* check 'pi' is correct and fix it if not */
+       if (sii->pub.buscoretype == PCIE_CORE_ID) {
+               reg16 = &pcieregs->sprom[SRSH_PI_OFFSET];
+       } else if (sii->pub.buscoretype == PCI_CORE_ID) {
+               reg16 = &pciregs->sprom[SRSH_PI_OFFSET];
+       }
+       pciidx = ai_coreidx(&sii->pub);
+       val16 = R_REG(reg16);
+       if (((val16 & SRSH_PI_MASK) >> SRSH_PI_SHIFT) != (u16) pciidx) {
+               val16 =
+                   (u16) (pciidx << SRSH_PI_SHIFT) | (val16 &
+                                                         ~SRSH_PI_MASK);
+               W_REG(reg16, val16);
+       }
+}
+
+/*
+ * precondition: current core is pci core
+ */
+void pcicore_pci_setup(void *pch, void *regs)
+{
+       pcicore_info_t *pi = (pcicore_info_t *) pch;
+       struct sbpciregs *pciregs = regs;
+       u32 w;
+
+       OR_REG(&pciregs->sbtopci2,
+              (SBTOPCI_PREF | SBTOPCI_BURST));
+
+       if (SI_INFO(pi->sih)->pub.buscorerev >= 11) {
+               OR_REG(&pciregs->sbtopci2,
+                      SBTOPCI_RC_READMULTI);
+               w = R_REG(&pciregs->clkrun);
+               W_REG(&pciregs->clkrun,
+                     (w | PCI_CLKRUN_DSBL));
+               w = R_REG(&pciregs->clkrun);
+       }
+}
index b32f1af460d2d50f16459b51866d2f2ec95e5337..f1441c572ae5d0e3d6bf4087f2186a6753a6d0cb 100644 (file)
@@ -67,5 +67,7 @@ extern void pcicore_sleep(void *pch);
 extern void pcicore_down(void *pch, int state);
 extern u8 pcicore_find_pci_capability(void *dev, u8 req_cap_id,
                                         unsigned char *buf, u32 *buflen);
+extern void pcicore_fixcfg(void *pch, void *regs);
+extern void pcicore_pci_setup(void *pch, void *regs);
 
 #endif /* _NICPCI_H */