Re: [PATCH] mwifiex: p2p: use separate device address

[kbuild test robot <lkp@xxxxxxxxx>]

Hi Xinming,

[auto build test WARNING on wireless-drivers-next/master]
[also build test WARNING on v4.13-rc3 next-20170804]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Xinming-Hu/mwifiex-p2p-use-separate-device-address/20170805-130855
base:   https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git master
config: mips-allmodconfig (attached as .config)
compiler: mips-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
        wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=mips 

All warnings (new ones prefixed by >>):

   In file included from include/linux/kernel.h:10:0,
                    from include/linux/list.h:8,
                    from include/linux/preempt.h:10,
                    from include/linux/spinlock.h:50,
                    from include/linux/swap.h:4,
                    from include/linux/suspend.h:4,
                    from drivers/net//wireless/marvell/mwifiex/main.c:20:
   drivers/net//wireless/marvell/mwifiex/main.c: In function 'mwifiex_set_mac_address':
   include/linux/bitops.h:6:24: warning: left shift count >= width of type [-Wshift-count-overflow]
    #define BIT(nr)   (1UL << (nr))
                           ^
>> drivers/net//wireless/marvell/mwifiex/main.c:953:14: note: in expansion of macro 'BIT'
     mac_addr |= BIT(MWIFIEX_MAC_LOCAL_ADMIN_BIT);
                 ^~~

vim +/BIT +953 drivers/net//wireless/marvell/mwifiex/main.c

   942	
   943	int mwifiex_set_mac_address(struct mwifiex_private *priv,
   944				    struct net_device *dev)
   945	{
   946		int ret;
   947		u64 mac_addr;
   948	
   949		if (priv->bss_type != MWIFIEX_BSS_TYPE_P2P)
   950			goto done;
   951	
   952		mac_addr = ether_addr_to_u64(priv->curr_addr);
 > 953		mac_addr |= BIT(MWIFIEX_MAC_LOCAL_ADMIN_BIT);
   954		u64_to_ether_addr(mac_addr, priv->curr_addr);
   955	
   956		/* Send request to firmware */
   957		ret = mwifiex_send_cmd(priv, HostCmd_CMD_802_11_MAC_ADDRESS,
   958				       HostCmd_ACT_GEN_SET, 0, NULL, true);
   959	
   960		if (ret) {
   961			mwifiex_dbg(priv->adapter, ERROR,
   962				    "set mac address failed: ret=%d\n", ret);
   963			return ret;
   964		}
   965	
   966	done:
   967		memcpy(dev->dev_addr, priv->curr_addr, ETH_ALEN);
   968		return 0;
   969	}
   970	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip