All of lore.kernel.org
 help / color / mirror / Atom feed
* drivers/net/wireless/ath/ath11k/mac.c:5408:22: error: parameter 'changed_flags' set but not used
@ 2021-11-06  3:54 ` kernel test robot
  0 siblings, 0 replies; 5+ messages in thread
From: kernel test robot @ 2021-11-06  3:54 UTC (permalink / raw)
  To: Seevalamuthu Mariappan
  Cc: llvm, kbuild-all, linux-kernel, Kalle Valo, Miles Hu,
	Vasanthakumar Thiagarajan, Jouni Malinen

[-- Attachment #1: Type: text/plain, Size: 3751 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   fe91c4725aeed35023ba4f7a1e1adfebb6878c23
commit: 689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b ath11k: monitor mode clean up to use separate APIs
date:   6 weeks ago
config: hexagon-buildonly-randconfig-r006-20211013 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project b6a8c695542b2987eb9a203d5663a0740cb4725f)
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
        # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b
        git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
        git fetch --no-tags linus master
        git checkout 689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=hexagon 

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

All errors (new ones prefixed by >>):

>> drivers/net/wireless/ath/ath11k/mac.c:5408:22: error: parameter 'changed_flags' set but not used [-Werror,-Wunused-but-set-parameter]
                                              unsigned int changed_flags,
                                                           ^
   1 error generated.


vim +/changed_flags +5408 drivers/net/wireless/ath/ath11k/mac.c

d5c65159f289537 Kalle Valo 2019-11-23  5396  
d5c65159f289537 Kalle Valo 2019-11-23  5397  /* FIXME: Has to be verified. */
d5c65159f289537 Kalle Valo 2019-11-23  5398  #define SUPPORTED_FILTERS			\
d5c65159f289537 Kalle Valo 2019-11-23  5399  	(FIF_ALLMULTI |				\
d5c65159f289537 Kalle Valo 2019-11-23  5400  	FIF_CONTROL |				\
d5c65159f289537 Kalle Valo 2019-11-23  5401  	FIF_PSPOLL |				\
d5c65159f289537 Kalle Valo 2019-11-23  5402  	FIF_OTHER_BSS |				\
d5c65159f289537 Kalle Valo 2019-11-23  5403  	FIF_BCN_PRBRESP_PROMISC |		\
d5c65159f289537 Kalle Valo 2019-11-23  5404  	FIF_PROBE_REQ |				\
d5c65159f289537 Kalle Valo 2019-11-23  5405  	FIF_FCSFAIL)
d5c65159f289537 Kalle Valo 2019-11-23  5406  
d5c65159f289537 Kalle Valo 2019-11-23  5407  static void ath11k_mac_op_configure_filter(struct ieee80211_hw *hw,
d5c65159f289537 Kalle Valo 2019-11-23 @5408  					   unsigned int changed_flags,
d5c65159f289537 Kalle Valo 2019-11-23  5409  					   unsigned int *total_flags,
d5c65159f289537 Kalle Valo 2019-11-23  5410  					   u64 multicast)
d5c65159f289537 Kalle Valo 2019-11-23  5411  {
d5c65159f289537 Kalle Valo 2019-11-23  5412  	struct ath11k *ar = hw->priv;
d5c65159f289537 Kalle Valo 2019-11-23  5413  
d5c65159f289537 Kalle Valo 2019-11-23  5414  	mutex_lock(&ar->conf_mutex);
d5c65159f289537 Kalle Valo 2019-11-23  5415  
d5c65159f289537 Kalle Valo 2019-11-23  5416  	changed_flags &= SUPPORTED_FILTERS;
d5c65159f289537 Kalle Valo 2019-11-23  5417  	*total_flags &= SUPPORTED_FILTERS;
d5c65159f289537 Kalle Valo 2019-11-23  5418  	ar->filter_flags = *total_flags;
d5c65159f289537 Kalle Valo 2019-11-23  5419  
d5c65159f289537 Kalle Valo 2019-11-23  5420  	mutex_unlock(&ar->conf_mutex);
d5c65159f289537 Kalle Valo 2019-11-23  5421  }
d5c65159f289537 Kalle Valo 2019-11-23  5422  

:::::: The code at line 5408 was first introduced by commit
:::::: d5c65159f2895379e11ca13f62feabe93278985d ath11k: driver for Qualcomm IEEE 802.11ax devices

:::::: TO: Kalle Valo <kvalo@codeaurora.org>
:::::: CC: Kalle Valo <kvalo@codeaurora.org>

---
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: 29053 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* drivers/net/wireless/ath/ath11k/mac.c:5408:22: error: parameter 'changed_flags' set but not used
@ 2021-11-06  3:54 ` kernel test robot
  0 siblings, 0 replies; 5+ messages in thread
From: kernel test robot @ 2021-11-06  3:54 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 3819 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   fe91c4725aeed35023ba4f7a1e1adfebb6878c23
commit: 689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b ath11k: monitor mode clean up to use separate APIs
date:   6 weeks ago
config: hexagon-buildonly-randconfig-r006-20211013 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project b6a8c695542b2987eb9a203d5663a0740cb4725f)
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
        # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b
        git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
        git fetch --no-tags linus master
        git checkout 689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=hexagon 

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

All errors (new ones prefixed by >>):

>> drivers/net/wireless/ath/ath11k/mac.c:5408:22: error: parameter 'changed_flags' set but not used [-Werror,-Wunused-but-set-parameter]
                                              unsigned int changed_flags,
                                                           ^
   1 error generated.


vim +/changed_flags +5408 drivers/net/wireless/ath/ath11k/mac.c

d5c65159f289537 Kalle Valo 2019-11-23  5396  
d5c65159f289537 Kalle Valo 2019-11-23  5397  /* FIXME: Has to be verified. */
d5c65159f289537 Kalle Valo 2019-11-23  5398  #define SUPPORTED_FILTERS			\
d5c65159f289537 Kalle Valo 2019-11-23  5399  	(FIF_ALLMULTI |				\
d5c65159f289537 Kalle Valo 2019-11-23  5400  	FIF_CONTROL |				\
d5c65159f289537 Kalle Valo 2019-11-23  5401  	FIF_PSPOLL |				\
d5c65159f289537 Kalle Valo 2019-11-23  5402  	FIF_OTHER_BSS |				\
d5c65159f289537 Kalle Valo 2019-11-23  5403  	FIF_BCN_PRBRESP_PROMISC |		\
d5c65159f289537 Kalle Valo 2019-11-23  5404  	FIF_PROBE_REQ |				\
d5c65159f289537 Kalle Valo 2019-11-23  5405  	FIF_FCSFAIL)
d5c65159f289537 Kalle Valo 2019-11-23  5406  
d5c65159f289537 Kalle Valo 2019-11-23  5407  static void ath11k_mac_op_configure_filter(struct ieee80211_hw *hw,
d5c65159f289537 Kalle Valo 2019-11-23 @5408  					   unsigned int changed_flags,
d5c65159f289537 Kalle Valo 2019-11-23  5409  					   unsigned int *total_flags,
d5c65159f289537 Kalle Valo 2019-11-23  5410  					   u64 multicast)
d5c65159f289537 Kalle Valo 2019-11-23  5411  {
d5c65159f289537 Kalle Valo 2019-11-23  5412  	struct ath11k *ar = hw->priv;
d5c65159f289537 Kalle Valo 2019-11-23  5413  
d5c65159f289537 Kalle Valo 2019-11-23  5414  	mutex_lock(&ar->conf_mutex);
d5c65159f289537 Kalle Valo 2019-11-23  5415  
d5c65159f289537 Kalle Valo 2019-11-23  5416  	changed_flags &= SUPPORTED_FILTERS;
d5c65159f289537 Kalle Valo 2019-11-23  5417  	*total_flags &= SUPPORTED_FILTERS;
d5c65159f289537 Kalle Valo 2019-11-23  5418  	ar->filter_flags = *total_flags;
d5c65159f289537 Kalle Valo 2019-11-23  5419  
d5c65159f289537 Kalle Valo 2019-11-23  5420  	mutex_unlock(&ar->conf_mutex);
d5c65159f289537 Kalle Valo 2019-11-23  5421  }
d5c65159f289537 Kalle Valo 2019-11-23  5422  

:::::: The code at line 5408 was first introduced by commit
:::::: d5c65159f2895379e11ca13f62feabe93278985d ath11k: driver for Qualcomm IEEE 802.11ax devices

:::::: TO: Kalle Valo <kvalo@codeaurora.org>
:::::: CC: Kalle Valo <kvalo@codeaurora.org>

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

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 29053 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: drivers/net/wireless/ath/ath11k/mac.c:5408:22: error: parameter 'changed_flags' set but not used
  2021-11-06  3:54 ` kernel test robot
  (?)
@ 2021-11-08 10:55   ` Kalle Valo
  -1 siblings, 0 replies; 5+ messages in thread
From: Kalle Valo @ 2021-11-08 10:55 UTC (permalink / raw)
  To: kernel test robot
  Cc: Seevalamuthu Mariappan, llvm, kbuild-all, linux-kernel, Miles Hu,
	Vasanthakumar Thiagarajan, Jouni Malinen, ath11k

+ ath11k list

kernel test robot <lkp@intel.com> writes:

> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   fe91c4725aeed35023ba4f7a1e1adfebb6878c23
> commit: 689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b ath11k: monitor mode clean up to use separate APIs
> date:   6 weeks ago
> config: hexagon-buildonly-randconfig-r006-20211013 (attached as .config)
> compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project b6a8c695542b2987eb9a203d5663a0740cb4725f)
> 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
>         # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b
>         git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>         git fetch --no-tags linus master
>         git checkout 689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=hexagon 
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All errors (new ones prefixed by >>):
>
>>> drivers/net/wireless/ath/ath11k/mac.c:5408:22: error: parameter 'changed_flags' set but not used [-Werror,-Wunused-but-set-parameter]
>                                               unsigned int changed_flags,
>                                                            ^
>    1 error generated.

Looks like a valid warning to me, can someone submit a patch?

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: drivers/net/wireless/ath/ath11k/mac.c:5408:22: error: parameter 'changed_flags' set but not used
@ 2021-11-08 10:55   ` Kalle Valo
  0 siblings, 0 replies; 5+ messages in thread
From: Kalle Valo @ 2021-11-08 10:55 UTC (permalink / raw)
  To: kernel test robot
  Cc: Seevalamuthu Mariappan, llvm, kbuild-all, linux-kernel, Miles Hu,
	Vasanthakumar Thiagarajan, Jouni Malinen, ath11k

+ ath11k list

kernel test robot <lkp@intel.com> writes:

> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   fe91c4725aeed35023ba4f7a1e1adfebb6878c23
> commit: 689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b ath11k: monitor mode clean up to use separate APIs
> date:   6 weeks ago
> config: hexagon-buildonly-randconfig-r006-20211013 (attached as .config)
> compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project b6a8c695542b2987eb9a203d5663a0740cb4725f)
> 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
>         # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b
>         git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>         git fetch --no-tags linus master
>         git checkout 689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=hexagon 
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All errors (new ones prefixed by >>):
>
>>> drivers/net/wireless/ath/ath11k/mac.c:5408:22: error: parameter 'changed_flags' set but not used [-Werror,-Wunused-but-set-parameter]
>                                               unsigned int changed_flags,
>                                                            ^
>    1 error generated.

Looks like a valid warning to me, can someone submit a patch?

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: drivers/net/wireless/ath/ath11k/mac.c:5408:22: error: parameter 'changed_flags' set but not used
@ 2021-11-08 10:55   ` Kalle Valo
  0 siblings, 0 replies; 5+ messages in thread
From: Kalle Valo @ 2021-11-08 10:55 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 1865 bytes --]

+ ath11k list

kernel test robot <lkp@intel.com> writes:

> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   fe91c4725aeed35023ba4f7a1e1adfebb6878c23
> commit: 689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b ath11k: monitor mode clean up to use separate APIs
> date:   6 weeks ago
> config: hexagon-buildonly-randconfig-r006-20211013 (attached as .config)
> compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project b6a8c695542b2987eb9a203d5663a0740cb4725f)
> 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
>         # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b
>         git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>         git fetch --no-tags linus master
>         git checkout 689a5e6fff75229ac7c2af7a9c51dc2d3ca1882b
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=hexagon 
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All errors (new ones prefixed by >>):
>
>>> drivers/net/wireless/ath/ath11k/mac.c:5408:22: error: parameter 'changed_flags' set but not used [-Werror,-Wunused-but-set-parameter]
>                                               unsigned int changed_flags,
>                                                            ^
>    1 error generated.

Looks like a valid warning to me, can someone submit a patch?

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-11-08 10:55 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-06  3:54 drivers/net/wireless/ath/ath11k/mac.c:5408:22: error: parameter 'changed_flags' set but not used kernel test robot
2021-11-06  3:54 ` kernel test robot
2021-11-08 10:55 ` Kalle Valo
2021-11-08 10:55   ` Kalle Valo
2021-11-08 10:55   ` Kalle Valo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.