2010-08-17 02:21:12

by Stephen Rothwell

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

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 [email protected]
http://www.canb.auug.org.au/~sfr/


Attachments:
(No filename) (492.00 B)
(No filename) (490.00 B)
Download all attachments

2010-08-17 23:30:41

by Liam Girdwood

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

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 <[email protected]>
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 <[email protected]>
---
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

2010-08-18 08:53:00

by Mark Brown

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

On Wed, Aug 18, 2010 at 12:30:30AM +0100, Liam Girdwood wrote:

> Mark, could you apply this fix.

Done.