[PATCH] wifi: ath11k: retrieve MAC address from system firmware if provided
kernel test robot
lkp at intel.com
Fri Sep 16 12:41:30 PDT 2022
Hi Jun,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on kvalo-ath/ath-next]
[also build test ERROR on wireless/main wireless-next/main linus/master v6.0-rc5 next-20220916]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Jun-Yu/wifi-ath11k-retrieve-MAC-address-from-system-firmware-if-provided/20220915-055714
base: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git ath-next
config: i386-randconfig-a005 (https://download.01.org/0day-ci/archive/20220917/202209170307.720C1Sx4-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-5) 11.3.0
reproduce (this is a W=1 build):
# https://github.com/intel-lab-lkp/linux/commit/7a1539aed9ed31bd3b93c177abda0a2f4fea3945
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Jun-Yu/wifi-ath11k-retrieve-MAC-address-from-system-firmware-if-provided/20220915-055714
git checkout 7a1539aed9ed31bd3b93c177abda0a2f4fea3945
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash drivers/net/wireless/ath/ath11k/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp at intel.com>
All errors (new ones prefixed by >>):
drivers/net/wireless/ath/ath11k/mac.c: In function 'ath11k_mac_register':
>> drivers/net/wireless/ath/ath11k/mac.c:8999:9: error: too many arguments to function 'device_get_mac_address'
8999 | device_get_mac_address(ab->dev, mac_addr, sizeof(ab->mac_addr));
| ^~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/ieee80211.h:20,
from include/net/mac80211.h:20,
from drivers/net/wireless/ath/ath11k/mac.c:7:
include/linux/etherdevice.h:35:5: note: declared here
35 | int device_get_mac_address(struct device *dev, char *addr);
| ^~~~~~~~~~~~~~~~~~~~~~
vim +/device_get_mac_address +8999 drivers/net/wireless/ath/ath11k/mac.c
8979
8980 int ath11k_mac_register(struct ath11k_base *ab)
8981 {
8982 struct ath11k *ar;
8983 struct ath11k_pdev *pdev;
8984 int i;
8985 int ret;
8986 u8 mac_addr[ETH_ALEN] = {0};
8987
8988 if (test_bit(ATH11K_FLAG_REGISTERED, &ab->dev_flags))
8989 return 0;
8990
8991 /* Initialize channel counters frequency value in hertz */
8992 ab->cc_freq_hz = IPQ8074_CC_FREQ_HERTZ;
8993 ab->free_vdev_map = (1LL << (ab->num_radios * TARGET_NUM_VDEVS(ab))) - 1;
8994
8995 ret = ath11k_peer_rhash_tbl_init(ab);
8996 if (ret)
8997 return ret;
8998
> 8999 device_get_mac_address(ab->dev, mac_addr, sizeof(ab->mac_addr));
9000
9001 for (i = 0; i < ab->num_radios; i++) {
9002 pdev = &ab->pdevs[i];
9003 ar = pdev->ar;
9004 if (ab->pdevs_macaddr_valid) {
9005 ether_addr_copy(ar->mac_addr, pdev->mac_addr);
9006 } else {
9007 if (is_zero_ether_addr(mac_addr))
9008 ether_addr_copy(ar->mac_addr, ab->mac_addr);
9009 else
9010 ether_addr_copy(ar->mac_addr, mac_addr);
9011 ar->mac_addr[4] += i;
9012 }
9013
9014 idr_init(&ar->txmgmt_idr);
9015 spin_lock_init(&ar->txmgmt_idr_lock);
9016
9017 ret = __ath11k_mac_register(ar);
9018 if (ret)
9019 goto err_cleanup;
9020
9021 init_waitqueue_head(&ar->txmgmt_empty_waitq);
9022 }
9023
9024 return 0;
9025
9026 err_cleanup:
9027 for (i = i - 1; i >= 0; i--) {
9028 pdev = &ab->pdevs[i];
9029 ar = pdev->ar;
9030 __ath11k_mac_unregister(ar);
9031 }
9032
9033 ath11k_peer_rhash_tbl_destroy(ab);
9034
9035 return ret;
9036 }
9037
--
0-DAY CI Kernel Test Service
https://01.org/lkp
More information about the ath11k
mailing list