Re: [Qemu-devel] [PATCH 1.1] es1370: Fix debug code
Am 28.05.2012 06:11, schrieb Peter Maydell: On 23 May 2012 22:26, Stefan Weils...@weilnetz.de wrote: When DEBUG_ES1370 is defined, the compiler shows these warnings: hw/es1370.c: In function ‘es1370_update_voices’: hw/es1370.c:414: warning: format ‘%d’ expects type ‘int’, but argument 3 has type ‘size_t’ The unicode quotes in this commit message seem to have got smashed to question-mark characters at some point between this email and it hitting git master :-( -- PMM Malc, could you please check your git workflow? Commit 24f50d7ea5896a30b0e78f68884586bb8b40ff97 and other commits also changed Andreas Färber to Andreas F?rber. See also a394aed235d6b3f048eeae83289f4d21eca7023c and lots more. The author is always correct, but UTF-8 characters in the commit message were replaced by ?. Regards, Stefan W.
Re: [Qemu-devel] [PATCH 1.1] es1370: Fix debug code
On Mon, 28 May 2012, Stefan Weil wrote: Am 28.05.2012 06:11, schrieb Peter Maydell: On 23 May 2012 22:26, Stefan Weils...@weilnetz.de wrote: When DEBUG_ES1370 is defined, the compiler shows these warnings: hw/es1370.c: In function ?es1370_update_voices?: hw/es1370.c:414: warning: format ?%d? expects type ?int?, but argument 3 has type ?size_t? The unicode quotes in this commit message seem to have got smashed to question-mark characters at some point between this email and it hitting git master :-( -- PMM Malc, could you please check your git workflow? Pine's '|' does that with my LC_CTYPE it seems, have to either ctrl-w after '|' or export, either way - can do. Commit 24f50d7ea5896a30b0e78f68884586bb8b40ff97 and other commits also changed Andreas F?rber to Andreas F?rber. See also a394aed235d6b3f048eeae83289f4d21eca7023c and lots more. The author is always correct, but UTF-8 characters in the commit message were replaced by ?. Regards, Stefan W. -- mailto:av1...@comtv.ru
Re: [Qemu-devel] [PATCH 1.1] es1370: Fix debug code
Am 28.05.2012 17:18, schrieb malc: On Mon, 28 May 2012, Stefan Weil wrote: Am 28.05.2012 06:11, schrieb Peter Maydell: On 23 May 2012 22:26, Stefan Weils...@weilnetz.de wrote: When DEBUG_ES1370 is defined, the compiler shows these warnings: hw/es1370.c: In function ?es1370_update_voices?: hw/es1370.c:414: warning: format ?%d? expects type ?int?, but argument 3 has type ?size_t? The unicode quotes in this commit message seem to have got smashed to question-mark characters at some point between this email and it hitting git master :-( -- PMM Malc, could you please check your git workflow? Pine's '|' does that with my LC_CTYPE it seems, have to either ctrl-w after '|' or export, either way - can do. Recently I suspected your mailer and on one of the TCG threads suggested to download from Patchwork instead. But Patchwork also has its flaws: It tries to add *-bys from the replies, in the order it receives them rather than where they were placed and, like Anthony's mailbox script, it might pick up false positives. And when there's line-wrapping in the original mail it might silently drop part of the patch or in the combination of line-wrapping and patch attachment drop and duplicate parts. So if you can fix by some clever scripting that'll be safest. :) Thanks, Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
Re: [Qemu-devel] [PATCH 1.1] es1370: Fix debug code
On 23 May 2012 22:26, Stefan Weil s...@weilnetz.de wrote: When DEBUG_ES1370 is defined, the compiler shows these warnings: hw/es1370.c: In function ‘es1370_update_voices’: hw/es1370.c:414: warning: format ‘%d’ expects type ‘int’, but argument 3 has type ‘size_t’ The unicode quotes in this commit message seem to have got smashed to question-mark characters at some point between this email and it hitting git master :-( -- PMM
[Qemu-devel] [PATCH 1.1] es1370: Fix debug code
When DEBUG_ES1370 is defined, the compiler shows these warnings:
hw/es1370.c: In function ‘es1370_update_voices’:
hw/es1370.c:414: warning: format ‘%d’ expects type ‘int’, but argument 3 has
type ‘size_t’
hw/es1370.c: In function ‘es1370_writel’:
hw/es1370.c:582: warning: format ‘%d’ expects type ‘int’, but argument 3 has
type ‘long int’
hw/es1370.c:592: warning: format ‘%d’ expects type ‘int’, but argument 3 has
type ‘long int’
hw/es1370.c:609: warning: format ‘%d’ expects type ‘int’, but argument 3 has
type ‘long int’
hw/es1370.c: In function ‘es1370_readl’:
hw/es1370.c:751: warning: suggest braces around empty body in an ‘if’ statement
Fix the format strings and add the missing braces.
Signed-off-by: Stefan Weil s...@weilnetz.de
---
hw/es1370.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/hw/es1370.c b/hw/es1370.c
index f19cef3..573f747 100644
--- a/hw/es1370.c
+++ b/hw/es1370.c
@@ -410,7 +410,7 @@ static void es1370_update_voices (ES1370State *s, uint32_t
ctl, uint32_t sctl)
if ((old_fmt != new_fmt) || (old_freq != new_freq)) {
d-shift = (new_fmt 1) + (new_fmt 1);
-ldebug (channel %d, freq = %d, nchannels %d, fmt %d, shift %d\n,
+ldebug (channel %zu, freq = %d, nchannels %d, fmt %d, shift %d\n,
i,
new_freq,
1 (new_fmt 1),
@@ -578,7 +578,7 @@ IO_WRITE_PROTO (es1370_writel)
d++;
case ES1370_REG_DAC1_SCOUNT:
d-scount = (val 0x) | (d-scount ~0x);
-ldebug (chan %d CURR_SAMP_CT %d, SAMP_CT %d\n,
+ldebug (chan %td CURR_SAMP_CT %d, SAMP_CT %d\n,
d - s-chan[0], val 16, (val 0x));
break;
@@ -588,7 +588,7 @@ IO_WRITE_PROTO (es1370_writel)
d++;
case ES1370_REG_DAC1_FRAMEADR:
d-frame_addr = val;
-ldebug (chan %d frame address %#x\n, d - s-chan[0], val);
+ldebug (chan %td frame address %#x\n, d - s-chan[0], val);
break;
case ES1370_REG_PHANTOM_FRAMECNT:
@@ -605,7 +605,7 @@ IO_WRITE_PROTO (es1370_writel)
case ES1370_REG_DAC1_FRAMECNT:
d-frame_cnt = val;
d-leftover = 0;
-ldebug (chan %d frame count %d, buffer size %d\n,
+ldebug (chan %td frame count %d, buffer size %d\n,
d - s-chan[0], val 16, val 0x);
break;
@@ -745,9 +745,10 @@ IO_READ_PROTO (es1370_readl)
{
uint32_t size = ((d-frame_cnt 0x) + 1) 2;
uint32_t curr = ((d-frame_cnt 16) + 1) 2;
-if (curr size)
+if (curr size) {
dolog (read framecnt curr %d, size %d %d\n, curr, size,
curr size);
+}
}
#endif
break;
--
1.7.10
Re: [Qemu-devel] [PATCH 1.1] es1370: Fix debug code
On Wed, 23 May 2012, Stefan Weil wrote: When DEBUG_ES1370 is defined, the compiler shows these warnings: [..snip..] Thanks, applied. -- mailto:av1...@comtv.ru
