From: kernel test robot <lkp@intel.com>
To: David Lin <yu-hao.lin@nxp.com>, linux-wireless@vger.kernel.org
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
linux-kernel@vger.kernel.org, briannorris@chromium.org,
kvalo@kernel.org, francesco@dolcini.it,
tsung-hsien.hsieh@nxp.com, David Lin <yu-hao.lin@nxp.com>
Subject: Re: [PATCH 42/43] wifi: nxpwifi: add Makefile and Kconfig files for nxpwifi compilation
Date: Wed, 26 Jun 2024 08:19:02 +0800 [thread overview]
Message-ID: <202406260755.fqLnsFhr-lkp@intel.com> (raw)
In-Reply-To: <20240621075208.513497-43-yu-hao.lin@nxp.com>
Hi David,
kernel test robot noticed the following build warnings:
[auto build test WARNING on 238d636723a30311e20fde0a361662e829fe488b]
url: https://github.com/intel-lab-lkp/linux/commits/David-Lin/wifi-nxpwifi-add-11ac-c/20240625-161306
base: 238d636723a30311e20fde0a361662e829fe488b
patch link: https://lore.kernel.org/r/20240621075208.513497-43-yu-hao.lin%40nxp.com
patch subject: [PATCH 42/43] wifi: nxpwifi: add Makefile and Kconfig files for nxpwifi compilation
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20240626/202406260755.fqLnsFhr-lkp@intel.com/config)
compiler: clang version 18.1.5 (https://github.com/llvm/llvm-project 617a15a9eac96088ae5e9134248d8236e34b91b1)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240626/202406260755.fqLnsFhr-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202406260755.fqLnsFhr-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/net/wireless/nxp/nxpwifi/11h.c:208:7: warning: variable 'chan2_offset' is used uninitialized whenever switch case is taken [-Wsometimes-uninitialized]
208 | case NL80211_CHAN_WIDTH_80P80:
| ^~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:214:16: note: uninitialized use occurs here
214 | *band_cfg = ((chan2_offset << BAND_CFG_CHAN2_SHIFT_BIT) &
| ^~~~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:209:7: warning: variable 'chan2_offset' is used uninitialized whenever switch case is taken [-Wsometimes-uninitialized]
209 | case NL80211_CHAN_WIDTH_160:
| ^~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:214:16: note: uninitialized use occurs here
214 | *band_cfg = ((chan2_offset << BAND_CFG_CHAN2_SHIFT_BIT) &
| ^~~~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:192:7: warning: variable 'chan2_offset' is used uninitialized whenever switch case is taken [-Wsometimes-uninitialized]
192 | case NL80211_CHAN_WIDTH_20_NOHT:
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:214:16: note: uninitialized use occurs here
214 | *band_cfg = ((chan2_offset << BAND_CFG_CHAN2_SHIFT_BIT) &
| ^~~~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:193:7: warning: variable 'chan2_offset' is used uninitialized whenever switch case is taken [-Wsometimes-uninitialized]
193 | case NL80211_CHAN_WIDTH_20:
| ^~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:214:16: note: uninitialized use occurs here
214 | *band_cfg = ((chan2_offset << BAND_CFG_CHAN2_SHIFT_BIT) &
| ^~~~~~~~~~~~
>> drivers/net/wireless/nxp/nxpwifi/11h.c:210:2: warning: variable 'chan2_offset' is used uninitialized whenever switch default is taken [-Wsometimes-uninitialized]
210 | default:
| ^~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:214:16: note: uninitialized use occurs here
214 | *band_cfg = ((chan2_offset << BAND_CFG_CHAN2_SHIFT_BIT) &
| ^~~~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:178:40: note: initialize the variable 'chan2_offset' to silence this warning
178 | u8 chan_band, chan_width, chan2_offset;
| ^
| = '\0'
>> drivers/net/wireless/nxp/nxpwifi/11h.c:208:7: warning: variable 'chan_width' is used uninitialized whenever switch case is taken [-Wsometimes-uninitialized]
208 | case NL80211_CHAN_WIDTH_80P80:
| ^~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:216:9: note: uninitialized use occurs here
216 | ((chan_width << BAND_CFG_CHAN_WIDTH_SHIFT_BIT) &
| ^~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:209:7: warning: variable 'chan_width' is used uninitialized whenever switch case is taken [-Wsometimes-uninitialized]
209 | case NL80211_CHAN_WIDTH_160:
| ^~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:216:9: note: uninitialized use occurs here
216 | ((chan_width << BAND_CFG_CHAN_WIDTH_SHIFT_BIT) &
| ^~~~~~~~~~
>> drivers/net/wireless/nxp/nxpwifi/11h.c:210:2: warning: variable 'chan_width' is used uninitialized whenever switch default is taken [-Wsometimes-uninitialized]
210 | default:
| ^~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:216:9: note: uninitialized use occurs here
216 | ((chan_width << BAND_CFG_CHAN_WIDTH_SHIFT_BIT) &
| ^~~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:178:26: note: initialize the variable 'chan_width' to silence this warning
178 | u8 chan_band, chan_width, chan2_offset;
| ^
| = '\0'
>> drivers/net/wireless/nxp/nxpwifi/11h.c:187:2: warning: variable 'chan_band' is used uninitialized whenever switch default is taken [-Wsometimes-uninitialized]
187 | default:
| ^~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:218:9: note: uninitialized use occurs here
218 | ((chan_band << BAND_CFG_CHAN_BAND_SHIFT_BIT) &
| ^~~~~~~~~
drivers/net/wireless/nxp/nxpwifi/11h.c:178:14: note: initialize the variable 'chan_band' to silence this warning
178 | u8 chan_band, chan_width, chan2_offset;
| ^
| = '\0'
9 warnings generated.
vim +/chan2_offset +208 drivers/net/wireless/nxp/nxpwifi/11h.c
647388e635d138 David Lin 2024-06-21 174
647388e635d138 David Lin 2024-06-21 175 static void nxpwifi_convert_chan_to_band_cfg(u8 *band_cfg,
647388e635d138 David Lin 2024-06-21 176 struct cfg80211_chan_def *chan_def)
647388e635d138 David Lin 2024-06-21 177 {
647388e635d138 David Lin 2024-06-21 178 u8 chan_band, chan_width, chan2_offset;
647388e635d138 David Lin 2024-06-21 179
647388e635d138 David Lin 2024-06-21 180 switch (chan_def->chan->band) {
647388e635d138 David Lin 2024-06-21 181 case NL80211_BAND_2GHZ:
647388e635d138 David Lin 2024-06-21 182 chan_band = BAND_2GHZ;
647388e635d138 David Lin 2024-06-21 183 break;
647388e635d138 David Lin 2024-06-21 184 case NL80211_BAND_5GHZ:
647388e635d138 David Lin 2024-06-21 185 chan_band = BAND_5GHZ;
647388e635d138 David Lin 2024-06-21 186 break;
647388e635d138 David Lin 2024-06-21 @187 default:
647388e635d138 David Lin 2024-06-21 188 break;
647388e635d138 David Lin 2024-06-21 189 }
647388e635d138 David Lin 2024-06-21 190
647388e635d138 David Lin 2024-06-21 191 switch (chan_def->width) {
647388e635d138 David Lin 2024-06-21 192 case NL80211_CHAN_WIDTH_20_NOHT:
647388e635d138 David Lin 2024-06-21 193 case NL80211_CHAN_WIDTH_20:
647388e635d138 David Lin 2024-06-21 194 chan_width = CHAN_BW_20MHZ;
647388e635d138 David Lin 2024-06-21 195 break;
647388e635d138 David Lin 2024-06-21 196 case NL80211_CHAN_WIDTH_40:
647388e635d138 David Lin 2024-06-21 197 chan_width = CHAN_BW_40MHZ;
647388e635d138 David Lin 2024-06-21 198 if (chan_def->center_freq1 > chan_def->chan->center_freq)
647388e635d138 David Lin 2024-06-21 199 chan2_offset = SEC_CHAN_ABOVE;
647388e635d138 David Lin 2024-06-21 200 else
647388e635d138 David Lin 2024-06-21 201 chan2_offset = SEC_CHAN_BELOW;
647388e635d138 David Lin 2024-06-21 202 break;
647388e635d138 David Lin 2024-06-21 203 case NL80211_CHAN_WIDTH_80:
647388e635d138 David Lin 2024-06-21 204 chan2_offset =
647388e635d138 David Lin 2024-06-21 205 nxpwifi_get_channel_2_offset(chan_def->chan->hw_value);
647388e635d138 David Lin 2024-06-21 206 chan_width = CHAN_BW_80MHZ;
647388e635d138 David Lin 2024-06-21 207 break;
647388e635d138 David Lin 2024-06-21 @208 case NL80211_CHAN_WIDTH_80P80:
647388e635d138 David Lin 2024-06-21 209 case NL80211_CHAN_WIDTH_160:
647388e635d138 David Lin 2024-06-21 @210 default:
647388e635d138 David Lin 2024-06-21 211 break;
647388e635d138 David Lin 2024-06-21 212 }
647388e635d138 David Lin 2024-06-21 213
647388e635d138 David Lin 2024-06-21 @214 *band_cfg = ((chan2_offset << BAND_CFG_CHAN2_SHIFT_BIT) &
647388e635d138 David Lin 2024-06-21 215 BAND_CFG_CHAN2_OFFSET_MASK) |
647388e635d138 David Lin 2024-06-21 @216 ((chan_width << BAND_CFG_CHAN_WIDTH_SHIFT_BIT) &
647388e635d138 David Lin 2024-06-21 217 BAND_CFG_CHAN_WIDTH_MASK) |
647388e635d138 David Lin 2024-06-21 218 ((chan_band << BAND_CFG_CHAN_BAND_SHIFT_BIT) &
647388e635d138 David Lin 2024-06-21 219 BAND_CFG_CHAN_BAND_MASK);
647388e635d138 David Lin 2024-06-21 220 }
647388e635d138 David Lin 2024-06-21 221
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2024-06-26 0:19 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-21 7:51 [PATCH 00/43] wifi: nxpwifi: create nxpwifi to support iw61x David Lin
2024-06-21 7:51 ` [PATCH 01/43] wifi: nxpwifi: add 11ac.c David Lin
2024-06-28 8:22 ` Abel Vesa
2024-07-01 0:46 ` [EXT] " David Lin
2024-06-21 7:51 ` [PATCH 02/43] wifi: nxpwifi: add 11ac.h David Lin
2024-06-21 7:51 ` [PATCH 03/43] wifi: nxpwifi: add 11h.c David Lin
2024-06-21 7:51 ` [PATCH 04/43] wifi: nxpwifi: add 11n.c David Lin
2024-06-21 7:51 ` [PATCH 05/43] wifi: nxpwifi: add 11n.h David Lin
2024-06-21 7:51 ` [PATCH 06/43] wifi: nxpwifi: add 11n_aggr.c David Lin
2024-06-21 7:51 ` [PATCH 07/43] wifi: nxpwifi: add 11n_aggr.h David Lin
2024-06-21 7:51 ` [PATCH 08/43] wifi: nxpwifi: add 11n_rxreorder.c David Lin
2024-06-21 7:51 ` [PATCH 09/43] wifi: nxpwifi: add 11n_rxreorder.h David Lin
2024-06-21 7:51 ` [PATCH 10/43] wifi: nxpwifi: add cfg80211.c David Lin
2024-06-21 7:51 ` [PATCH 11/43] wifi: nxpwifi: add cfg80211.h David Lin
2024-06-21 7:51 ` [PATCH 12/43] wifi: nxpwifi: add cfp.c David Lin
2024-06-21 7:51 ` [PATCH 13/43] wifi: nxpwifi: add cmdevt.c David Lin
2024-06-21 7:51 ` [PATCH 14/43] wifi: nxpwifi: add cmdevt.h David Lin
2024-06-21 7:51 ` [PATCH 15/43] wifi: nxpwifi: add debugfs.c David Lin
2024-06-21 7:51 ` [PATCH 16/43] wifi: nxpwifi: add decl.h David Lin
2024-06-21 7:51 ` [PATCH 17/43] wifi: nxpwifi: add ethtool.c David Lin
2024-06-21 7:51 ` [PATCH 18/43] wifi: nxpwifi: add fw.h David Lin
2024-06-21 7:51 ` [PATCH 19/43] wifi: nxpwifi: add ie.c David Lin
2024-06-21 7:51 ` [PATCH 20/43] wifi: nxpwifi: add init.c David Lin
2024-06-21 7:51 ` [PATCH 21/43] wifi: nxpwifi: add ioctl.h David Lin
2024-06-21 7:51 ` [PATCH 22/43] wifi: nxpwifi: add join.c David Lin
2024-06-21 7:51 ` [PATCH 23/43] wifi: nxpwifi: add main.c David Lin
2024-06-21 7:51 ` [PATCH 24/43] wifi: nxpwifi: add main.h David Lin
2024-06-21 7:51 ` [PATCH 25/43] wifi: nxpwifi: add scan.c David Lin
2024-06-21 7:51 ` [PATCH 26/43] wifi: nxpwifi: add sdio.c David Lin
2024-06-26 11:40 ` [EXTERNAL] " Nemanov, Michael
2024-06-27 3:37 ` [EXT] " David Lin
2024-06-27 6:26 ` Nemanov, Michael
2024-06-27 6:33 ` [EXT] " David Lin
2024-06-21 7:51 ` [PATCH 27/43] wifi: nxpwifi: add sdio.h David Lin
2024-06-21 7:51 ` [PATCH 28/43] wifi: nxpwifi: add sta_cmd.c David Lin
2024-06-21 7:51 ` [PATCH 29/43] wifi: nxpwifi: add sta_event.c David Lin
2024-06-21 7:51 ` [PATCH 30/43] wifi: nxpwifi: add sta_ioctl.c David Lin
2024-06-21 7:51 ` [PATCH 31/43] wifi: nxpwifi: add sta_rx.c David Lin
2024-06-21 7:51 ` [PATCH 32/43] wifi: nxpwifi: add sta_tx.c David Lin
2024-06-21 7:51 ` [PATCH 33/43] wifi: nxpwifi: add txrx.c David Lin
2024-06-21 7:51 ` [PATCH 34/43] wifi: nxpwifi: add uap_cmd.c David Lin
2024-06-21 7:52 ` [PATCH 35/43] wifi: nxpwifi: add uap_event.c David Lin
2024-06-21 7:52 ` [PATCH 36/43] wifi: nxpwifi: add uap_txrx.c David Lin
2024-06-21 7:52 ` [PATCH 37/43] wifi: nxpwifi: add util.c David Lin
2024-06-21 7:52 ` [PATCH 38/43] wifi: nxpwifi: add util.h David Lin
2024-06-21 7:52 ` [PATCH 39/43] wifi: nxpwifi: add wmm.c David Lin
2024-06-21 7:52 ` [PATCH 40/43] wifi: nxpwifi: add wmm.h David Lin
2024-06-21 7:52 ` [PATCH 41/43] wifi: nxpwifi: add nxp sdio vendor id and iw61x device id David Lin
2024-06-21 7:52 ` [PATCH 42/43] wifi: nxpwifi: add Makefile and Kconfig files for nxpwifi compilation David Lin
2024-06-26 0:19 ` kernel test robot [this message]
2024-06-26 0:24 ` kernel test robot
2024-06-28 4:46 ` kernel test robot
2024-06-21 7:52 ` [PATCH 43/43] wifi: nxpwifi: add nxpwifi related information to MAINTAINERS David Lin
2024-06-21 17:53 ` [PATCH 00/43] wifi: nxpwifi: create nxpwifi to support iw61x Brian Norris
2024-06-21 18:20 ` Johannes Berg
2024-07-01 1:08 ` [EXT] " David Lin
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=202406260755.fqLnsFhr-lkp@intel.com \
--to=lkp@intel.com \
--cc=briannorris@chromium.org \
--cc=francesco@dolcini.it \
--cc=kvalo@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=tsung-hsien.hsieh@nxp.com \
--cc=yu-hao.lin@nxp.com \
/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
all inboxes | Powered by JetHome®