* [PATCH] Sound WM8958 DSP2: Use correct format string in dev_err() call
@ 2012-01-23 20:15 Jesper Juhl
2012-01-23 21:11 ` Mark Brown
0 siblings, 1 reply; 3+ messages in thread
From: Jesper Juhl @ 2012-01-23 20:15 UTC (permalink / raw)
To: Mark Brown
Cc: Takashi Iwai, alsa-devel, linux-kernel, Jaroslav Kysela,
Liam Girdwood, Dimitris Papastamos, Ian Lartey
To print a value of type size_t one should use %zd, not %d.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
---
sound/soc/codecs/wm8958-dsp2.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/sound/soc/codecs/wm8958-dsp2.c b/sound/soc/codecs/wm8958-dsp2.c
index 8d4ea43..40ac888 100644
--- a/sound/soc/codecs/wm8958-dsp2.c
+++ b/sound/soc/codecs/wm8958-dsp2.c
@@ -55,7 +55,7 @@ static int wm8958_dsp2_fw(struct snd_soc_codec *codec, const char *name,
return 0;
if (fw->size < 32) {
- dev_err(codec->dev, "%s: firmware too short (%d bytes)\n",
+ dev_err(codec->dev, "%s: firmware too short (%zd bytes)\n",
name, fw->size);
goto err;
}
--
1.7.8.4
--
Jesper Juhl <jj@chaosbits.net> http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] Sound WM8958 DSP2: Use correct format string in dev_err() call
2012-01-23 20:15 [PATCH] Sound WM8958 DSP2: Use correct format string in dev_err() call Jesper Juhl
@ 2012-01-23 21:11 ` Mark Brown
2012-01-23 21:29 ` Jesper Juhl
0 siblings, 1 reply; 3+ messages in thread
From: Mark Brown @ 2012-01-23 21:11 UTC (permalink / raw)
To: Jesper Juhl
Cc: Takashi Iwai, alsa-devel, linux-kernel, Jaroslav Kysela,
Liam Girdwood, Dimitris Papastamos, Ian Lartey
On Mon, Jan 23, 2012 at 09:15:48PM +0100, Jesper Juhl wrote:
> To print a value of type size_t one should use %zd, not %d.
Applied, thanks, but please do at least try to use a subject line in the
style for the subsystem.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Sound WM8958 DSP2: Use correct format string in dev_err() call
2012-01-23 21:11 ` Mark Brown
@ 2012-01-23 21:29 ` Jesper Juhl
0 siblings, 0 replies; 3+ messages in thread
From: Jesper Juhl @ 2012-01-23 21:29 UTC (permalink / raw)
To: Mark Brown
Cc: Takashi Iwai, alsa-devel, linux-kernel, Jaroslav Kysela,
Liam Girdwood, Dimitris Papastamos, Ian Lartey
On Mon, 23 Jan 2012, Mark Brown wrote:
> On Mon, Jan 23, 2012 at 09:15:48PM +0100, Jesper Juhl wrote:
> > To print a value of type size_t one should use %zd, not %d.
>
> Applied,
Thank you.
> thanks, but please do at least try to use a subject line in the
> style for the subsystem.
I'll try. I hope I hit closer to that mark with the other patch I just
mailed you. :-)
--
Jesper Juhl <jj@chaosbits.net> http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-01-23 21:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-23 20:15 [PATCH] Sound WM8958 DSP2: Use correct format string in dev_err() call Jesper Juhl
2012-01-23 21:11 ` Mark Brown
2012-01-23 21:29 ` Jesper Juhl
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox