staging: brcm80211: remove devpath related nvram variable lookup
authorArend van Spriel <arend@broadcom.com>
Tue, 13 Sep 2011 07:49:50 +0000 (09:49 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 16 Sep 2011 18:41:53 +0000 (20:41 +0200)
The lookup function used for the devpath related functions did
not provide a variable table so the lookup would always fail. As
this makes the functions pretty useless these have been removed.

Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-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/aiutils.h

index 9510d838009d3d7cdb3b44210b22bd5f4c983eaa..b7b0bdf19828e5477ef646270a0dac754a1baf3d 100644 (file)
@@ -1018,17 +1018,9 @@ static __used void ai_nvram_process(struct si_info *sii, char *pvars)
 
        /* do a pci config read to get subsystem id and subvendor id */
        pci_read_config_dword(sii->pbus, PCI_SUBSYSTEM_VENDOR_ID, &w);
-       /* Let nvram variables override subsystem Vend/ID */
-       sii->pub.boardvendor = (u16)ai_getdevpathintvar(&sii->pub,
-                                                       "boardvendor");
-       if (sii->pub.boardvendor == 0)
-               sii->pub.boardvendor = w & 0xffff;
-
-       sii->pub.boardtype = (u16)ai_getdevpathintvar(&sii->pub,
-               "boardtype");
-       if (sii->pub.boardtype == 0)
-               sii->pub.boardtype = (w >> 16) & 0xffff;
 
+       sii->pub.boardvendor = w & 0xffff;
+       sii->pub.boardtype = (w >> 16) & 0xffff;
        sii->pub.boardflags = getintvar(pvars, "boardflags");
 }
 
@@ -1864,56 +1856,6 @@ int ai_devpath(struct si_pub *sih, char *path, int size)
        return 0;
 }
 
-/* Concatenate the dev path with a varname into the given 'var' buffer
- * and return the 'var' pointer. Nothing is done to the arguments if
- * len == 0 or var is NULL, var is still returned. On overflow, the
- * first char will be set to '\0'.
- */
-static char *ai_devpathvar(struct si_pub *sih, char *var, int len,
-                          const char *name)
-{
-       uint path_len;
-
-       if (!var || len <= 0)
-               return var;
-
-       if (ai_devpath(sih, var, len) == 0) {
-               path_len = strlen(var);
-
-               if (strlen(name) + 1 > (uint) (len - path_len))
-                       var[0] = '\0';
-               else
-                       strncpy(var + path_len, name, len - path_len - 1);
-       }
-
-       return var;
-}
-
-/* Get a variable, but only if it has a devpath prefix */
-char *ai_getdevpathvar(struct si_pub *sih, const char *name)
-{
-       char varname[SI_DEVPATH_BUFSZ + 32];
-
-       ai_devpathvar(sih, varname, sizeof(varname), name);
-
-       return getvar(NULL, varname);
-}
-
-/* Get a variable, but only if it has a devpath prefix */
-int ai_getdevpathintvar(struct si_pub *sih, const char *name)
-{
-       char varname[SI_DEVPATH_BUFSZ + 32];
-
-       ai_devpathvar(sih, varname, sizeof(varname), name);
-
-       return getintvar(NULL, varname);
-}
-
-char *ai_getnvramflvar(struct si_pub *sih, const char *name)
-{
-       return getvar(NULL, name);
-}
-
 bool ai_pci_war16165(struct si_pub *sih)
 {
        struct si_info *sii;
index 0d4fa85f9be90f2e0cec8ac7cbcc66146d2914a5..a7769451cffbd25b01ccabd272e04cc892d505d3 100644 (file)
@@ -369,9 +369,6 @@ extern bool ai_is_sprom_available(struct si_pub *sih);
  * Return 0 on success, nonzero otherwise.
  */
 extern int ai_devpath(struct si_pub *sih, char *path, int size);
-/* Read variable with prepending the devpath to the name */
-extern char *ai_getdevpathvar(struct si_pub *sih, const char *name);
-extern int ai_getdevpathintvar(struct si_pub *sih, const char *name);
 
 extern void ai_pci_sleep(struct si_pub *sih);
 extern void ai_pci_down(struct si_pub *sih);
@@ -382,6 +379,4 @@ extern void ai_chipcontrl_epa4331(struct si_pub *sih, bool on);
 /* Enable Ex-PA for 4313 */
 extern void ai_epa_4313war(struct si_pub *sih);
 
-char *ai_getnvramflvar(struct si_pub *sih, const char *name);
-
 #endif                         /* _BRCM_AIUTILS_H_ */