staging: brcm80211: use address space qualifier in brcmfmac
authorArend van Spriel <arend@broadcom.com>
Fri, 2 Sep 2011 14:00:30 +0000 (16:00 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 6 Sep 2011 23:46:26 +0000 (16:46 -0700)
Several sparse warning were issued due to missing __user qualifier
for brcmfmac variables. This patch adds those.

Reported-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com>
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/brcmfmac/dhd.h
drivers/staging/brcm80211/brcmfmac/dhd_linux.c
drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c

index ee09c17db58d9c40e4388aabae4526895ebe3383..4d06d874e17187ae24126cc7fdf42c0df9b2f11d 100644 (file)
@@ -779,7 +779,7 @@ extern void brcmf_c_pktfilter_offload_enable(struct brcmf_pub *drvr, char *arg,
 /* Linux network driver ioctl encoding */
 struct brcmf_c_ioctl {
        uint cmd;               /* common ioctl definition */
-       void *buf;              /* pointer to user buffer */
+       void __user *buf;       /* pointer to user buffer */
        uint len;               /* length of user buffer */
        bool set;               /* get or set request (optional) */
        uint used;              /* bytes read or written (optional) */
index 470759a1f1013b0aa8f43daef30f49d2a9705ea9..bb0f71866a6abe225206b47670f73ec4004261dd 100644 (file)
@@ -835,7 +835,7 @@ static struct ethtool_ops brcmf_ethtool_ops = {
        .get_drvinfo = brcmf_ethtool_get_drvinfo
 };
 
-static int brcmf_ethtool(struct brcmf_info *drvr_priv, void *uaddr)
+static int brcmf_ethtool(struct brcmf_info *drvr_priv, void __user *uaddr)
 {
        struct ethtool_drvinfo info;
        char drvname[sizeof(info.driver)];
@@ -1001,7 +1001,7 @@ static int brcmf_netdev_ioctl_entry(struct net_device *net, struct ifreq *ifr,
        }
 
        /* To differentiate read 4 more byes */
-       if ((copy_from_user(&driver, (char *)ifr->ifr_data +
+       if ((copy_from_user(&driver, (char __user *)ifr->ifr_data +
                            sizeof(struct brcmf_ioctl), sizeof(uint)) != 0)) {
                bcmerror = -EINVAL;
                goto done;
index 60a04b37a3d25382ed17b0a4bdeb99e6b59ec52d..db51c466f2e577548afe6c0566e20917beadc93b 100644 (file)
@@ -540,7 +540,7 @@ brcmf_dev_ioctl(struct net_device *dev, u32 cmd, void *arg, u32 len)
        ioc.buf = arg;
        ioc.len = len;
        strcpy(ifr.ifr_name, dev->name);
-       ifr.ifr_data = (caddr_t)&ioc;
+       ifr.ifr_data = (char __user *)&ioc;
 
        fs = get_fs();
        set_fs(get_ds());