Search Linux Wireless

Re: [PATCH v3 2/7] wcn36xx: Add wcn36xx_smd_set_sta_default_vht_params()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Bryan,

I love your patch! Perhaps something to improve:

[auto build test WARNING on wireless-drivers-next/master]
[also build test WARNING on wireless-drivers/master ath6kl/ath-next v5.9-rc4 next-20200908]
[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]

url:    https://github.com/0day-ci/linux/commits/Bryan-O-Donoghue/wcn36xx-Add-a-set-of-helpers-to-enable-VHT-parameter-passing/20200909-115630
base:   https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git master
config: i386-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
        # save the attached .config to linux build tree
        make W=1 ARCH=i386 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All warnings (new ones prefixed by >>):

   In file included from drivers/net/wireless/ath/wcn36xx/smd.h:20,
                    from drivers/net/wireless/ath/wcn36xx/smd.c:23:
   drivers/net/wireless/ath/wcn36xx/wcn36xx.h:272:42: warning: 'struct wcn36xx_hal_supported_rates_v1' declared inside parameter list will not be visible outside of this definition or declaration
     272 | void wcn36xx_set_default_rates_v1(struct wcn36xx_hal_supported_rates_v1 *rates);
         |                                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/net/wireless/ath/wcn36xx/smd.c:192:6: warning: no previous prototype for 'wcn36xx_smd_set_sta_default_vht_params' [-Wmissing-prototypes]
     192 | void wcn36xx_smd_set_sta_default_vht_params(struct wcn36xx *wcn,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/net/wireless/ath/wcn36xx/smd.c: In function 'wcn36xx_smd_set_sta_default_vht_params':
   drivers/net/wireless/ath/wcn36xx/smd.c:195:20: error: 'RF_IRIS_WCN3680' undeclared (first use in this function); did you mean 'RF_IRIS_WCN3620'?
     195 |  if (wcn->rf_id == RF_IRIS_WCN3680) {
         |                    ^~~~~~~~~~~~~~~
         |                    RF_IRIS_WCN3620
   drivers/net/wireless/ath/wcn36xx/smd.c:195:20: note: each undeclared identifier is reported only once for each function it appears in
   drivers/net/wireless/ath/wcn36xx/smd.c:196:15: error: 'struct wcn36xx_hal_config_sta_params_v1' has no member named 'vht_capable'; did you mean 'ht_capable'?
     196 |   sta_params->vht_capable = 1;
         |               ^~~~~~~~~~~
         |               ht_capable
   drivers/net/wireless/ath/wcn36xx/smd.c:197:13: error: 'struct wcn36xx_hal_config_sta_params_v1' has no member named 'vht_tx_mu_beamformee_capable'
     197 |   sta_params->vht_tx_mu_beamformee_capable = 1;
         |             ^~
   drivers/net/wireless/ath/wcn36xx/smd.c:199:15: error: 'struct wcn36xx_hal_config_sta_params_v1' has no member named 'vht_capable'; did you mean 'ht_capable'?
     199 |   sta_params->vht_capable = 0;
         |               ^~~~~~~~~~~
         |               ht_capable
   drivers/net/wireless/ath/wcn36xx/smd.c:200:13: error: 'struct wcn36xx_hal_config_sta_params_v1' has no member named 'vht_tx_mu_beamformee_capable'
     200 |   sta_params->vht_tx_mu_beamformee_capable = 0;
         |             ^~
   drivers/net/wireless/ath/wcn36xx/smd.c:203:12: error: 'struct wcn36xx_hal_config_sta_params_v1' has no member named 'vht_ldpc_enabled'
     203 |  sta_params->vht_ldpc_enabled = 0;
         |            ^~
   drivers/net/wireless/ath/wcn36xx/smd.c:204:14: error: 'struct wcn36xx_hal_config_sta_params_v1' has no member named 'vht_tx_channel_width_set'; did you mean 'tx_channel_width_set'?
     204 |  sta_params->vht_tx_channel_width_set = 0;
         |              ^~~~~~~~~~~~~~~~~~~~~~~~
         |              tx_channel_width_set
   drivers/net/wireless/ath/wcn36xx/smd.c:205:12: error: 'struct wcn36xx_hal_config_sta_params_v1' has no member named 'vht_tx_bf_enabled'
     205 |  sta_params->vht_tx_bf_enabled = 0;
         |            ^~

# https://github.com/0day-ci/linux/commit/0b840a269eba144bf378cdbd20160813582915a2
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Bryan-O-Donoghue/wcn36xx-Add-a-set-of-helpers-to-enable-VHT-parameter-passing/20200909-115630
git checkout 0b840a269eba144bf378cdbd20160813582915a2
vim +/wcn36xx_smd_set_sta_default_vht_params +192 drivers/net/wireless/ath/wcn36xx/smd.c

   191	
 > 192	void wcn36xx_smd_set_sta_default_vht_params(struct wcn36xx *wcn,
   193			struct wcn36xx_hal_config_sta_params_v1 *sta_params)
   194	{
   195		if (wcn->rf_id == RF_IRIS_WCN3680) {
   196			sta_params->vht_capable = 1;
   197			sta_params->vht_tx_mu_beamformee_capable = 1;
   198		} else {
   199			sta_params->vht_capable = 0;
   200			sta_params->vht_tx_mu_beamformee_capable = 0;
   201		}
   202	
   203		sta_params->vht_ldpc_enabled = 0;
   204		sta_params->vht_tx_channel_width_set = 0;
   205		sta_params->vht_tx_bf_enabled = 0;
   206	}
   207	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Wireless Regulations]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux