public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

  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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox