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

[kbuild test robot]

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

.config.gz
Description: application/gzip