BCC: l...@intel.com
CC: kbuild-...@lists.01.org
In-Reply-To: <20220830022017.51017-2-kieran.fre...@morsemicro.com>
References: <20220830022017.51017-2-kieran.fre...@morsemicro.com>
TO: Kieran Frewen <kieran.fre...@morsemicro.com>
TO: johan...@sipsolutions.net
CC: linux-wirel...@vger.kernel.org
CC: quic_jjohn...@quicinc.com
CC: Kieran Frewen <kieran.fre...@morsemicro.com>

Hi Kieran,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on wireless/main]
[also build test WARNING on linus/master v6.0-rc3 next-20220901]
[cannot apply to wireless-next/main]
[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/Kieran-Frewen/Additional-Support-for-802-11ah-S1G/20220830-102217
base:   https://git.kernel.org/pub/scm/linux/kernel/git/wireless/wireless.git 
main
:::::: branch date: 3 days ago
:::::: commit date: 3 days ago
config: csky-randconfig-m031-20220901 
(https://download.01.org/0day-ci/archive/20220902/202209020018.s33cd0dc-...@intel.com/config)
compiler: csky-linux-gcc (GCC) 12.1.0

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <l...@intel.com>
Reported-by: Dan Carpenter <dan.carpen...@oracle.com>

smatch warnings:
net/wireless/reg.c:2597 handle_channel_custom() error: uninitialized symbol 
'center_freq_khz'.

vim +/center_freq_khz +2597 net/wireless/reg.c

d7549cbb9ab067 Sven Neumann      2011-08-30  2577  
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2578  static void 
handle_channel_custom(struct wiphy *wiphy,
fdc9d7b2863ce6 Johannes Berg     2012-12-03  2579                               
  struct ieee80211_channel *chan,
c4b9d655e445a8 Ganapathi Bhat    2019-12-20  2580                               
  const struct ieee80211_regdomain *regd,
c4b9d655e445a8 Ganapathi Bhat    2019-12-20  2581                               
  u32 min_bw)
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2582  {
038659e7c6b385 Luis R. Rodriguez 2009-05-02  2583       u32 bw_flags = 0;
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2584       const struct 
ieee80211_reg_rule *reg_rule = NULL;
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2585       const struct 
ieee80211_power_rule *power_rule = NULL;
934f4c7dd3a544 Thomas Pedersen   2020-04-01  2586       u32 bw, center_freq_khz;
7c0e7a45e22911 Kieran Frewen     2022-08-30  2587       bool is_s1g = 
chan->band == NL80211_BAND_S1GHZ;
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2588  
7c0e7a45e22911 Kieran Frewen     2022-08-30  2589       if (is_s1g) {
7c0e7a45e22911 Kieran Frewen     2022-08-30  2590               bw = 
MHZ_TO_KHZ(16);
7c0e7a45e22911 Kieran Frewen     2022-08-30  2591               min_bw = 
MHZ_TO_KHZ(1);
7c0e7a45e22911 Kieran Frewen     2022-08-30  2592       } else {
7c0e7a45e22911 Kieran Frewen     2022-08-30  2593               bw = 
MHZ_TO_KHZ(20);
7c0e7a45e22911 Kieran Frewen     2022-08-30  2594       }
7c0e7a45e22911 Kieran Frewen     2022-08-30  2595  
7c0e7a45e22911 Kieran Frewen     2022-08-30  2596       for (; bw >= min_bw; bw 
= bw / 2) {
934f4c7dd3a544 Thomas Pedersen   2020-04-01 @2597               reg_rule = 
freq_reg_info_regd(center_freq_khz, regd, bw);
4edd56981c8fbb Matthias May      2015-07-17  2598               if 
(!IS_ERR(reg_rule))
4edd56981c8fbb Matthias May      2015-07-17  2599                       break;
4edd56981c8fbb Matthias May      2015-07-17  2600       }
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2601  
a7ee7d44b57c9a Johannes Berg     2020-02-21  2602       if 
(IS_ERR_OR_NULL(reg_rule)) {
934f4c7dd3a544 Thomas Pedersen   2020-04-01  2603               
pr_debug("Disabling freq %d.%03d MHz as custom regd has no rule that fits it\n",
934f4c7dd3a544 Thomas Pedersen   2020-04-01  2604                        
chan->center_freq, chan->freq_offset);
db8dfee57d37d2 Arik Nemtsov      2014-12-15  2605               if 
(wiphy->regulatory_flags & REGULATORY_WIPHY_SELF_MANAGED) {
db8dfee57d37d2 Arik Nemtsov      2014-12-15  2606                       
chan->flags |= IEEE80211_CHAN_DISABLED;
db8dfee57d37d2 Arik Nemtsov      2014-12-15  2607               } else {
cc493e4f5296f4 Luis R. Rodriguez 2013-11-06  2608                       
chan->orig_flags |= IEEE80211_CHAN_DISABLED;
cc493e4f5296f4 Luis R. Rodriguez 2013-11-06  2609                       
chan->flags = chan->orig_flags;
db8dfee57d37d2 Arik Nemtsov      2014-12-15  2610               }
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2611               return;
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2612       }
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2613  
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2614       power_rule = 
&reg_rule->power_rule;
1aeb135f84fe40 Michal Sojka      2015-11-23  2615       bw_flags = 
reg_rule_to_chan_bw_flags(regd, reg_rule, chan);
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2616  
2e18b38fc8fb03 Arik Nemtsov      2014-11-16  2617       chan->dfs_state_entered 
= jiffies;
c7ab508190aee6 Arik Nemtsov      2014-11-16  2618       chan->dfs_state = 
NL80211_DFS_USABLE;
c7ab508190aee6 Arik Nemtsov      2014-11-16  2619  
c7ab508190aee6 Arik Nemtsov      2014-11-16  2620       chan->beacon_found = 
false;
db8dfee57d37d2 Arik Nemtsov      2014-12-15  2621  
db8dfee57d37d2 Arik Nemtsov      2014-12-15  2622       if 
(wiphy->regulatory_flags & REGULATORY_WIPHY_SELF_MANAGED)
db8dfee57d37d2 Arik Nemtsov      2014-12-15  2623               chan->flags = 
chan->orig_flags | bw_flags |
db8dfee57d37d2 Arik Nemtsov      2014-12-15  2624                             
map_regdom_flags(reg_rule->flags);
db8dfee57d37d2 Arik Nemtsov      2014-12-15  2625       else
038659e7c6b385 Luis R. Rodriguez 2009-05-02  2626               chan->flags |= 
map_regdom_flags(reg_rule->flags) | bw_flags;
db8dfee57d37d2 Arik Nemtsov      2014-12-15  2627  
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2628       chan->max_antenna_gain 
= (int) MBI_TO_DBI(power_rule->max_antenna_gain);
279f0f55249820 Felix Fietkau     2012-10-17  2629       chan->max_reg_power = 
chan->max_power =
279f0f55249820 Felix Fietkau     2012-10-17  2630               (int) 
MBM_TO_DBM(power_rule->max_eirp);
2e18b38fc8fb03 Arik Nemtsov      2014-11-16  2631  
2e18b38fc8fb03 Arik Nemtsov      2014-11-16  2632       if (chan->flags & 
IEEE80211_CHAN_RADAR) {
2e18b38fc8fb03 Arik Nemtsov      2014-11-16  2633               if 
(reg_rule->dfs_cac_ms)
2e18b38fc8fb03 Arik Nemtsov      2014-11-16  2634                       
chan->dfs_cac_ms = reg_rule->dfs_cac_ms;
2e18b38fc8fb03 Arik Nemtsov      2014-11-16  2635               else
2e18b38fc8fb03 Arik Nemtsov      2014-11-16  2636                       
chan->dfs_cac_ms = IEEE80211_DFS_MIN_CAC_TIME_MS;
2e18b38fc8fb03 Arik Nemtsov      2014-11-16  2637       }
2e18b38fc8fb03 Arik Nemtsov      2014-11-16  2638  
2e18b38fc8fb03 Arik Nemtsov      2014-11-16  2639       chan->max_power = 
chan->max_reg_power;
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2640  }
1fa25e413659f9 Luis R. Rodriguez 2009-01-22  2641  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp
_______________________________________________
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org

Reply via email to