[U-Boot-Users] [PATCH 1/1] Fix some more printf() format issues.

[Jean-Christophe PLAGNIOL-VILLARD]

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD [EMAIL PROTECTED]
---
 common/cmd_flash.c |4 ++--
 drivers/usb/usbdcore.c |2 +-
 fs/jffs2/jffs2_1pass.c |4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/common/cmd_flash.c b/common/cmd_flash.c
index 9bd8074..d959c3d 100644
--- a/common/cmd_flash.c
+++ b/common/cmd_flash.c
@@ -342,7 +342,7 @@ int do_flerase (cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
puts (Bad sector specification\n);
return 1;
}
-   printf (Erase Flash Sectors %d-%d in Bank # %d ,
+   printf (Erase Flash Sectors %d-%d in Bank # %zu ,
sect_first, sect_last, (info-flash_info)+1);
rcode = flash_erase(info, sect_first, sect_last);
return rcode;
@@ -534,7 +534,7 @@ int do_protect (cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
puts (Bad sector specification\n);
return 1;
}
-   printf(%sProtect Flash Sectors %d-%d in Bank # %d\n,
+   printf(%sProtect Flash Sectors %d-%d in Bank # %zu\n,
p ?  : Un-, sect_first, sect_last,
(info-flash_info)+1);
for (i = sect_first; i = sect_last; i++) {
diff --git a/drivers/usb/usbdcore.c b/drivers/usb/usbdcore.c
index 808da9f..53ed669 100644
--- a/drivers/usb/usbdcore.c
+++ b/drivers/usb/usbdcore.c
@@ -552,7 +552,7 @@ struct urb *usbd_alloc_urb (struct usb_device_instance 
*device,
struct urb *urb;
 
if (!(urb = (struct urb *) malloc (sizeof (struct urb {
-   usberr ( F A T A L:  malloc(%u) FAILED,
+   usberr ( F A T A L:  malloc(%zu) FAILED,
sizeof (struct urb));
return NULL;
}
diff --git a/fs/jffs2/jffs2_1pass.c b/fs/jffs2/jffs2_1pass.c
index 5c1d265..3c454f6 100644
--- a/fs/jffs2/jffs2_1pass.c
+++ b/fs/jffs2/jffs2_1pass.c
@@ -1213,12 +1213,12 @@ jffs2_1pass_build_lists(struct part_info * part)
} else if (node-nodetype == 
JFFS2_NODETYPE_CLEANMARKER) {
if (node-totlen != sizeof(struct 
jffs2_unknown_node))
printf(OOPS Cleanmarker has bad size 
-   %d != %u\n, node-totlen,
+   %d != %zu\n, node-totlen,
sizeof(struct 
jffs2_unknown_node));
} else if (node-nodetype == JFFS2_NODETYPE_PADDING) {
if (node-totlen  sizeof(struct 
jffs2_unknown_node))
printf(OOPS Padding has bad size 
-   %d  %u\n, node-totlen,
+   %d  %zu\n, node-totlen,
sizeof(struct 
jffs2_unknown_node));
} else {
printf(Unknown node type: %x len %d 
-- 
1.5.6


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH 1/1] Fix some more printf() format issues.

[Wolfgang Denk]

In message [EMAIL PROTECTED] you wrote:
 Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD [EMAIL PROTECTED]
 ---
  common/cmd_flash.c |4 ++--
  drivers/usb/usbdcore.c |2 +-
  fs/jffs2/jffs2_1pass.c |4 ++--
  3 files changed, 5 insertions(+), 5 deletions(-)

Thanks, but your patch catches only a  small  part  of  all  uses  of
sizeof_t  in  printf() and debug() calls. I tried to find and fix the
other locations, too - see [PATCH] Fix printf() format  issues  with
sizeof_t types by using %zu

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
The universe does not have laws - it has habits, and  habits  can  be
broken.

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users