linux-next: build warnings after merge of the sound-asoc-fixes tree

linux-next: build warnings after merge of the sound-asoc-fixes tree

[Stephen Rothwell]

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

Hi all,

After merging the sound-asoc-fixes tree, today's linux-next build
(x86_64 allmodconfig) produced these warnings:

WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_acpi_scan from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_probe from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_process_wakeen_event from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_startup from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_exit from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_enable_irq from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_thread from namespace SOUNDWIRE_INTEL_INIT, but does not import it.

Presumably introduced by commit

  82cb71d68c22 ("ASoC: SOF: Intel: add build support for SoundWire")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warnings after merge of the sound-asoc-fixes tree

[Pierre-Louis Bossart]

On 8/20/20 6:39 PM, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the sound-asoc-fixes tree, today's linux-next build
> (x86_64 allmodconfig) produced these warnings:
> 
> WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_acpi_scan from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_probe from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_process_wakeen_event from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_startup from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_exit from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_enable_irq from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_thread from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> 
> Presumably introduced by commit
> 
>    82cb71d68c22 ("ASoC: SOF: Intel: add build support for SoundWire")

There is a companion patch 1eb629363aa35 ("ASoC: SOF: Intel: hda: import 
SOUNDWIRE_INIT namespace") that does the import, not sure what causes 
the warning?

Re: linux-next: build warnings after merge of the sound-asoc-fixes tree

[Stephen Rothwell]

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

Hi Pierre-Louis,

On Thu, 20 Aug 2020 19:01:15 -0500 Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> wrote:
>
> On 8/20/20 6:39 PM, Stephen Rothwell wrote:
> > Hi all,
> > 
> > After merging the sound-asoc-fixes tree, today's linux-next build
> > (x86_64 allmodconfig) produced these warnings:
> > 
> > WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_acpi_scan from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> > WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_probe from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> > WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_process_wakeen_event from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> > WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_startup from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> > WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_exit from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> > WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_enable_irq from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> > WARNING: modpost: module snd-sof-intel-hda-common uses symbol sdw_intel_thread from namespace SOUNDWIRE_INTEL_INIT, but does not import it.
> > 
> > Presumably introduced by commit
> > 
> >    82cb71d68c22 ("ASoC: SOF: Intel: add build support for SoundWire")  
> 
> There is a companion patch 1eb629363aa35 ("ASoC: SOF: Intel: hda: import 
> SOUNDWIRE_INIT namespace") that does the import, not sure what causes 
> the warning?

That latter patch is in the sound-asoc tree (but not in the
sound-asoc-fixes tree).

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warnings after merge of the sound-asoc-fixes tree

[Mark Brown]

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

On Thu, Aug 20, 2020 at 07:01:15PM -0500, Pierre-Louis Bossart wrote:

> There is a companion patch 1eb629363aa35 ("ASoC: SOF: Intel: hda: import
> SOUNDWIRE_INIT namespace") that does the import, not sure what causes the
> warning?

A patch which was not sent as part of the same series and where no
dependency was mentioned :(

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warnings after merge of the sound-asoc-fixes tree

[Pierre-Louis Bossart]

On 8/21/20 6:01 AM, Mark Brown wrote:
> On Thu, Aug 20, 2020 at 07:01:15PM -0500, Pierre-Louis Bossart wrote:
> 
>> There is a companion patch 1eb629363aa35 ("ASoC: SOF: Intel: hda: import
>> SOUNDWIRE_INIT namespace") that does the import, not sure what causes the
>> warning?
> 
> A patch which was not sent as part of the same series and where no
> dependency was mentioned :(

yes, sorry about that, i just noticed the two patches were sent one 
after the other but without the dependency.
Do you have a means to fix this on your side or do you need Bard/I to 
resend it?

Re: linux-next: build warnings after merge of the sound-asoc-fixes tree

[Mark Brown]

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

On Fri, Aug 21, 2020 at 09:25:53AM -0500, Pierre-Louis Bossart wrote:
> On 8/21/20 6:01 AM, Mark Brown wrote:

> > A patch which was not sent as part of the same series and where no
> > dependency was mentioned :(

> yes, sorry about that, i just noticed the two patches were sent one after
> the other but without the dependency.
> Do you have a means to fix this on your side or do you need Bard/I to resend
> it?

I've just moved the patch to for-5.10, it seemed like the functionality
is only half there in Linus' tree anyway and it's a bit late to be
enabling new stuff for 5.9.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warnings after merge of the sound-asoc-fixes tree

[Pierre-Louis Bossart]

On 8/21/20 9:48 AM, Mark Brown wrote:
> On Fri, Aug 21, 2020 at 09:25:53AM -0500, Pierre-Louis Bossart wrote:
>> On 8/21/20 6:01 AM, Mark Brown wrote:
> 
>>> A patch which was not sent as part of the same series and where no
>>> dependency was mentioned :(
> 
>> yes, sorry about that, i just noticed the two patches were sent one after
>> the other but without the dependency.
>> Do you have a means to fix this on your side or do you need Bard/I to resend
>> it?
> 
> I've just moved the patch to for-5.10, it seemed like the functionality
> is only half there in Linus' tree anyway and it's a bit late to be
> enabling new stuff for 5.9.

That's fine, thanks Mark.