From: Jeroen Hofstee <jeroen@myspectrum.nl>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/10] Provide inttypes.h to avoid 32/64 bit problems with printf()
Date: Fri, 24 Oct 2014 19:06:33 +0200 [thread overview]
Message-ID: <544A8719.3090104@myspectrum.nl> (raw)
In-Reply-To: <CAPnjgZ2DCRoZxib8jXBXrX3Wp1bi8oszXwfDq++mf6FgdSqiPg@mail.gmail.com>
Hi Simon, attempt two..
On 24-10-14 03:03, Simon Glass wrote:
> +Jeroen
>
> Hi,
>
> On 15 October 2014 04:38, Simon Glass <sjg@chromium.org> wrote:
>> In quite a few situations we have to print a 64-bit value. Unfortunately
>> the type used for 64-bit can vary depending on the machine. For 64-bit
>> machines it might be 'long' and for 32-bit machines it might be
>> 'long long'.
>>
>> As a result we need to use either %ld or %lld depending on the architecture.
>> Add the inttypes.h header file to provide defines for this and clean up the
>> code to use these defines in a few places.
>>
>> The stdint.h file is bundled with recent versions of gcc and it is generally
>> better to use this rather than our own versions. Add an option to use the
>> internal stdint.h file.
>>
>>
>> Gabe Black (2):
>> Provide option to avoid defining a custom version of uintptr_t.
>> Add some standard headers external code might need
>>
>> Simon Glass (8):
>> ext4: Use inttypes for printf() string
>> Use uint64_t for time types
>> Use uint64_t instead of u64 in put_dec()
>> Tidy up data sizes and function comment in display_options
>> x86: Use correct printf() format string for uintptr_t
>> scsi: Use correct printf() format string for uintptr_t
>> usb: Use correct printf() format string for uintptr_t
>> test: Add a simple test to detected warnings with uint64_t, uintptr_t
> Are there any comments on this series? I'm keen to clean up the
> printf() types a bit. Also this simplifies building withe external
> libraries, and with more work might reduce the difference between
> U-Boot code in /tools and the rest of it.
No comments from my side. clang seem to digest this fine.
Regards,
Jeroen
next prev parent reply other threads:[~2014-10-24 17:06 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-15 10:38 [U-Boot] [PATCH 0/10] Provide inttypes.h to avoid 32/64 bit problems with printf() Simon Glass
2014-10-15 10:38 ` [U-Boot] [PATCH 01/10] Provide option to avoid defining a custom version of uintptr_t Simon Glass
2014-10-27 22:20 ` [U-Boot] [U-Boot, " Tom Rini
2014-10-15 10:38 ` [U-Boot] [PATCH 02/10] Add some standard headers external code might need Simon Glass
2014-10-27 22:20 ` [U-Boot] [U-Boot, " Tom Rini
2014-10-28 16:25 ` [U-Boot] [PATCH " Masahiro YAMADA
2014-10-28 16:29 ` Simon Glass
2014-10-28 16:38 ` Masahiro YAMADA
2014-10-28 17:33 ` Simon Glass
2014-10-28 17:46 ` Jeroen Hofstee
2014-10-28 18:24 ` Simon Glass
2014-10-29 14:06 ` Masahiro YAMADA
2014-10-30 1:43 ` Simon Glass
2014-10-30 7:53 ` Masahiro Yamada
2014-10-31 2:43 ` Simon Glass
2014-10-15 10:38 ` [U-Boot] [PATCH 03/10] ext4: Use inttypes for printf() string Simon Glass
2014-10-27 22:20 ` [U-Boot] [U-Boot,03/10] " Tom Rini
2014-10-15 10:38 ` [U-Boot] [PATCH 04/10] Use uint64_t for time types Simon Glass
2014-10-27 22:20 ` [U-Boot] [U-Boot,04/10] " Tom Rini
2014-12-15 16:55 ` [U-Boot] [PATCH 04/10] " Masahiro YAMADA
2014-12-15 18:38 ` Simon Glass
2014-12-16 1:38 ` Masahiro YAMADA
2014-12-17 4:38 ` Simon Glass
2014-12-22 10:22 ` Masahiro Yamada
2014-10-15 10:38 ` [U-Boot] [PATCH 05/10] Use uint64_t instead of u64 in put_dec() Simon Glass
2014-10-27 22:20 ` [U-Boot] [U-Boot, " Tom Rini
2014-10-15 10:38 ` [U-Boot] [PATCH 06/10] Tidy up data sizes and function comment in display_options Simon Glass
2014-10-27 22:20 ` [U-Boot] [U-Boot, " Tom Rini
2014-10-15 10:38 ` [U-Boot] [PATCH 07/10] x86: Use correct printf() format string for uintptr_t Simon Glass
2014-10-27 22:20 ` [U-Boot] [U-Boot, " Tom Rini
2014-10-15 10:38 ` [U-Boot] [PATCH 08/10] scsi: " Simon Glass
2014-10-27 22:20 ` [U-Boot] [U-Boot, " Tom Rini
2014-12-15 18:32 ` [U-Boot] [PATCH " Masahiro YAMADA
2014-10-15 10:38 ` [U-Boot] [PATCH 09/10] usb: " Simon Glass
2014-10-27 22:20 ` [U-Boot] [U-Boot, " Tom Rini
2014-10-15 10:38 ` [U-Boot] [PATCH 10/10] test: Add a simple test to detected warnings with uint64_t, uintptr_t Simon Glass
2014-10-27 22:20 ` [U-Boot] [U-Boot, " Tom Rini
2014-10-24 1:03 ` [U-Boot] [PATCH 0/10] Provide inttypes.h to avoid 32/64 bit problems with printf() Simon Glass
2014-10-24 7:19 ` Jeroen Hofstee
2014-10-24 17:06 ` Jeroen Hofstee [this message]
2014-10-24 17:18 ` Simon Glass
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=544A8719.3090104@myspectrum.nl \
--to=jeroen@myspectrum.nl \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.