* linux-next: build warning after merge of the sound-asoc tree
@ 2023-07-13 2:16 Stephen Rothwell
2023-07-13 12:19 ` Andy Shevchenko
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2023-07-13 2:16 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Andy Shevchenko, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 305 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (htmldocs)
produced this warning:
include/linux/int_log.h:1: warning: no structured comments found
Introduced by commit
f97fa3dcb2db ("lib/math: Move dvb_math.c into lib/math/int_log.c")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2023-07-13 2:16 linux-next: build warning after merge of the sound-asoc tree Stephen Rothwell
@ 2023-07-13 12:19 ` Andy Shevchenko
2023-07-13 12:41 ` Andy Shevchenko
0 siblings, 1 reply; 75+ messages in thread
From: Andy Shevchenko @ 2023-07-13 12:19 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, Linux Kernel Mailing List,
Linux Next Mailing List
On Thu, Jul 13, 2023 at 12:16:27PM +1000, Stephen Rothwell wrote:
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (htmldocs)
> produced this warning:
>
> include/linux/int_log.h:1: warning: no structured comments found
>
> Introduced by commit
>
> f97fa3dcb2db ("lib/math: Move dvb_math.c into lib/math/int_log.c")
Can you elaborate a bit, please?
Seems to me to be a false positive, or unveils a bug somewhere else.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2023-07-13 12:19 ` Andy Shevchenko
@ 2023-07-13 12:41 ` Andy Shevchenko
2023-07-14 9:18 ` Andy Shevchenko
0 siblings, 1 reply; 75+ messages in thread
From: Andy Shevchenko @ 2023-07-13 12:41 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, Linux Kernel Mailing List,
Linux Next Mailing List
On Thu, Jul 13, 2023 at 03:19:04PM +0300, Andy Shevchenko wrote:
> On Thu, Jul 13, 2023 at 12:16:27PM +1000, Stephen Rothwell wrote:
> > Hi all,
> >
> > After merging the sound-asoc tree, today's linux-next build (htmldocs)
> > produced this warning:
> >
> > include/linux/int_log.h:1: warning: no structured comments found
> >
> > Introduced by commit
> >
> > f97fa3dcb2db ("lib/math: Move dvb_math.c into lib/math/int_log.c")
>
> Can you elaborate a bit, please?
>
> Seems to me to be a false positive, or unveils a bug somewhere else.
Ah, now I understand. There is a header file that has descriptions but
EXPORT_SYMBOL. We have to drop keyword export from the index.
Thank you for the report, I'll cook the patch ASAP.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2023-07-13 12:41 ` Andy Shevchenko
@ 2023-07-14 9:18 ` Andy Shevchenko
2023-07-24 3:58 ` Stephen Rothwell
0 siblings, 1 reply; 75+ messages in thread
From: Andy Shevchenko @ 2023-07-14 9:18 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, Linux Kernel Mailing List,
Linux Next Mailing List
On Thu, Jul 13, 2023 at 03:41:09PM +0300, Andy Shevchenko wrote:
> On Thu, Jul 13, 2023 at 03:19:04PM +0300, Andy Shevchenko wrote:
> > On Thu, Jul 13, 2023 at 12:16:27PM +1000, Stephen Rothwell wrote:
> > > Hi all,
> > >
> > > After merging the sound-asoc tree, today's linux-next build (htmldocs)
> > > produced this warning:
> > >
> > > include/linux/int_log.h:1: warning: no structured comments found
> > >
> > > Introduced by commit
> > >
> > > f97fa3dcb2db ("lib/math: Move dvb_math.c into lib/math/int_log.c")
> >
> > Can you elaborate a bit, please?
> >
> > Seems to me to be a false positive, or unveils a bug somewhere else.
>
> Ah, now I understand. There is a header file that has descriptions but
> EXPORT_SYMBOL. We have to drop keyword export from the index.
>
> Thank you for the report, I'll cook the patch ASAP.
The 20230713165320.14199-1-andriy.shevchenko@linux.intel.com had been sent.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2023-07-14 9:18 ` Andy Shevchenko
@ 2023-07-24 3:58 ` Stephen Rothwell
2023-07-24 8:26 ` Andy Shevchenko
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2023-07-24 3:58 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Mark Brown, Liam Girdwood, Linux Kernel Mailing List,
Linux Next Mailing List, Randy Dunlap
[-- Attachment #1: Type: text/plain, Size: 1163 bytes --]
Hi Aall,
On Fri, 14 Jul 2023 12:18:55 +0300 Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
>
> On Thu, Jul 13, 2023 at 03:41:09PM +0300, Andy Shevchenko wrote:
> > On Thu, Jul 13, 2023 at 03:19:04PM +0300, Andy Shevchenko wrote:
> > > On Thu, Jul 13, 2023 at 12:16:27PM +1000, Stephen Rothwell wrote:
> > > >
> > > > After merging the sound-asoc tree, today's linux-next build (htmldocs)
> > > > produced this warning:
> > > >
> > > > include/linux/int_log.h:1: warning: no structured comments found
> > > >
> > > > Introduced by commit
> > > >
> > > > f97fa3dcb2db ("lib/math: Move dvb_math.c into lib/math/int_log.c")
> > >
> > > Can you elaborate a bit, please?
> > >
> > > Seems to me to be a false positive, or unveils a bug somewhere else.
> >
> > Ah, now I understand. There is a header file that has descriptions but
> > EXPORT_SYMBOL. We have to drop keyword export from the index.
> >
> > Thank you for the report, I'll cook the patch ASAP.
>
> The 20230713165320.14199-1-andriy.shevchenko@linux.intel.com had been sent.
But not yet applied by anyone :-(
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2023-07-24 3:58 ` Stephen Rothwell
@ 2023-07-24 8:26 ` Andy Shevchenko
2023-07-24 12:36 ` Mark Brown
0 siblings, 1 reply; 75+ messages in thread
From: Andy Shevchenko @ 2023-07-24 8:26 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, Linux Kernel Mailing List,
Linux Next Mailing List, Randy Dunlap
On Mon, Jul 24, 2023 at 01:58:58PM +1000, Stephen Rothwell wrote:
> On Fri, 14 Jul 2023 12:18:55 +0300 Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
> > On Thu, Jul 13, 2023 at 03:41:09PM +0300, Andy Shevchenko wrote:
> > > On Thu, Jul 13, 2023 at 03:19:04PM +0300, Andy Shevchenko wrote:
> > > > On Thu, Jul 13, 2023 at 12:16:27PM +1000, Stephen Rothwell wrote:
> > > > >
> > > > > After merging the sound-asoc tree, today's linux-next build (htmldocs)
> > > > > produced this warning:
> > > > >
> > > > > include/linux/int_log.h:1: warning: no structured comments found
> > > > >
> > > > > Introduced by commit
> > > > >
> > > > > f97fa3dcb2db ("lib/math: Move dvb_math.c into lib/math/int_log.c")
> > > >
> > > > Can you elaborate a bit, please?
> > > >
> > > > Seems to me to be a false positive, or unveils a bug somewhere else.
> > >
> > > Ah, now I understand. There is a header file that has descriptions but
> > > EXPORT_SYMBOL. We have to drop keyword export from the index.
> > >
> > > Thank you for the report, I'll cook the patch ASAP.
> >
> > The 20230713165320.14199-1-andriy.shevchenko@linux.intel.com had been sent.
>
> But not yet applied by anyone :-(
Since the culprit went via ASoC tree, I think that tree is the best for the fix.
Mark, can you apply that, please?
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2023-07-24 8:26 ` Andy Shevchenko
@ 2023-07-24 12:36 ` Mark Brown
2023-07-24 13:05 ` Andy Shevchenko
0 siblings, 1 reply; 75+ messages in thread
From: Mark Brown @ 2023-07-24 12:36 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Stephen Rothwell, Liam Girdwood, Linux Kernel Mailing List,
Linux Next Mailing List, Randy Dunlap
[-- Attachment #1: Type: text/plain, Size: 716 bytes --]
On Mon, Jul 24, 2023 at 11:26:58AM +0300, Andy Shevchenko wrote:
> On Mon, Jul 24, 2023 at 01:58:58PM +1000, Stephen Rothwell wrote:
> > On Fri, 14 Jul 2023 12:18:55 +0300 Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
> > > The 20230713165320.14199-1-andriy.shevchenko@linux.intel.com had been sent.
For the benefit of those playing at home that's "Documentation:
core-api: Drop :export: for int_log.h".
> > But not yet applied by anyone :-(
> Since the culprit went via ASoC tree, I think that tree is the best for the fix.
> Mark, can you apply that, please?
Jon's usually fairly responsive, perhaps there's something worrying
there, though I do note you only sent it a bit more than a week ago.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2023-07-24 12:36 ` Mark Brown
@ 2023-07-24 13:05 ` Andy Shevchenko
2023-07-24 13:11 ` Mark Brown
0 siblings, 1 reply; 75+ messages in thread
From: Andy Shevchenko @ 2023-07-24 13:05 UTC (permalink / raw)
To: Mark Brown
Cc: Stephen Rothwell, Liam Girdwood, Linux Kernel Mailing List,
Linux Next Mailing List, Randy Dunlap
On Mon, Jul 24, 2023 at 01:36:53PM +0100, Mark Brown wrote:
> On Mon, Jul 24, 2023 at 11:26:58AM +0300, Andy Shevchenko wrote:
> > On Mon, Jul 24, 2023 at 01:58:58PM +1000, Stephen Rothwell wrote:
> > > On Fri, 14 Jul 2023 12:18:55 +0300 Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
>
> > > > The 20230713165320.14199-1-andriy.shevchenko@linux.intel.com had been sent.
>
> For the benefit of those playing at home that's "Documentation:
> core-api: Drop :export: for int_log.h".
>
> > > But not yet applied by anyone :-(
>
> > Since the culprit went via ASoC tree, I think that tree is the best for the fix.
> > Mark, can you apply that, please?
>
> Jon's usually fairly responsive, perhaps there's something worrying
> there,
Hmm... maybe my understanding of the :export: is wrong? Or what do you suggest?
> though I do note you only sent it a bit more than a week ago.
I fully aware of that and you can see that this reply has been induced by
the ping from Stephen.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2023-07-24 13:05 ` Andy Shevchenko
@ 2023-07-24 13:11 ` Mark Brown
2023-07-24 13:44 ` Andy Shevchenko
0 siblings, 1 reply; 75+ messages in thread
From: Mark Brown @ 2023-07-24 13:11 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Stephen Rothwell, Liam Girdwood, Linux Kernel Mailing List,
Linux Next Mailing List, Randy Dunlap
[-- Attachment #1: Type: text/plain, Size: 639 bytes --]
On Mon, Jul 24, 2023 at 04:05:34PM +0300, Andy Shevchenko wrote:
> On Mon, Jul 24, 2023 at 01:36:53PM +0100, Mark Brown wrote:
> > Jon's usually fairly responsive, perhaps there's something worrying
> > there,
> Hmm... maybe my understanding of the :export: is wrong? Or what do you suggest?
I have no idea what any of that is doing. I'm hoping Jon does.
> > though I do note you only sent it a bit more than a week ago.
> I fully aware of that and you can see that this reply has been induced by
> the ping from Stephen.
Sure, but that means that it's possible he's just not got round to it
yet rather than that there's a problem.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2023-07-24 13:11 ` Mark Brown
@ 2023-07-24 13:44 ` Andy Shevchenko
2023-07-24 15:18 ` Jonathan Corbet
0 siblings, 1 reply; 75+ messages in thread
From: Andy Shevchenko @ 2023-07-24 13:44 UTC (permalink / raw)
To: Mark Brown, Jonathan Corbet
Cc: Stephen Rothwell, Liam Girdwood, Linux Kernel Mailing List,
Linux Next Mailing List, Randy Dunlap
+Cc: Jon
On Mon, Jul 24, 2023 at 02:11:00PM +0100, Mark Brown wrote:
> On Mon, Jul 24, 2023 at 04:05:34PM +0300, Andy Shevchenko wrote:
> > On Mon, Jul 24, 2023 at 01:36:53PM +0100, Mark Brown wrote:
>
> > > Jon's usually fairly responsive, perhaps there's something worrying
> > > there,
>
> > Hmm... maybe my understanding of the :export: is wrong? Or what do you suggest?
>
> I have no idea what any of that is doing. I'm hoping Jon does.
Okay, so far we are waiting for his reply...
> > > though I do note you only sent it a bit more than a week ago.
>
> > I fully aware of that and you can see that this reply has been induced by
> > the ping from Stephen.
>
> Sure, but that means that it's possible he's just not got round to it
> yet rather than that there's a problem.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2023-07-24 13:44 ` Andy Shevchenko
@ 2023-07-24 15:18 ` Jonathan Corbet
0 siblings, 0 replies; 75+ messages in thread
From: Jonathan Corbet @ 2023-07-24 15:18 UTC (permalink / raw)
To: Andy Shevchenko, Mark Brown
Cc: Stephen Rothwell, Liam Girdwood, Linux Kernel Mailing List,
Linux Next Mailing List, Randy Dunlap
Andy Shevchenko <andriy.shevchenko@linux.intel.com> writes:
> +Cc: Jon
>
> On Mon, Jul 24, 2023 at 02:11:00PM +0100, Mark Brown wrote:
>> On Mon, Jul 24, 2023 at 04:05:34PM +0300, Andy Shevchenko wrote:
>> > On Mon, Jul 24, 2023 at 01:36:53PM +0100, Mark Brown wrote:
>>
>> > > Jon's usually fairly responsive, perhaps there's something worrying
>> > > there,
>>
>> > Hmm... maybe my understanding of the :export: is wrong? Or what do you suggest?
>>
>> I have no idea what any of that is doing. I'm hoping Jon does.
>
> Okay, so far we are waiting for his reply...
Sorry, it took me a bit to reconstruct why I hadn't applied this .. the
simple fact is that it doesn't apply to docs-next.
include/linux/int_log.h doesn't exist in mainline, so the kernel-doc
directive to include from it doesn't either. All of that is introduced
in linux-next ... so the fix really needs to take the same path that the
rest of the changes did.
I guess I'd just assumed that was going to happen and didn't reply,
apologies for that.
Feel free to add:
Acked-by: Jonathan Corbet <corbet@lwn.net>
if you feel so inclined.
Thanks,
jon
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2023-12-12 0:16 Stephen Rothwell
2023-12-12 9:35 ` Charles Keepax
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2023-12-12 0:16 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Charles Keepax, Linux Kernel Mailing List,
Linux Next Mailing List, Arnd Bergmann
[-- Attachment #1: Type: text/plain, Size: 751 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/codecs/cs42l43.c:142:6: warning: no previous prototype for 'cs42l43_hp_ilimit_clear_work' [-Wmissing-prototypes]
142 | void cs42l43_hp_ilimit_clear_work(struct work_struct *work)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
sound/soc/codecs/cs42l43.c:159:6: warning: no previous prototype for 'cs42l43_hp_ilimit_work' [-Wmissing-prototypes]
159 | void cs42l43_hp_ilimit_work(struct work_struct *work)
| ^~~~~~~~~~~~~~~~~~~~~~
Introduced by commit
bbbc18d8c27c ("ASoC: cs42l43: Allow HP amp to cool off after current limit")
Maybe they should just be static?
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2023-12-12 0:16 Stephen Rothwell
@ 2023-12-12 9:35 ` Charles Keepax
0 siblings, 0 replies; 75+ messages in thread
From: Charles Keepax @ 2023-12-12 9:35 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, Linux Kernel Mailing List,
Linux Next Mailing List, Arnd Bergmann
On Tue, Dec 12, 2023 at 11:16:25AM +1100, Stephen Rothwell wrote:
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> sound/soc/codecs/cs42l43.c:142:6: warning: no previous prototype for 'cs42l43_hp_ilimit_clear_work' [-Wmissing-prototypes]
> 142 | void cs42l43_hp_ilimit_clear_work(struct work_struct *work)
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
> sound/soc/codecs/cs42l43.c:159:6: warning: no previous prototype for 'cs42l43_hp_ilimit_work' [-Wmissing-prototypes]
> 159 | void cs42l43_hp_ilimit_work(struct work_struct *work)
> | ^~~~~~~~~~~~~~~~~~~~~~
>
> Introduced by commit
>
> bbbc18d8c27c ("ASoC: cs42l43: Allow HP amp to cool off after current limit")
>
> Maybe they should just be static?
>
Yeah they should be static sorry, will send a patch today.
Thanks,
Charles
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2021-10-26 7:54 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2021-10-26 7:54 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Ajit Kumar Pandey, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 630 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (powerpc
allyesconfig) produced this warning:
WARNING: unmet direct dependencies detected for SND_SOC_AMD_MACH_COMMON
Depends on [n]: SOUND [=y] && !UML && SND [=y] && SND_SOC [=y] && X86 && PCI [=y] && I2C [=y]
Selected by [y]:
- SND_SOC_AMD_LEGACY_MACH [=y] && SOUND [=y] && !UML && SND [=y] && SND_SOC [=y]
- SND_SOC_AMD_SOF_MACH [=y] && SOUND [=y] && !UML && SND [=y] && SND_SOC [=y]
Presuambly introduced by commit
9d8a7be88b33 ("ASoC: amd: acp: Add legacy sound card support for Chrome audio")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2021-08-13 2:17 Stephen Rothwell
2021-08-13 14:35 ` Doug Anderson
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2021-08-13 2:17 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Douglas Anderson, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 524 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/codecs/rt5682-i2c.c: In function 'rt5682_i2c_remove':
sound/soc/codecs/rt5682-i2c.c:297:22: warning: unused variable 'rt5682' [-Wunused-variable]
297 | struct rt5682_priv *rt5682 = i2c_get_clientdata(client);
| ^~~~~~
Introduced by commit
772d44526e20 ("ASoC: rt5682: Properly turn off regulators if wrong device ID")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2021-04-19 3:41 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2021-04-19 3:41 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Thierry Reding, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 938 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:
sound/soc/generic/simple-card.c: In function 'simple_parse_of':
sound/soc/generic/simple-card.c:478:1: warning: the frame size of 1552 bytes is larger than 1024 bytes [-Wframe-larger-than=]
478 | }
| ^
sound/soc/generic/simple-card.c: In function 'asoc_simple_probe':
sound/soc/generic/simple-card.c:706:1: warning: the frame size of 1552 bytes is larger than 1024 bytes [-Wframe-larger-than=]
706 | }
| ^
sound/soc/generic/audio-graph-card.c: In function 'audio_graph_parse_of':
sound/soc/generic/audio-graph-card.c:612:1: warning: the frame size of 1552 bytes is larger than 1024 bytes [-Wframe-larger-than=]
612 | }
| ^
Presumably introduced by commit
343e55e71877 ("ASoC: simple-card-utils: Increase maximum number of links to 128")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2021-03-23 5:43 Stephen Rothwell
2021-03-23 6:13 ` Mukunda,Vijendar
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2021-03-23 5:43 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Vijendar Mukunda, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 676 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (powerpc
allyesconfig) produced this warning:
sound/soc/amd/acp-da7219-max98357a.c:684:28: warning: 'cz_rt5682_card' defined but not used [-Wunused-variable]
684 | static struct snd_soc_card cz_rt5682_card = {
| ^~~~~~~~~~~~~~
sound/soc/amd/acp-da7219-max98357a.c:671:28: warning: 'cz_card' defined but not used [-Wunused-variable]
671 | static struct snd_soc_card cz_card = {
| ^~~~~~~
Introduced by commit
7e71b48f9e27 ("ASoC: amd: Add support for RT5682 codec in machine driver")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2021-03-23 5:43 Stephen Rothwell
@ 2021-03-23 6:13 ` Mukunda,Vijendar
0 siblings, 0 replies; 75+ messages in thread
From: Mukunda,Vijendar @ 2021-03-23 6:13 UTC (permalink / raw)
To: Stephen Rothwell, Mark Brown, Liam Girdwood
Cc: Linux Kernel Mailing List, Linux Next Mailing List
On 3/23/21 11:13 AM, Stephen Rothwell wrote:
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (powerpc
> allyesconfig) produced this warning:
>
> sound/soc/amd/acp-da7219-max98357a.c:684:28: warning: 'cz_rt5682_card' defined but not used [-Wunused-variable]
> 684 | static struct snd_soc_card cz_rt5682_card = {
> | ^~~~~~~~~~~~~~
> sound/soc/amd/acp-da7219-max98357a.c:671:28: warning: 'cz_card' defined but not used [-Wunused-variable]
> 671 | static struct snd_soc_card cz_card = {
> | ^~~~~~~
>
> Introduced by commit
>
> 7e71b48f9e27 ("ASoC: amd: Add support for RT5682 codec in machine driver")
>
Will add ACPI dependency in Kconfig and submit the supplement patch.
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2020-11-11 2:34 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2020-11-11 2:34 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Shane.Chien, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 626 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/codecs/mt6359.c: In function 'mt6359_platform_driver_remove':
sound/soc/codecs/mt6359.c:2823:6: warning: unused variable 'ret' [-Wunused-variable]
2823 | int ret;
| ^~~
sound/soc/codecs/mt6359.c:2822:22: warning: unused variable 'priv' [-Wunused-variable]
2822 | struct mt6359_priv *priv = dev_get_drvdata(&pdev->dev);
| ^~~~
Introduced by commit
64a70744b778 ("ASoC: Fix vaud18 power leakage of mt6359")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2020-10-02 4:14 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2020-10-02 4:14 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Kernel Mailing List, Linux Next Mailing List, Tang Bin,
Zhang Shengju
[-- Attachment #1: Type: text/plain, Size: 492 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/fsl/fsl_spdif.c: In function 'fsl_spdif_probe':
sound/soc/fsl/fsl_spdif.c:1255:22: warning: unused variable 'np' [-Wunused-variable]
1255 | struct device_node *np = pdev->dev.of_node;
| ^~
Introduced by commit
601fd3a7d849 ("ASoC: fsl_spdif: Fix unnecessary check in fsl_spdif_probe()")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2020-09-18 3:21 Stephen Rothwell
2020-09-18 9:21 ` Kai Vehmanen
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2020-09-18 3:21 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Iulian Olaru, Linux Next Mailing List, Linux Kernel Mailing List,
Kai Vehmanen
[-- Attachment #1: Type: text/plain, Size: 330 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
WARNING: modpost: missing MODULE_LICENSE() in sound/soc/sof/imx/imx-common.o
Introduced by commit
18ebffe4d043 ("ASoC: SOF: imx: Add debug support for imx platforms")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2020-09-18 3:21 Stephen Rothwell
@ 2020-09-18 9:21 ` Kai Vehmanen
0 siblings, 0 replies; 75+ messages in thread
From: Kai Vehmanen @ 2020-09-18 9:21 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, Iulian Olaru, Linux Next Mailing List,
Linux Kernel Mailing List, Kai Vehmanen, daniel.baluta,
pierre-louis.bossart
Hello,
+Daniel,+Pierre
On Fri, 18 Sep 2020, Stephen Rothwell wrote:
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> WARNING: modpost: missing MODULE_LICENSE() in sound/soc/sof/imx/imx-common.o
thanks for the report. I made a patch and once I get acks from
stakeholders, will submit to asoc tree:
https://github.com/thesofproject/linux/pull/2450
We'll also follow up how this slipped through our CI.
Br, Kai
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2020-08-19 1:34 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2020-08-19 1:34 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Samuel Holland
[-- Attachment #1: Type: text/plain, Size: 511 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/sunxi/sun8i-codec.c:532:12: warning: 'sun8i_codec_component_probe' defined but not used [-Wunused-function]
532 | static int sun8i_codec_component_probe(struct snd_soc_component *component)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
Introduced by commit
90cac932976e ("ASoC: sun8i-codec: Fix DAPM to match the hardware topology")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2020-05-28 3:57 Stephen Rothwell
2020-05-29 13:30 ` Mark Brown
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2020-05-28 3:57 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Ranjani Sridharan
[-- Attachment #1: Type: text/plain, Size: 878 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/sof/intel/byt.c:464:12: warning: 'byt_remove' defined but not used [-Wunused-function]
464 | static int byt_remove(struct snd_sof_dev *sdev)
| ^~~~~~~~~~
sound/soc/sof/intel/byt.c:454:12: warning: 'byt_resume' defined but not used [-Wunused-function]
454 | static int byt_resume(struct snd_sof_dev *sdev)
| ^~~~~~~~~~
sound/soc/sof/intel/byt.c:447:12: warning: 'byt_suspend' defined but not used [-Wunused-function]
447 | static int byt_suspend(struct snd_sof_dev *sdev, u32 target_state)
| ^~~~~~~~~~~
Introduced by commits
ddcccd543f5d ("ASoC: SOF: Intel: byt: Add PM callbacks")
c691f0c6e267 ("ASoC: SOF: Intel: BYT: add .remove op")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2020-05-28 3:57 Stephen Rothwell
@ 2020-05-29 13:30 ` Mark Brown
2020-05-29 13:55 ` Pierre-Louis Bossart
0 siblings, 1 reply; 75+ messages in thread
From: Mark Brown @ 2020-05-29 13:30 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Liam Girdwood, Linux Next Mailing List,
Linux Kernel Mailing List, Ranjani Sridharan,
Pierre-Louis Bossart
[-- Attachment #1: Type: text/plain, Size: 969 bytes --]
On Thu, May 28, 2020 at 01:57:17PM +1000, Stephen Rothwell wrote:
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> sound/soc/sof/intel/byt.c:464:12: warning: 'byt_remove' defined but not used [-Wunused-function]
> 464 | static int byt_remove(struct snd_sof_dev *sdev)
> | ^~~~~~~~~~
> sound/soc/sof/intel/byt.c:454:12: warning: 'byt_resume' defined but not used [-Wunused-function]
> 454 | static int byt_resume(struct snd_sof_dev *sdev)
> | ^~~~~~~~~~
> sound/soc/sof/intel/byt.c:447:12: warning: 'byt_suspend' defined but not used [-Wunused-function]
> 447 | static int byt_suspend(struct snd_sof_dev *sdev, u32 target_state)
> | ^~~~~~~~~~~
>
> Introduced by commits
>
> ddcccd543f5d ("ASoC: SOF: Intel: byt: Add PM callbacks")
> c691f0c6e267 ("ASoC: SOF: Intel: BYT: add .remove op")
Ranjani, Pierre?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2020-05-29 13:30 ` Mark Brown
@ 2020-05-29 13:55 ` Pierre-Louis Bossart
2020-05-29 14:27 ` Pierre-Louis Bossart
0 siblings, 1 reply; 75+ messages in thread
From: Pierre-Louis Bossart @ 2020-05-29 13:55 UTC (permalink / raw)
To: Mark Brown, Stephen Rothwell
Cc: Liam Girdwood, Linux Next Mailing List,
Linux Kernel Mailing List, Ranjani Sridharan
On 5/29/20 8:30 AM, Mark Brown wrote:
> On Thu, May 28, 2020 at 01:57:17PM +1000, Stephen Rothwell wrote:
>> Hi all,
>>
>> After merging the sound-asoc tree, today's linux-next build (x86_64
>> allmodconfig) produced this warning:
>>
>> sound/soc/sof/intel/byt.c:464:12: warning: 'byt_remove' defined but not used [-Wunused-function]
>> 464 | static int byt_remove(struct snd_sof_dev *sdev)
>> | ^~~~~~~~~~
>> sound/soc/sof/intel/byt.c:454:12: warning: 'byt_resume' defined but not used [-Wunused-function]
>> 454 | static int byt_resume(struct snd_sof_dev *sdev)
>> | ^~~~~~~~~~
>> sound/soc/sof/intel/byt.c:447:12: warning: 'byt_suspend' defined but not used [-Wunused-function]
>> 447 | static int byt_suspend(struct snd_sof_dev *sdev, u32 target_state)
>> | ^~~~~~~~~~~
>>
>> Introduced by commits
>>
>> ddcccd543f5d ("ASoC: SOF: Intel: byt: Add PM callbacks")
>> c691f0c6e267 ("ASoC: SOF: Intel: BYT: add .remove op")
>
> Ranjani, Pierre?
Humm, I am not sure what happened here or why kbuild didn't report this
earlier. This was added in
ddcccd543f5dbd ('ASoC: SOF: Intel: byt: Add PM callbacks')
And I do see them used in the code:
sound/soc/sof/intel/byt.c
/* PM */
.suspend = byt_suspend,
.resume = byt_resume,
Will run a check and fix ASAP, my guess it's a Kconfig issue or the
functions not protected by the usual SND_SOC_SOF_BAYTRAIL.
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2020-05-29 13:55 ` Pierre-Louis Bossart
@ 2020-05-29 14:27 ` Pierre-Louis Bossart
0 siblings, 0 replies; 75+ messages in thread
From: Pierre-Louis Bossart @ 2020-05-29 14:27 UTC (permalink / raw)
To: Mark Brown, Stephen Rothwell
Cc: Liam Girdwood, Linux Next Mailing List,
Linux Kernel Mailing List, Ranjani Sridharan
On 5/29/20 8:55 AM, Pierre-Louis Bossart wrote:
>
>
> On 5/29/20 8:30 AM, Mark Brown wrote:
>> On Thu, May 28, 2020 at 01:57:17PM +1000, Stephen Rothwell wrote:
>>> Hi all,
>>>
>>> After merging the sound-asoc tree, today's linux-next build (x86_64
>>> allmodconfig) produced this warning:
>>>
>>> sound/soc/sof/intel/byt.c:464:12: warning: 'byt_remove' defined but
>>> not used [-Wunused-function]
>>> 464 | static int byt_remove(struct snd_sof_dev *sdev)
>>> | ^~~~~~~~~~
>>> sound/soc/sof/intel/byt.c:454:12: warning: 'byt_resume' defined but
>>> not used [-Wunused-function]
>>> 454 | static int byt_resume(struct snd_sof_dev *sdev)
>>> | ^~~~~~~~~~
>>> sound/soc/sof/intel/byt.c:447:12: warning: 'byt_suspend' defined but
>>> not used [-Wunused-function]
>>> 447 | static int byt_suspend(struct snd_sof_dev *sdev, u32
>>> target_state)
>>> | ^~~~~~~~~~~
>>>
>>> Introduced by commits
>>>
>>> ddcccd543f5d ("ASoC: SOF: Intel: byt: Add PM callbacks")
>>> c691f0c6e267 ("ASoC: SOF: Intel: BYT: add .remove op")
>>
>> Ranjani, Pierre?
>
> Humm, I am not sure what happened here or why kbuild didn't report this
> earlier. This was added in
>
> ddcccd543f5dbd ('ASoC: SOF: Intel: byt: Add PM callbacks')
>
> And I do see them used in the code:
>
> sound/soc/sof/intel/byt.c
>
> /* PM */
> .suspend = byt_suspend,
> .resume = byt_resume,
>
> Will run a check and fix ASAP, my guess it's a Kconfig issue or the
> functions not protected by the usual SND_SOC_SOF_BAYTRAIL.
it's both. allmodconfig disables SND_SOC_SOF_BAYTRAIL due to mutual
exclusion with the legacy driver, but enables SND_SOC_SOF_MERRIFIELD
(this should be fixed in a separate patch) and in this case we haven't
tested suspend/resume on merrifield so didn't use this code. Will send a
patch shortly.
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2020-04-24 6:26 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2020-04-24 6:26 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List
[-- Attachment #1: Type: text/plain, Size: 556 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (powerpc
allyesconfig) produced this warning:
WARNING: unmet direct dependencies detected for SND_SOC_SOF_IMX8M
Depends on [n]: SOUND [=y] && !UML && SND [=y] && SND_SOC [=y] && SND_SOC_SOF_TOPLEVEL [=y] && SND_SOC_SOF_IMX_TOPLEVEL [=y] && IMX_DSP
Selected by [y]:
- SND_SOC_SOF_IMX_OF [=y] && SOUND [=y] && !UML && SND [=y] && SND_SOC [=y] && SND_SOC_SOF_TOPLEVEL [=y] && SND_SOC_SOF_IMX_TOPLEVEL [=y] && SND_SOC_SOF_IMX8M_SUPPORT [=y]
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2020-03-03 1:29 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2020-03-03 1:29 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Akshu Agrawal
[-- Attachment #1: Type: text/plain, Size: 1421 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
In file included from include/linux/device.h:15,
from include/sound/core.h:10,
from sound/soc/amd/acp3x-rt5682-max9836.c:7:
sound/soc/amd/acp3x-rt5682-max9836.c: In function 'acp3x_probe':
sound/soc/amd/acp3x-rt5682-max9836.c:341:23: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long int' [-Wformat=]
341 | dev_err(&pdev->dev, "DMIC gpio failed err=%d\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
19 | #define dev_fmt(fmt) fmt
| ^~~
sound/soc/amd/acp3x-rt5682-max9836.c:341:3: note: in expansion of macro 'dev_err'
341 | dev_err(&pdev->dev, "DMIC gpio failed err=%d\n",
| ^~~~~~~
sound/soc/amd/acp3x-rt5682-max9836.c:341:46: note: format string is defined here
341 | dev_err(&pdev->dev, "DMIC gpio failed err=%d\n",
| ~^
| |
| int
| %ld
Introduced by commit
72c3b2b09fcd ("ASoc: amd: Add DMIC switch capability to machine driver")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2020-03-01 23:57 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2020-03-01 23:57 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Jack Yu
[-- Attachment #1: Type: text/plain, Size: 473 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/codecs/rt1015.c:844:31: warning: 'rt1015_aif_dai_ops' defined but not used [-Wunused-variable]
844 | static struct snd_soc_dai_ops rt1015_aif_dai_ops = {
| ^~~~~~~~~~~~~~~~~~
Exposed by commit
4a88b7dec331 ("ASoC: rt1015: modify some structure to be static.")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2019-10-22 23:45 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2019-10-22 23:45 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Ben Zhang
[-- Attachment #1: Type: text/plain, Size: 530 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/codecs/rt5677-spi.c: In function 'rt5677_spi_pcm_close':
sound/soc/codecs/rt5677-spi.c:114:30: warning: unused variable 'rtd' [-Wunused-variable]
114 | struct snd_soc_pcm_runtime *rtd = substream->private_data;
| ^~~
Introduced by commit
a0e0d135427c ("ASoC: rt5677: Add a PCM device for streaming hotword via SPI")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2019-10-07 23:58 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2019-10-07 23:58 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Stephen Boyd
[-- Attachment #1: Type: text/plain, Size: 492 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/jz4740/jz4740-i2s.c: In function 'jz4740_i2s_dev_probe':
sound/soc/jz4740/jz4740-i2s.c:500:29: warning: unused variable 'match' [-Wunused-variable]
500 | const struct of_device_id *match;
| ^~~~~
Introduced by commit
67ad656bdd70 ("ASoC: jz4740: Use of_device_get_match_data()")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2019-04-26 3:05 Stephen Rothwell
2019-04-26 3:14 ` Nicolin Chen
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2019-04-26 3:05 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List,
Daniel Baluta, Shengjiu Wang, Nicolin Chen
[-- Attachment #1: Type: text/plain, Size: 417 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:
sound/soc/fsl/fsl_sai.c: In function 'fsl_sai_remove':
sound/soc/fsl/fsl_sai.c:921:1: warning: no return statement in function returning non-void [-Wreturn-type]
}
^
Introduced by commit
812ad463e089 ("ASoC: fsl_sai: Add support for runtime pm")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2019-04-26 3:05 Stephen Rothwell
@ 2019-04-26 3:14 ` Nicolin Chen
0 siblings, 0 replies; 75+ messages in thread
From: Nicolin Chen @ 2019-04-26 3:14 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, Linux Next Mailing List,
Linux Kernel Mailing List, Daniel Baluta, Shengjiu Wang
On Fri, Apr 26, 2019 at 01:05:49PM +1000, Stephen Rothwell wrote:
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (arm
> multi_v7_defconfig) produced this warning:
>
> sound/soc/fsl/fsl_sai.c: In function 'fsl_sai_remove':
> sound/soc/fsl/fsl_sai.c:921:1: warning: no return statement in function returning non-void [-Wreturn-type]
> }
> ^
>
> Introduced by commit
>
> 812ad463e089 ("ASoC: fsl_sai: Add support for runtime pm")
Thanks. I am submitting a fix.
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2019-04-09 1:46 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2019-04-09 1:46 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Paul Cercueil
[-- Attachment #1: Type: text/plain, Size: 526 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/intel/boards/bytcht_es8316.c: In function 'snd_byt_cht_es8316_mc_probe':
sound/soc/intel/boards/bytcht_es8316.c:508:11: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
quirk = (int)dmi_id->driver_data;
^
Introduced by commit
a8d218f4fe81 ("ASoC: Intel: bytcht_es8316: Add quirk for the Teclast X98+ II")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2019-03-25 23:01 Stephen Rothwell
2019-04-08 23:12 ` Stephen Rothwell
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2019-03-25 23:01 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Annaliese McDermond
[-- Attachment #1: Type: text/plain, Size: 1902 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/codecs/tlv320aic32x4-clk.c: In function 'clk_aic32x4_pll_recalc_rate':
sound/soc/codecs/tlv320aic32x4-clk.c:149:38: warning: 'settings.d' may be used uninitialized in this function [-Wmaybe-uninitialized]
((settings->j * 10000) + settings->d);
~~~~~~~~^~~
sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.d' was declared here
struct clk_aic32x4_pll_muldiv settings;
^~~~~~~~
sound/soc/codecs/tlv320aic32x4-clk.c:149:15: warning: 'settings.j' may be used uninitialized in this function [-Wmaybe-uninitialized]
((settings->j * 10000) + settings->d);
~~~~~~~~^~~
sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.j' was declared here
struct clk_aic32x4_pll_muldiv settings;
^~~~~~~~
sound/soc/codecs/tlv320aic32x4-clk.c:148:37: warning: 'settings.r' may be used uninitialized in this function [-Wmaybe-uninitialized]
rate = (u64) parent_rate * settings->r *
~~~~~~~~^~~
sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.r' was declared here
struct clk_aic32x4_pll_muldiv settings;
^~~~~~~~
sound/soc/codecs/tlv320aic32x4-clk.c:151:56: warning: 'settings.p' may be used uninitialized in this function [-Wmaybe-uninitialized]
return (unsigned long) DIV_ROUND_UP_ULL(rate, settings->p * 10000);
^~
sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.p' was declared here
struct clk_aic32x4_pll_muldiv settings;
^~~~~~~~
Introduced by commit
514b044cba66 ("ASoC: tlv320aic32x4: Model PLL in CCF")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2019-03-25 23:01 Stephen Rothwell
@ 2019-04-08 23:12 ` Stephen Rothwell
2019-04-09 4:04 ` Annaliese McDermond
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2019-04-08 23:12 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Annaliese McDermond
[-- Attachment #1: Type: text/plain, Size: 2090 bytes --]
Hi all,
On Tue, 26 Mar 2019 10:01:45 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> sound/soc/codecs/tlv320aic32x4-clk.c: In function 'clk_aic32x4_pll_recalc_rate':
> sound/soc/codecs/tlv320aic32x4-clk.c:149:38: warning: 'settings.d' may be used uninitialized in this function [-Wmaybe-uninitialized]
> ((settings->j * 10000) + settings->d);
> ~~~~~~~~^~~
> sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.d' was declared here
> struct clk_aic32x4_pll_muldiv settings;
> ^~~~~~~~
> sound/soc/codecs/tlv320aic32x4-clk.c:149:15: warning: 'settings.j' may be used uninitialized in this function [-Wmaybe-uninitialized]
> ((settings->j * 10000) + settings->d);
> ~~~~~~~~^~~
> sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.j' was declared here
> struct clk_aic32x4_pll_muldiv settings;
> ^~~~~~~~
> sound/soc/codecs/tlv320aic32x4-clk.c:148:37: warning: 'settings.r' may be used uninitialized in this function [-Wmaybe-uninitialized]
> rate = (u64) parent_rate * settings->r *
> ~~~~~~~~^~~
> sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.r' was declared here
> struct clk_aic32x4_pll_muldiv settings;
> ^~~~~~~~
> sound/soc/codecs/tlv320aic32x4-clk.c:151:56: warning: 'settings.p' may be used uninitialized in this function [-Wmaybe-uninitialized]
> return (unsigned long) DIV_ROUND_UP_ULL(rate, settings->p * 10000);
> ^~
> sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.p' was declared here
> struct clk_aic32x4_pll_muldiv settings;
> ^~~~~~~~
>
> Introduced by commit
>
> 514b044cba66 ("ASoC: tlv320aic32x4: Model PLL in CCF")
I am still getting these warnings.
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2019-04-08 23:12 ` Stephen Rothwell
@ 2019-04-09 4:04 ` Annaliese McDermond
0 siblings, 0 replies; 75+ messages in thread
From: Annaliese McDermond @ 2019-04-09 4:04 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, Linux Next Mailing List,
Linux Kernel Mailing List
Stephen --
I’m testing a potential solution now and I’ll be submitting something in the next couple of days. Thanks for the heads up.
--
Annaliese McDermond
nh6z@nh6z.net
> On Apr 8, 2019, at 4:12 PM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> Hi all,
>
> On Tue, 26 Mar 2019 10:01:45 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>>
>> After merging the sound-asoc tree, today's linux-next build (x86_64
>> allmodconfig) produced this warning:
>>
>> sound/soc/codecs/tlv320aic32x4-clk.c: In function 'clk_aic32x4_pll_recalc_rate':
>> sound/soc/codecs/tlv320aic32x4-clk.c:149:38: warning: 'settings.d' may be used uninitialized in this function [-Wmaybe-uninitialized]
>> ((settings->j * 10000) + settings->d);
>> ~~~~~~~~^~~
>> sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.d' was declared here
>> struct clk_aic32x4_pll_muldiv settings;
>> ^~~~~~~~
>> sound/soc/codecs/tlv320aic32x4-clk.c:149:15: warning: 'settings.j' may be used uninitialized in this function [-Wmaybe-uninitialized]
>> ((settings->j * 10000) + settings->d);
>> ~~~~~~~~^~~
>> sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.j' was declared here
>> struct clk_aic32x4_pll_muldiv settings;
>> ^~~~~~~~
>> sound/soc/codecs/tlv320aic32x4-clk.c:148:37: warning: 'settings.r' may be used uninitialized in this function [-Wmaybe-uninitialized]
>> rate = (u64) parent_rate * settings->r *
>> ~~~~~~~~^~~
>> sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.r' was declared here
>> struct clk_aic32x4_pll_muldiv settings;
>> ^~~~~~~~
>> sound/soc/codecs/tlv320aic32x4-clk.c:151:56: warning: 'settings.p' may be used uninitialized in this function [-Wmaybe-uninitialized]
>> return (unsigned long) DIV_ROUND_UP_ULL(rate, settings->p * 10000);
>> ^~
>> sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.p' was declared here
>> struct clk_aic32x4_pll_muldiv settings;
>> ^~~~~~~~
>>
>> Introduced by commit
>>
>> 514b044cba66 ("ASoC: tlv320aic32x4: Model PLL in CCF")
>
> I am still getting these warnings.
>
> --
> Cheers,
> Stephen Rothwell
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2019-03-19 23:45 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2019-03-19 23:45 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List,
Charles Keepax, Wen Shi
[-- Attachment #1: Type: text/plain, Size: 2484 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
In file included from include/linux/pm_runtime.h:12,
from sound/soc/codecs/wm_adsp.c:21:
sound/soc/codecs/wm_adsp.c: In function 'wmfw_parse_id_header':
sound/soc/codecs/wm_adsp.c:45:22: warning: format '%zu' expects argument of type 'size_t', but argument 8 has type 'int' [-Wformat=]
dev_info(_dsp->dev, "%s: " fmt, _dsp->name, ##__VA_ARGS__)
^~~~~~
include/linux/device.h:1400:22: note: in definition of macro 'dev_fmt'
#define dev_fmt(fmt) fmt
^~~
sound/soc/codecs/wm_adsp.c:45:2: note: in expansion of macro 'dev_info'
dev_info(_dsp->dev, "%s: " fmt, _dsp->name, ##__VA_ARGS__)
^~~~~~~~
sound/soc/codecs/wm_adsp.c:2145:2: note: in expansion of macro 'adsp_info'
adsp_info(dsp, "Firmware: %x v%d.%d.%d, %zu algorithms\n",
^~~~~~~~~
sound/soc/codecs/wm_adsp.c:2145:44: note: format string is defined here
adsp_info(dsp, "Firmware: %x v%d.%d.%d, %zu algorithms\n",
~~^
%u
In file included from include/linux/pm_runtime.h:12,
from sound/soc/codecs/wm_adsp.c:21:
sound/soc/codecs/wm_adsp.c: In function 'wmfw_v3_parse_id_header':
sound/soc/codecs/wm_adsp.c:45:22: warning: format '%zu' expects argument of type 'size_t', but argument 9 has type 'int' [-Wformat=]
dev_info(_dsp->dev, "%s: " fmt, _dsp->name, ##__VA_ARGS__)
^~~~~~
include/linux/device.h:1400:22: note: in definition of macro 'dev_fmt'
#define dev_fmt(fmt) fmt
^~~
sound/soc/codecs/wm_adsp.c:45:2: note: in expansion of macro 'dev_info'
dev_info(_dsp->dev, "%s: " fmt, _dsp->name, ##__VA_ARGS__)
^~~~~~~~
sound/soc/codecs/wm_adsp.c:2158:2: note: in expansion of macro 'adsp_info'
adsp_info(dsp, "Firmware: %x vendor: 0x%x v%d.%d.%d, %zu algorithms\n",
^~~~~~~~~
sound/soc/codecs/wm_adsp.c:2158:57: note: format string is defined here
adsp_info(dsp, "Firmware: %x vendor: 0x%x v%d.%d.%d, %zu algorithms\n",
~~^
%u
Introduced by commits
a5dcb24d70ff ("ASoC: wm_adsp: Factor out parsing of firmware ID header")
170b1e123f38 ("ASoC: wm_adsp: Add support for new Halo core DSPs")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2019-02-21 2:13 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2019-02-21 2:13 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Sylwester Nawrocki
[-- Attachment #1: Type: text/plain, Size: 559 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:
sound/soc/samsung/odroid.c: In function 'odroid_audio_probe':
sound/soc/samsung/odroid.c:298:22: warning: 'cpu_dai' may be used uninitialized in this function [-Wmaybe-uninitialized]
priv->clk_i2s_bus = of_clk_get_by_name(cpu_dai, "iis");
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Introduced by commit
d832d2b246c5 ("ASoC: samsung: odroid: Fix of_node refcount unbalance")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
[parent not found: <CGME20190213013212epcas5p436da6762dea86287ed3aea31ef2bb830@epcas5p4.samsung.com>]
* linux-next: build warning after merge of the sound-asoc tree
@ 2019-02-13 1:31 ` Stephen Rothwell
2019-02-13 10:35 ` Sylwester Nawrocki
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2019-02-13 1:31 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux Next Mailing List, Linux Kernel Mailing List, Sylwester Nawrocki
[-- Attachment #1: Type: text/plain, Size: 513 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:
sound/soc/samsung/i2s.c: In function 'samsung_i2s_remove':
sound/soc/samsung/i2s.c:1469:27: warning: unused variable 'priv' [-Wunused-variable]
struct samsung_i2s_priv *priv = dev_get_drvdata(&pdev->dev);
^~~~
Introduced by commit
a404b72d2bdd ("ASoC: samsung: i2s: Convert to single component with multiple DAIs")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2019-02-13 1:31 ` Stephen Rothwell
@ 2019-02-13 10:35 ` Sylwester Nawrocki
0 siblings, 0 replies; 75+ messages in thread
From: Sylwester Nawrocki @ 2019-02-13 10:35 UTC (permalink / raw)
To: Stephen Rothwell, Mark Brown
Cc: Liam Girdwood, Linux Next Mailing List, Linux Kernel Mailing List
Hi Stephen,
On 2/13/19 02:31, Stephen Rothwell wrote:
> After merging the sound-asoc tree, today's linux-next build (arm
> multi_v7_defconfig) produced this warning:
>
> sound/soc/samsung/i2s.c: In function 'samsung_i2s_remove':
> sound/soc/samsung/i2s.c:1469:27: warning: unused variable 'priv' [-Wunused-variable]
> struct samsung_i2s_priv *priv = dev_get_drvdata(&pdev->dev);
^~~~
>
> Introduced by commit
>
> a404b72d2bdd ("ASoC: samsung: i2s: Convert to single component with multiple DAIs")
Thanks for the information. Due to my messed up rebase the line
quoted above ended up in the above mentioned commit instead of
in the next on in the series, which has not yet been applied:
"ASoC: samsung: i2s: Restore support for the secondary PCM"
--
Regards
Sylwester
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2018-09-07 0:57 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2018-09-07 0:57 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Charles Keepax
[-- Attachment #1: Type: text/plain, Size: 592 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:
sound/soc/soc-dapm.c: In function 'snd_soc_dai_link_event':
sound/soc/soc-dapm.c:3771:9: warning: 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized]
return ret;
^~~
This looks like a (subtle) false positive. You have to assume that all
the loops will be executed at least once.
Probably introduced by commit
4a75aae17b2a ("ASoC: dapm: Add support for multi-CODEC CODEC to CODEC links")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2018-08-02 4:06 Stephen Rothwell
2018-08-02 14:37 ` Mark Brown
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2018-08-02 4:06 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Rohit kumar
[-- Attachment #1: Type: text/plain, Size: 499 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
WARNING: unmet direct dependencies detected for SND_SOC_QDSP6
Depends on [n]: SOUND [=m] && !UML && SND [=m] && SND_SOC [=m] && QCOM_APR [=n] && HAS_DMA [=y]
Selected by [m]:
- SND_SOC_SDM845 [=m] && SOUND [=m] && !UML && SND [=m] && SND_SOC [=m]
Introduced by commit
6b1687bf76ef ("ASoC: qcom: add sdm845 sound card support")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2018-08-02 4:06 Stephen Rothwell
@ 2018-08-02 14:37 ` Mark Brown
0 siblings, 0 replies; 75+ messages in thread
From: Mark Brown @ 2018-08-02 14:37 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Liam Girdwood, Linux-Next Mailing List,
Linux Kernel Mailing List, Rohit kumar
[-- Attachment #1: Type: text/plain, Size: 283 bytes --]
On Thu, Aug 02, 2018 at 02:06:30PM +1000, Stephen Rothwell wrote:
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
I was sent a patch for this already, should be fixed tomorrow. Thanks
for reporting.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2018-07-26 2:19 Stephen Rothwell
2018-07-26 5:58 ` Agrawal, Akshu
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2018-07-26 2:19 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Akshu Agrawal
[-- Attachment #1: Type: text/plain, Size: 621 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/amd/acp-da7219-max98357a.c: In function 'cz_probe':
sound/soc/amd/acp-da7219-max98357a.c:367:3: warning: 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized]
dev_err(&pdev->dev, "Failed to register regulator: %d\n",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
ret);
~~~~
Introduced by commit
7b5317aa809f ("ASoC: AMD: Add a fix voltage regulator for DA7219 and ADAU7002")
This is a real bug.
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2018-07-26 2:19 Stephen Rothwell
@ 2018-07-26 5:58 ` Agrawal, Akshu
0 siblings, 0 replies; 75+ messages in thread
From: Agrawal, Akshu @ 2018-07-26 5:58 UTC (permalink / raw)
To: Stephen Rothwell, Mark Brown, Liam Girdwood
Cc: Linux-Next Mailing List, Linux Kernel Mailing List
On 7/26/2018 7:49 AM, Stephen Rothwell wrote:
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> sound/soc/amd/acp-da7219-max98357a.c: In function 'cz_probe':
> sound/soc/amd/acp-da7219-max98357a.c:367:3: warning: 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized]
> dev_err(&pdev->dev, "Failed to register regulator: %d\n",
> ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> ret);
> ~~~~
>
> Introduced by commit
>
> 7b5317aa809f ("ASoC: AMD: Add a fix voltage regulator for DA7219 and ADAU7002")
>
> This is a real bug.
>
Posted the fix for the above warning message:
https://patchwork.kernel.org/patch/10545235/
Thanks,
Akshu
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2018-05-10 3:09 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2018-05-10 3:09 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Peter Ujfalusi
[-- Attachment #1: Type: text/plain, Size: 515 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
WARNING: modpost: missing MODULE_LICENSE() in sound/soc/omap/snd-soc-sdma.o
see include/linux/module.h for more information
WARNING: modpost: missing MODULE_LICENSE() in sound/soc/omap/snd-soc-sdma.o
see include/linux/module.h for more information
Introduced by commit
dde637f2daf1 ("ASoC: omap: Introduce the generic_dmaengine_pcm based sdma-pcm")
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2018-02-13 1:51 Stephen Rothwell
2018-02-13 2:55 ` Kuninori Morimoto
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2018-02-13 1:51 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Kuninori Morimoto
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/codecs/wm9081.c: In function 'wm9081_digital_mute':
sound/soc/codecs/wm9081.c:1161:7: warning: 'reg' may be used uninitialized in this function [-Wmaybe-uninitialized]
reg |= WM9081_DAC_MUTE;
Introduced by commit
48c338764296 ("ASoC: wm9081: replace codec to component")
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2018-02-13 1:51 Stephen Rothwell
@ 2018-02-13 2:55 ` Kuninori Morimoto
0 siblings, 0 replies; 75+ messages in thread
From: Kuninori Morimoto @ 2018-02-13 2:55 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, Linux-Next Mailing List,
Linux Kernel Mailing List
Hi Stephen
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> sound/soc/codecs/wm9081.c: In function 'wm9081_digital_mute':
> sound/soc/codecs/wm9081.c:1161:7: warning: 'reg' may be used uninitialized in this function [-Wmaybe-uninitialized]
> reg |= WM9081_DAC_MUTE;
>
> Introduced by commit
>
> 48c338764296 ("ASoC: wm9081: replace codec to component")
Thank you for your report.
I posted fixup patch for it.
Best regards
---
Kuninori Morimoto
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2017-11-01 4:13 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2017-11-01 4:13 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Olivier Moysan
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/stm/stm32_sai_sub.c: In function 'stm32_sai_hw_params':
sound/soc/stm/stm32_sai_sub.c:485:7: warning: 'cr1' may be used uninitialized in this function [-Wmaybe-uninitialized]
cr1 |= SAI_XCR1_DS_SET(SAI_DATASIZE_8);
^
sound/soc/stm/stm32_sai_sub.c:469:6: note: 'cr1' was declared here
int cr1, cr1_mask, ret;
^
Introduced by commit
61fb4ff70377 ("ASoC: stm32: sai: Move static settings to DAI init")
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2017-09-04 2:27 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2017-09-04 2:27 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Li Xu
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/codecs/cs43130.c: In function 'cs43130_imp_meas':
sound/soc/codecs/cs43130.c:2089:18: warning: 'hpload_seq' may be used uninitialized in this function [-Wmaybe-uninitialized]
hpload_seq[i].msk, ac_idx);
^
Introduced by commit
8f1e5bf9b440 ("ASoC: cs43130: Add support for CS43130 codec")
This is probably a false positive.
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2017-08-23 2:20 Stephen Rothwell
2017-08-23 6:03 ` Takashi Iwai
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2017-08-23 2:20 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Takashi Iwai
Hi all,
After merging the sound-asoc tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:
sound/soc/tegra/tegra_wm8903.c: In function 'tegra_wm8903_remove':
sound/soc/tegra/tegra_wm8903.c:206:23: warning: unused variable 'machine' [-Wunused-variable]
struct tegra_wm8903 *machine = snd_soc_card_get_drvdata(card);
^
Introduced by commit
2a8bd83af30e ("ASoC: tegra: Remove superfluous snd_soc_jack_free_gpios() call")
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2017-08-23 2:20 Stephen Rothwell
@ 2017-08-23 6:03 ` Takashi Iwai
0 siblings, 0 replies; 75+ messages in thread
From: Takashi Iwai @ 2017-08-23 6:03 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, Linux-Next Mailing List,
Linux Kernel Mailing List
On Wed, 23 Aug 2017 04:20:07 +0200,
Stephen Rothwell wrote:
>
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (arm
> multi_v7_defconfig) produced this warning:
>
> sound/soc/tegra/tegra_wm8903.c: In function 'tegra_wm8903_remove':
> sound/soc/tegra/tegra_wm8903.c:206:23: warning: unused variable 'machine' [-Wunused-variable]
> struct tegra_wm8903 *machine = snd_soc_card_get_drvdata(card);
> ^
>
> Introduced by commit
>
> 2a8bd83af30e ("ASoC: tegra: Remove superfluous snd_soc_jack_free_gpios() call")
My bad, will submit the fix patch.
thanks,
Takashi
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2017-07-03 2:26 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2017-07-03 2:26 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Vunny Sodhi
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
In file included from sound/soc/intel/skylake/skl-debug.c:23:0:
sound/soc/intel/skylake/../common/sst-dsp-priv.h:63:42: warning: 'struct sst_pdata' declared inside parameter list
int (*init)(struct sst_dsp *sst, struct sst_pdata *pdata);
^
sound/soc/intel/skylake/../common/sst-dsp-priv.h:63:42: warning: its scope is only this definition or declaration, which is probably not what you want
Exposed by commit
bdd0384a5ada ("ASoC: Intel: Skylake: Add support to read firmware registers")
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2017-04-27 1:29 Stephen Rothwell
2017-04-27 2:27 ` Vinod Koul
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2017-04-27 1:29 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: Linux-Next Mailing List, Linux Kernel Mailing List, G Kranthi,
Subhransu S. Prusty, Vinod Koul
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/intel/skylake/bxt-sst.c: In function 'bxt_sst_dsp_init':
sound/soc/intel/skylake/bxt-sst.c:567:14: warning: 'skl' may be used uninitialized in this function [-Wmaybe-uninitialized]
dev_err(skl->dev, "%s: no device\n", __func__);
^
Introduced by commit
9fe9c7119283 ("ASoC: Intel: Skylake: Move sst common initialization to a helper function")
This looks like an actual bug ...
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2017-04-27 1:29 Stephen Rothwell
@ 2017-04-27 2:27 ` Vinod Koul
0 siblings, 0 replies; 75+ messages in thread
From: Vinod Koul @ 2017-04-27 2:27 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, Linux-Next Mailing List,
Linux Kernel Mailing List, G Kranthi, Subhransu S. Prusty
On Thu, Apr 27, 2017 at 11:29:20AM +1000, Stephen Rothwell wrote:
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> sound/soc/intel/skylake/bxt-sst.c: In function 'bxt_sst_dsp_init':
> sound/soc/intel/skylake/bxt-sst.c:567:14: warning: 'skl' may be used uninitialized in this function [-Wmaybe-uninitialized]
> dev_err(skl->dev, "%s: no device\n", __func__);
> ^
>
Yeah looks like that is the case, fortunately fix is quite simple, I will
send out soon
Thanks for reporting
> Introduced by commit
>
> 9fe9c7119283 ("ASoC: Intel: Skylake: Move sst common initialization to a helper function")
>
> This looks like an actual bug ...
>
> --
> Cheers,
> Stephen Rothwell
--
~Vinod
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2017-02-01 2:14 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2017-02-01 2:14 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: linux-next, linux-kernel, Pierre-Louis Bossart
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/intel/boards/cht_bsw_rt5645.c: In function 'cht_codec_init':
sound/soc/intel/boards/cht_bsw_rt5645.c:265:22: warning: unused variable 'codec_dai' [-Wunused-variable]
struct snd_soc_dai *codec_dai = runtime->codec_dai;
^
Introduced by commit
7bde09dfcf60 ("ASoC: Intel: cht-bsw-rt5645: fix DAI formats")
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2016-12-06 0:11 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2016-12-06 0:11 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: linux-next, linux-kernel, Mengdong Lin
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/soc-topology.c: In function 'soc_tplg_dapm_widget_create':
sound/soc/soc-topology.c:1566:36: warning: 'kcontrol_type' may be used uninitialized in this function [-Wmaybe-uninitialized]
widget->dobj.widget.kcontrol_type = kcontrol_type;
^
Introduced by commit
eea3dd4f1247 ("ASoC: topology: Only free TLV for volume mixers of a widget")
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2015-07-13 3:23 Stephen Rothwell
2015-07-13 6:14 ` Valentin Rothberg
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2015-07-13 3:23 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: linux-next, linux-kernel, Arnaud Pouliquen
[-- Attachment #1: Type: text/plain, Size: 428 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (powerpc
allyesconfig) produced this warning:
warning: (SND_SOC_ALL_CODECS) selects SND_SOC_STI_SAS which has unmet direct dependencies (SOUND && !M68K && !UML && SND && SND_SOC && SND_SOC_STI)
Introduced by commit
32a726b2e089 ("ASoC: sti-sas: Add sti platform codec")
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2015-07-13 3:23 Stephen Rothwell
@ 2015-07-13 6:14 ` Valentin Rothberg
0 siblings, 0 replies; 75+ messages in thread
From: Valentin Rothberg @ 2015-07-13 6:14 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, linux-next, linux-kernel,
Arnaud Pouliquen, Paul Bolle, Andreas Ruprecht, hengelein Stefan
Hi,
On Mon, Jul 13, 2015 at 5:23 AM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (powerpc
> allyesconfig) produced this warning:
>
> warning: (SND_SOC_ALL_CODECS) selects SND_SOC_STI_SAS which has unmet direct dependencies (SOUND && !M68K && !UML && SND && SND_SOC && SND_SOC_STI)
Here's the introducing hunk:
+config SND_SOC_STI_SAS
+ tristate "codec Audio support for STI SAS codec"
+ depends on SND_SOC_STI
SND_SOC_STI is not defined in Kconfig.
Kind regards,
Valentin
> Introduced by commit
>
> 32a726b2e089 ("ASoC: sti-sas: Add sti platform codec")
>
> --
> Cheers,
> Stephen Rothwell sfr@canb.auug.org.au
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2015-03-19 3:45 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2015-03-19 3:45 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: linux-next, linux-kernel, Nikesh Oswal
[-- Attachment #1: Type: text/plain, Size: 524 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:
sound/soc/soc-dapm.c: In function 'snd_soc_dapm_new_pcm':
sound/soc/soc-dapm.c:3389:4: warning: passing argument 1 of 'snprintf' discards 'const' qualifier from pointer target type
snprintf(w_param_text[count], len,
^
Introduced by commit c66150824b8a ("ASoC: dapm: add code to configure
dai link parameters").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2014-04-15 1:44 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2014-04-15 1:44 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: linux-next, linux-kernel, Lars-Peter Clausen
[-- Attachment #1: Type: text/plain, Size: 672 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/intel/sst-haswell-pcm.c: In function 'hsw_stream_volume_put':
sound/soc/intel/sst-haswell-pcm.c:139:38: warning: initialization from incompatible pointer type [enabled by default]
struct snd_soc_platform *platform = snd_soc_kcontrol_platform(kcontrol);
^
And several others ... really, you should try build testing this stuff :-(
Caused by commit f6272ff8a5f4 ("ASoC: Add snd_soc_kcontrol_platform()
helper function").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2014-02-20 3:01 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2014-02-20 3:01 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: linux-next, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 536 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/codecs/cs42l51.c: In function 'cs42l51_probe':
sound/soc/codecs/cs42l51.c:490:26: warning: unused variable 'cs42l51' [-Wunused-variable]
struct cs42l51_private *cs42l51 = snd_soc_codec_get_drvdata(codec);
^
Introduced by commit da0714897624 ("ASoC: cs42l51: Convert to direct
regmap API usage").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2012-06-08 4:03 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2012-06-08 4:03 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: linux-next, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 484 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/soc-dapm.c: In function 'dapm_clock_event':
sound/soc/soc-dapm.c:1022:1: warning: control reaches end of non-void function [-Wreturn-type]
This is with CONFIG_HAVE_CLK not set. Introduced by commit ec02995adad5
("ASoC: dapm: Bodge for lack of a widely available clk API").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2012-04-05 1:03 Stephen Rothwell
0 siblings, 0 replies; 75+ messages in thread
From: Stephen Rothwell @ 2012-04-05 1:03 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: linux-next, linux-kernel, M R Swami Reddy
[-- Attachment #1: Type: text/plain, Size: 633 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/codecs/lm49453.c: In function 'lm49453_set_dai_fmt':
sound/soc/codecs/lm49453.c:1189:4: warning: overflow in implicit constant conversion [-Woverflow]
sound/soc/codecs/lm49453.c:1193:4: warning: overflow in implicit constant conversion [-Woverflow]
sound/soc/codecs/lm49453.c:1197:4: warning: overflow in implicit constant conversion [-Woverflow]
Introduced by commit dbf7a733f5fb (ASoC: Support TI LM49453 Audio driver"").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2012-01-09 1:55 Stephen Rothwell
2012-01-09 3:03 ` Axel Lin
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2012-01-09 1:55 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: linux-next, linux-kernel, Axel Lin
[-- Attachment #1: Type: text/plain, Size: 680 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (powerpc
ppc64_defconfig and x86_64 allmodconfig) produced this warning:
drivers/misc/Kconfig:212:error: recursive dependency detected!
drivers/misc/Kconfig:212: symbol ATMEL_SSC is selected by SND_ATMEL_SOC_SSC
sound/soc/atmel/Kconfig:9: symbol SND_ATMEL_SOC_SSC is selected by SND_AT91_SOC_SAM9G20_WM8731
sound/soc/atmel/Kconfig:18: symbol SND_AT91_SOC_SAM9G20_WM8731 depends on ATMEL_SSC
Probably introduced by commit 739be96ab837 ("ASoC: Fix build dependency
for SND_ATMEL_SOC_SSC").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2012-01-09 1:55 Stephen Rothwell
@ 2012-01-09 3:03 ` Axel Lin
2012-01-09 3:10 ` Mark Brown
0 siblings, 1 reply; 75+ messages in thread
From: Axel Lin @ 2012-01-09 3:03 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: Mark Brown, Liam Girdwood, linux-next, linux-kernel
2012/1/9 Stephen Rothwell <sfr@canb.auug.org.au>:
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (powerpc
> ppc64_defconfig and x86_64 allmodconfig) produced this warning:
>
> drivers/misc/Kconfig:212:error: recursive dependency detected!
> drivers/misc/Kconfig:212: symbol ATMEL_SSC is selected by SND_ATMEL_SOC_SSC
> sound/soc/atmel/Kconfig:9: symbol SND_ATMEL_SOC_SSC is selected by SND_AT91_SOC_SAM9G20_WM8731
> sound/soc/atmel/Kconfig:18: symbol SND_AT91_SOC_SAM9G20_WM8731 depends on ATMEL_SSC
>
> Probably introduced by commit 739be96ab837 ("ASoC: Fix build dependency
> for SND_ATMEL_SOC_SSC").
Hi Mark,
I think this commit should be replaced by [PATCH] ASoC: Fix build
dependency for SND_AT91_SOC_AFEB9260
http://mailman.alsa-project.org/pipermail/alsa-devel/2012-January/047861.html
Should I send a revert patch for commit 739be96ab837? Or you can do it
by git-revert command directly?
Thanks,
Axel
>
> --
> Cheers,
> Stephen Rothwell sfr@canb.auug.org.au
> http://www.canb.auug.org.au/~sfr/
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2010-12-27 23:32 Stephen Rothwell
2010-12-28 9:18 ` Peter Ujfalusi
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2010-12-27 23:32 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: linux-next, linux-kernel, Peter Ujfalusi
[-- Attachment #1: Type: text/plain, Size: 493 bytes --]
Hi all,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/codecs/tpa6130a2.c: In function 'tpa6130a2_add_controls':
sound/soc/codecs/tpa6130a2.c:342: warning: unused variable 'dapm'
Introduced by commit 39646871a47fd8808c08de0ce7d7ca8393af2805 ("ASoC:
tpa6130a2: Replace DAPM code with direct interface").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 490 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2010-12-27 23:32 Stephen Rothwell
@ 2010-12-28 9:18 ` Peter Ujfalusi
0 siblings, 0 replies; 75+ messages in thread
From: Peter Ujfalusi @ 2010-12-28 9:18 UTC (permalink / raw)
To: ext Stephen Rothwell; +Cc: Mark Brown, Liam Girdwood, linux-next, linux-kernel
Hi,
On 12/28/2010 01:32 AM, ext Stephen Rothwell wrote:
> Hi all,
>
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> sound/soc/codecs/tpa6130a2.c: In function 'tpa6130a2_add_controls':
> sound/soc/codecs/tpa6130a2.c:342: warning: unused variable 'dapm'
>
> Introduced by commit 39646871a47fd8808c08de0ce7d7ca8393af2805 ("ASoC:
> tpa6130a2: Replace DAPM code with direct interface").
Sorry about that.
I'll send the fix in a few minutes.
--
Péter
^ permalink raw reply [flat|nested] 75+ messages in thread
* linux-next: build warning after merge of the sound-asoc tree
@ 2010-08-17 2:21 Stephen Rothwell
2010-08-17 23:30 ` Liam Girdwood
0 siblings, 1 reply; 75+ messages in thread
From: Stephen Rothwell @ 2010-08-17 2:21 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood; +Cc: linux-next, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 506 bytes --]
Hi ,
After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:
sound/soc/soc-core.c: In function 'snd_soc_register_dais':
sound/soc/soc-core.c:2919: warning: format '%d' expects type 'int', but argument 5 has type 'size_t'
Caused by commit f0fba2ad1b6b53d5360125c41953b7afcd6deff0 ("ASoC:
multi-component - ASoC Multi-Component Support").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 490 bytes --]
^ permalink raw reply [flat|nested] 75+ messages in thread
* Re: linux-next: build warning after merge of the sound-asoc tree
2010-08-17 2:21 Stephen Rothwell
@ 2010-08-17 23:30 ` Liam Girdwood
2010-08-18 8:53 ` Mark Brown
0 siblings, 1 reply; 75+ messages in thread
From: Liam Girdwood @ 2010-08-17 23:30 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: Mark Brown, linux-next, linux-kernel
On Tue, 2010-08-17 at 12:21 +1000, Stephen Rothwell wrote:
> Hi ,
>
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> sound/soc/soc-core.c: In function 'snd_soc_register_dais':
> sound/soc/soc-core.c:2919: warning: format '%d' expects type 'int', but argument 5 has type 'size_t'
>
> Caused by commit f0fba2ad1b6b53d5360125c41953b7afcd6deff0 ("ASoC:
> multi-component - ASoC Multi-Component Support").
Ah, this never showed up on ARM since size_t is 32 bits.
Mark, could you apply this fix.
Thanks
Liam
>From 4c3f9d5fcb46d769f4a52a044fead863419c1d58 Mon Sep 17 00:00:00 2001
From: Liam Girdwood <lrg@slimlogic.co.uk>
Date: Wed, 18 Aug 2010 00:25:12 +0100
Subject: [PATCH] ASoC: core - fix build warning on x86_64
Output size_t type as a "%Zu" to avoid warnings.
Signed-off-by: Liam Girdwood <lrg@slimlogic.co.uk>
---
sound/soc/soc-core.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c
index 3d480eb..7093c17 100644
--- a/sound/soc/soc-core.c
+++ b/sound/soc/soc-core.c
@@ -2916,7 +2916,7 @@ int snd_soc_register_dais(struct device *dev,
struct snd_soc_dai *dai;
int i, ret = 0;
- dev_dbg(dev, "dai register %s #%d\n", dev_name(dev), count);
+ dev_dbg(dev, "dai register %s #%Zu\n", dev_name(dev), count);
for (i = 0; i < count; i++) {
--
1.7.0.4
^ permalink raw reply related [flat|nested] 75+ messages in thread
end of thread, other threads:[~2023-12-12 9:35 UTC | newest]
Thread overview: 75+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-13 2:16 linux-next: build warning after merge of the sound-asoc tree Stephen Rothwell
2023-07-13 12:19 ` Andy Shevchenko
2023-07-13 12:41 ` Andy Shevchenko
2023-07-14 9:18 ` Andy Shevchenko
2023-07-24 3:58 ` Stephen Rothwell
2023-07-24 8:26 ` Andy Shevchenko
2023-07-24 12:36 ` Mark Brown
2023-07-24 13:05 ` Andy Shevchenko
2023-07-24 13:11 ` Mark Brown
2023-07-24 13:44 ` Andy Shevchenko
2023-07-24 15:18 ` Jonathan Corbet
-- strict thread matches above, loose matches on Subject: below --
2023-12-12 0:16 Stephen Rothwell
2023-12-12 9:35 ` Charles Keepax
2021-10-26 7:54 Stephen Rothwell
2021-08-13 2:17 Stephen Rothwell
2021-08-13 14:35 ` Doug Anderson
2021-04-19 3:41 Stephen Rothwell
2021-03-23 5:43 Stephen Rothwell
2021-03-23 6:13 ` Mukunda,Vijendar
2020-11-11 2:34 Stephen Rothwell
2020-10-02 4:14 Stephen Rothwell
2020-09-18 3:21 Stephen Rothwell
2020-09-18 9:21 ` Kai Vehmanen
2020-08-19 1:34 Stephen Rothwell
2020-05-28 3:57 Stephen Rothwell
2020-05-29 13:30 ` Mark Brown
2020-05-29 13:55 ` Pierre-Louis Bossart
2020-05-29 14:27 ` Pierre-Louis Bossart
2020-04-24 6:26 Stephen Rothwell
2020-03-03 1:29 Stephen Rothwell
2020-03-01 23:57 Stephen Rothwell
2019-10-22 23:45 Stephen Rothwell
2019-10-07 23:58 Stephen Rothwell
2019-04-26 3:05 Stephen Rothwell
2019-04-26 3:14 ` Nicolin Chen
2019-04-09 1:46 Stephen Rothwell
2019-03-25 23:01 Stephen Rothwell
2019-04-08 23:12 ` Stephen Rothwell
2019-04-09 4:04 ` Annaliese McDermond
2019-03-19 23:45 Stephen Rothwell
2019-02-21 2:13 Stephen Rothwell
[not found] <CGME20190213013212epcas5p436da6762dea86287ed3aea31ef2bb830@epcas5p4.samsung.com>
2019-02-13 1:31 ` Stephen Rothwell
2019-02-13 10:35 ` Sylwester Nawrocki
2018-09-07 0:57 Stephen Rothwell
2018-08-02 4:06 Stephen Rothwell
2018-08-02 14:37 ` Mark Brown
2018-07-26 2:19 Stephen Rothwell
2018-07-26 5:58 ` Agrawal, Akshu
2018-05-10 3:09 Stephen Rothwell
2018-02-13 1:51 Stephen Rothwell
2018-02-13 2:55 ` Kuninori Morimoto
2017-11-01 4:13 Stephen Rothwell
2017-09-04 2:27 Stephen Rothwell
2017-08-23 2:20 Stephen Rothwell
2017-08-23 6:03 ` Takashi Iwai
2017-07-03 2:26 Stephen Rothwell
2017-04-27 1:29 Stephen Rothwell
2017-04-27 2:27 ` Vinod Koul
2017-02-01 2:14 Stephen Rothwell
2016-12-06 0:11 Stephen Rothwell
2015-07-13 3:23 Stephen Rothwell
2015-07-13 6:14 ` Valentin Rothberg
2015-03-19 3:45 Stephen Rothwell
2014-04-15 1:44 Stephen Rothwell
2014-02-20 3:01 Stephen Rothwell
2012-06-08 4:03 Stephen Rothwell
2012-04-05 1:03 Stephen Rothwell
2012-01-09 1:55 Stephen Rothwell
2012-01-09 3:03 ` Axel Lin
2012-01-09 3:10 ` Mark Brown
2010-12-27 23:32 Stephen Rothwell
2010-12-28 9:18 ` Peter Ujfalusi
2010-08-17 2:21 Stephen Rothwell
2010-08-17 23:30 ` Liam Girdwood
2010-08-18 8:53 ` Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).