b43legacy: Fix sparse warnings
authorLarry Finger <Larry.Finger@lwfinger.net>
Wed, 10 Dec 2008 05:35:41 +0000 (23:35 -0600)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 12 Dec 2008 19:02:08 +0000 (14:02 -0500)
Sparse yields the following warnings for b43legacy:

  CHECK   drivers/net/wireless/b43legacy/phy.c
drivers/net/wireless/b43legacy/phy.c:1304:31: warning: potentially expensive pointer subtraction
drivers/net/wireless/b43legacy/phy.c:1304:31: warning: potentially expensive pointer subtraction
drivers/net/wireless/b43legacy/phy.c:1304:31: warning: potentially expensive pointer subtraction
  CHECK   drivers/net/wireless/b43legacy/debugfs.c
drivers/net/wireless/b43legacy/debugfs.c:243:9: warning: memset with byte count of 131072

Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/b43legacy/debugfs.c
drivers/net/wireless/b43legacy/phy.c

index 03ce0821a60eb86e9f327675c9069bc1b64ecb1b..1f85ac569fec0065868fb4341971dfec9c83480f 100644 (file)
@@ -211,7 +211,7 @@ static ssize_t b43legacy_debugfs_read(struct file *file, char __user *userbuf,
        struct b43legacy_dfs_file *dfile;
        ssize_t uninitialized_var(ret);
        char *buf;
-       const size_t bufsize = 1024 * 128;
+       const size_t bufsize = 1024 * 16; /* 16 KiB buffer */
        const size_t buforder = get_order(bufsize);
        int err = 0;
 
index 4c9442b16f3fb51ce9505ac5ac013d2d453d6f49..11319ec2d64a75d0788a3e6ea97357973b766da5 100644 (file)
@@ -1296,12 +1296,10 @@ void b43legacy_lo_write(struct b43legacy_wldev *dev,
        /* Sanity check. */
        if (pair->low < -8 || pair->low > 8 ||
            pair->high < -8 || pair->high > 8) {
-               struct b43legacy_phy *phy = &dev->phy;
                b43legacydbg(dev->wl,
                       "WARNING: Writing invalid LOpair "
-                      "(low: %d, high: %d, index: %lu)\n",
-                      pair->low, pair->high,
-                      (unsigned long)(pair - phy->_lo_pairs));
+                      "(low: %d, high: %d)\n",
+                      pair->low, pair->high);
                dump_stack();
        }
 #endif