From: kbuild test robot <lkp@intel.com>
To: Pradeep Kumar Chitrapu <pradeepc@codeaurora.org>,
ath11k@lists.infradead.org
Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com,
linux-wireless@vger.kernel.org,
Pradeep Kumar Chitrapu <pradeepc@codeaurora.org>
Subject: Re: [PATCH v4 5/8] ath11k: set psc channel flag when sending channel list to firmware.
Date: Mon, 1 Jun 2020 09:10:56 +0800 [thread overview]
Message-ID: <202006010909.iqs415R7%lkp@intel.com> (raw)
In-Reply-To: <20200531185600.20789-6-pradeepc@codeaurora.org>
[-- Attachment #1: Type: text/plain, Size: 5105 bytes --]
Hi Pradeep,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on ath6kl/ath-next]
[also build test ERROR on next-20200529]
[cannot apply to v5.7-rc7]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Pradeep-Kumar-Chitrapu/add-6GHz-radio-support-in-ath11k-driver/20200601-045939
base: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git ath-next
config: x86_64-allyesconfig (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 2388a096e7865c043e83ece4e26654bd3d1a20d5)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>
All errors (new ones prefixed by >>, old ones prefixed by <<):
>> drivers/net/wireless/ath/ath11k/reg.c:165:8: error: implicit declaration of function 'cfg80211_channel_is_psc' [-Werror,-Wimplicit-function-declaration]
cfg80211_channel_is_psc(channel))
^
drivers/net/wireless/ath/ath11k/reg.c:165:8: note: did you mean 'cfg80211_chandef_is_edmg'?
include/net/cfg80211.h:732:1: note: 'cfg80211_chandef_is_edmg' declared here
cfg80211_chandef_is_edmg(const struct cfg80211_chan_def *chandef)
^
1 error generated.
vim +/cfg80211_channel_is_psc +165 drivers/net/wireless/ath/ath11k/reg.c
89
90 int ath11k_reg_update_chan_list(struct ath11k *ar)
91 {
92 struct ieee80211_supported_band **bands;
93 struct scan_chan_list_params *params;
94 struct ieee80211_channel *channel;
95 struct ieee80211_hw *hw = ar->hw;
96 struct channel_param *ch;
97 enum nl80211_band band;
98 int num_channels = 0;
99 int params_len;
100 int i, ret;
101
102 bands = hw->wiphy->bands;
103 for (band = 0; band < NUM_NL80211_BANDS; band++) {
104 if (!bands[band])
105 continue;
106
107 for (i = 0; i < bands[band]->n_channels; i++) {
108 if (bands[band]->channels[i].flags &
109 IEEE80211_CHAN_DISABLED)
110 continue;
111
112 num_channels++;
113 }
114 }
115
116 if (WARN_ON(!num_channels))
117 return -EINVAL;
118
119 params_len = sizeof(struct scan_chan_list_params) +
120 num_channels * sizeof(struct channel_param);
121 params = kzalloc(params_len, GFP_KERNEL);
122
123 if (!params)
124 return -ENOMEM;
125
126 params->pdev_id = ar->pdev->pdev_id;
127 params->nallchans = num_channels;
128
129 ch = params->ch_param;
130
131 for (band = 0; band < NUM_NL80211_BANDS; band++) {
132 if (!bands[band])
133 continue;
134
135 for (i = 0; i < bands[band]->n_channels; i++) {
136 channel = &bands[band]->channels[i];
137
138 if (channel->flags & IEEE80211_CHAN_DISABLED)
139 continue;
140
141 /* TODO: Set to true/false based on some condition? */
142 ch->allow_ht = true;
143 ch->allow_vht = true;
144 ch->allow_he = true;
145
146 ch->dfs_set =
147 !!(channel->flags & IEEE80211_CHAN_RADAR);
148 ch->is_chan_passive = !!(channel->flags &
149 IEEE80211_CHAN_NO_IR);
150 ch->is_chan_passive |= ch->dfs_set;
151 ch->mhz = channel->center_freq;
152 ch->cfreq1 = channel->center_freq;
153 ch->minpower = 0;
154 ch->maxpower = channel->max_power * 2;
155 ch->maxregpower = channel->max_reg_power * 2;
156 ch->antennamax = channel->max_antenna_gain * 2;
157
158 /* TODO: Use appropriate phymodes */
159 if (channel->band == NL80211_BAND_2GHZ)
160 ch->phy_mode = MODE_11G;
161 else
162 ch->phy_mode = MODE_11A;
163
164 if (channel->band == NL80211_BAND_6GHZ &&
> 165 cfg80211_channel_is_psc(channel))
166 ch->psc_channel = true;
167
168 ath11k_dbg(ar->ab, ATH11K_DBG_WMI,
169 "mac channel [%d/%d] freq %d maxpower %d regpower %d antenna %d mode %d\n",
170 i, params->nallchans,
171 ch->mhz, ch->maxpower, ch->maxregpower,
172 ch->antennamax, ch->phy_mode);
173
174 ch++;
175 /* TODO: use quarrter/half rate, cfreq12, dfs_cfreq2
176 * set_agile, reg_class_idx
177 */
178 }
179 }
180
181 ret = ath11k_wmi_send_scan_chan_list_cmd(ar, params);
182 kfree(params);
183
184 return ret;
185 }
186
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 73513 bytes --]
next prev parent reply other threads:[~2020-06-01 2:21 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-31 18:55 [PATCH v4 0/8] add 6GHz radio support in ath11k driver Pradeep Kumar Chitrapu
2020-05-31 18:55 ` [PATCH v4 1/8] ath11k: add 6G frequency list supported by driver Pradeep Kumar Chitrapu
2020-05-31 18:55 ` [PATCH v4 2/8] ath11k: add support for 6GHz radio in driver Pradeep Kumar Chitrapu
2020-05-31 18:55 ` [PATCH v4 3/8] ath11k: Use freq instead of channel number in rx path Pradeep Kumar Chitrapu
2020-05-31 18:55 ` [PATCH v4 4/8] ath11k: extend peer_assoc_cmd for 6GHz band Pradeep Kumar Chitrapu
2020-05-31 18:55 ` [PATCH v4 5/8] ath11k: set psc channel flag when sending channel list to firmware Pradeep Kumar Chitrapu
2020-06-01 1:10 ` kbuild test robot [this message]
2020-06-01 5:34 ` kbuild test robot
2020-05-31 18:55 ` [PATCH v4 6/8] ath11k: Add 6G scan dwell time parameter in scan request command Pradeep Kumar Chitrapu
2020-05-31 18:55 ` [PATCH v4 7/8] ath11k: Send multiple scan_chan_list messages if required Pradeep Kumar Chitrapu
2020-05-31 18:56 ` [PATCH v4 8/8] ath11k: Add support for 6g scan hint Pradeep Kumar Chitrapu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202006010909.iqs415R7%lkp@intel.com \
--to=lkp@intel.com \
--cc=ath11k@lists.infradead.org \
--cc=clang-built-linux@googlegroups.com \
--cc=kbuild-all@lists.01.org \
--cc=linux-wireless@vger.kernel.org \
--cc=pradeepc@codeaurora.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).