Christian Lamparter <chunk...@gmail.com> writes: > On Thursday, 27 August 2020 11:33:51 CEST Lee Jones wrote: >> 'ar9170_qmap' is used in some source files which include carl9170.h, >> but not all of them. A 'defined but not used' warning is thrown when >> compiling the ones which do not use it. >> >> Fixes the following W=1 kernel build warning(s) >> >> from drivers/net/wireless/ath/carl9170/carl9170.h:57, >> In file included from drivers/net/wireless/ath/carl9170/carl9170.h:57, >> drivers/net/wireless/ath/carl9170/carl9170.h:71:17: warning: >> ‘ar9170_qmap’ defined but not used [-Wunused-const-variable=] >> >> NB: Snipped - lots of these repeat >> >> Cc: Christian Lamparter <chunk...@googlemail.com> >> Cc: Kalle Valo <kv...@codeaurora.org> >> Cc: "David S. Miller" <da...@davemloft.net> >> Cc: Jakub Kicinski <k...@kernel.org> >> Cc: Johannes Berg <johan...@sipsolutions.net> >> Cc: linux-wirel...@vger.kernel.org >> Cc: net...@vger.kernel.org >> Suggested-by: Rasmus Villemoes <li...@rasmusvillemoes.dk> >> Signed-off-by: Lee Jones <lee.jo...@linaro.org> >> --- > > For what it's worth: > Acked-by: Christian Lamparter <chunk...@gmail.com>
BTW for me Acked-by tags from the maintainer are very useful. Patchwork even collects them automatically and shows the statistics so I can quickly see what patches are ready to be applied. So please do send them if you can :) -- https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
