2019-01-30 23:46:40

by Stephen Rothwell

[permalink] [raw]
Subject: linux-next: build warning after merge of the wireless-drivers-next tree

Hi all,

After merging the wireless-drivers-next tree, today's linux-next build
(x86_64 allmodconfig) produced this warning:

drivers/net/wireless/intel/iwlwifi/mvm/rxmq.c:195:13: warning: 'iwl_mvm_add_rtap_sniffer_config' defined but not used [-Wunused-function]
static void iwl_mvm_add_rtap_sniffer_config(struct iwl_mvm *mvm,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Introduced by commit

9bf13bee2d74 ("iwlwifi: mvm: include configured sniffer AID in radiotap")

--
Cheers,
Stephen Rothwell


Attachments:
(No filename) (488.00 B)
OpenPGP digital signature

2019-01-30 23:52:08

by Luca Coelho

[permalink] [raw]
Subject: Re: linux-next: build warning after merge of the wireless-drivers-next tree

On Thu, 2019-01-31 at 10:46 +1100, Stephen Rothwell wrote:
> Hi all,
>
> After merging the wireless-drivers-next tree, today's linux-next
> build
> (x86_64 allmodconfig) produced this warning:
>
> drivers/net/wireless/intel/iwlwifi/mvm/rxmq.c:195:13: warning:
> 'iwl_mvm_add_rtap_sniffer_config' defined but not used [-Wunused-
> function]
> static void iwl_mvm_add_rtap_sniffer_config(struct iwl_mvm *mvm,
> ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> Introduced by commit
>
> 9bf13bee2d74 ("iwlwifi: mvm: include configured sniffer AID in
> radiotap")

This was a conflict resolution damage in one of the patches I applied.
We already have a fix for it[1] and Kalle will apply it to wireless-
drivers-next soon.

Sorry for the trouble, but somehow I didn't see this warning and
kbuildbot also reported successful compilation with it. :(

[1] https://patchwork.kernel.org/patch/10788503/

--
Cheers,
Luca.


2019-01-31 13:46:15

by Kalle Valo

[permalink] [raw]
Subject: Re: linux-next: build warning after merge of the wireless-drivers-next tree

Luca Coelho <[email protected]> writes:

> On Thu, 2019-01-31 at 10:46 +1100, Stephen Rothwell wrote:
>> Hi all,
>>
>> After merging the wireless-drivers-next tree, today's linux-next
>> build
>> (x86_64 allmodconfig) produced this warning:
>>
>> drivers/net/wireless/intel/iwlwifi/mvm/rxmq.c:195:13: warning:
>> 'iwl_mvm_add_rtap_sniffer_config' defined but not used [-Wunused-
>> function]
>> static void iwl_mvm_add_rtap_sniffer_config(struct iwl_mvm *mvm,
>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>
>> Introduced by commit
>>
>> 9bf13bee2d74 ("iwlwifi: mvm: include configured sniffer AID in
>> radiotap")
>
> This was a conflict resolution damage in one of the patches I applied.
> We already have a fix for it[1] and Kalle will apply it to wireless-
> drivers-next soon.
>
> Sorry for the trouble, but somehow I didn't see this warning and
> kbuildbot also reported successful compilation with it. :(
>
> [1] https://patchwork.kernel.org/patch/10788503/

I applied the fix now:

https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git/commit/?id=3864be551405b582ae38fbcdfdbb6e49052f5f6d

--
Kalle Valo

2019-01-31 20:52:22

by Stephen Rothwell

[permalink] [raw]
Subject: Re: linux-next: build warning after merge of the wireless-drivers-next tree

Hi all,

On Thu, 31 Jan 2019 15:46:03 +0200 Kalle Valo <[email protected]> wrote:
>
> Luca Coelho <[email protected]> writes:
>
> >
> > This was a conflict resolution damage in one of the patches I applied.
> > We already have a fix for it[1] and Kalle will apply it to wireless-
> > drivers-next soon.
> >
> > Sorry for the trouble, but somehow I didn't see this warning and
> > kbuildbot also reported successful compilation with it. :(
> >
> > [1] https://patchwork.kernel.org/patch/10788503/
>
> I applied the fix now:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git/commit/?id=3864be551405b582ae38fbcdfdbb6e49052f5f6d

Thanks.

--
Cheers,
Stephen Rothwell


Attachments:
(No filename) (488.00 B)
OpenPGP digital signature

2021-06-24 05:50:41

by Kalle Valo

[permalink] [raw]
Subject: Re: linux-next: build warning after merge of the wireless-drivers-next tree

Stephen Rothwell <[email protected]> writes:

> After merging the wireless-drivers-next tree, today's linux-next build
> (x86_64 allmodconfig) produced this warning:
>
> drivers/net/wireless/intel/iwlwifi/fw/acpi.c:720:12: warning: 'iwl_acpi_eval_dsm_func' defined but not used [-Wunused-function]
> 720 | static u32 iwl_acpi_eval_dsm_func(struct device *dev, enum iwl_dsm_funcs_rev_0 eval_func)
> | ^~~~~~~~~~~~~~~~~~~~~~
>
> Introduced by commit
>
> 7119f02b5d34 ("iwlwifi: mvm: support BIOS enable/disable for 11ax in Russia")

Thanks for the report, I missed the warning as I had CONFIG_ACPI disabled in my
build tests. I enabled it now and sent a fix:

https://patchwork.kernel.org/project/linux-wireless/patch/[email protected]/

Luca, please review. I also assigned it to me on patchwork and planning
to apply it directly to wireless-drivers-next.

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

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