From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 01/18] tiny-printf: Always print zeroes
Date: Tue, 5 Jan 2016 18:08:18 +0100 [thread overview]
Message-ID: <568BF882.6050009@denx.de> (raw)
In-Reply-To: <1452011474-15207-2-git-send-email-sjg@chromium.org>
On 05.01.2016 17:30, Simon Glass wrote:
> At present this does not print zero values in numeric format (hex and
> decimal). Add a special case for this.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
> lib/tiny-printf.c | 16 ++++++++++++----
> 1 file changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/lib/tiny-printf.c b/lib/tiny-printf.c
> index efe5c25..a06abed 100644
> --- a/lib/tiny-printf.c
> +++ b/lib/tiny-printf.c
> @@ -82,13 +82,21 @@ int vprintf(const char *fmt, va_list va)
> num = -(int)num;
> out('-');
> }
> - for (div = 1000000000; div; div /= 10)
> - div_out(&num, div);
> + if (!num) {
> + out_dgt(0);
> + } else {
> + for (div = 1000000000; div; div /= 10)
> + div_out(&num, div);
> + }
> break;
> case 'x':
> num = va_arg(va, unsigned int);
> - for (div = 0x10000000; div; div /= 0x10)
> - div_out(&num, div);
> + if (!num) {
> + out_dgt(0);
> + } else {
> + for (div = 0x10000000; div; div /= 0x10)
> + div_out(&num, div);
> + }
> break;
> case 'c':
> out((char)(va_arg(va, int)));
>
Reviewed-by: Stefan Roese <sr@denx.de>
Thanks,
Stefan
next prev parent reply other threads:[~2016-01-05 17:08 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-05 16:30 [U-Boot] [PATCH 00/18] dm: video: Introduce initial driver-model video support Simon Glass
2016-01-05 16:30 ` [U-Boot] [PATCH 01/18] tiny-printf: Always print zeroes Simon Glass
2016-01-05 17:08 ` Stefan Roese [this message]
2016-01-13 21:55 ` Tom Rini
2016-01-16 1:26 ` Simon Glass
2016-01-05 16:30 ` [U-Boot] [PATCH 02/18] sandbox: Support the bmp command Simon Glass
2016-01-13 21:55 ` Tom Rini
2016-01-16 1:26 ` Simon Glass
2016-01-05 16:30 ` [U-Boot] [PATCH 03/18] dm: core: Call uclass post_bind() after the driver's bind() method Simon Glass
2016-01-13 21:55 ` Tom Rini
2016-01-16 1:26 ` Simon Glass
2016-01-05 16:31 ` [U-Boot] [PATCH 04/18] bzip2: Support compression for sandbox Simon Glass
2016-01-13 21:55 ` Tom Rini
2016-01-16 1:26 ` Simon Glass
2016-01-05 16:31 ` [U-Boot] [PATCH 05/18] dm: video: Add a video uclass Simon Glass
2016-01-17 18:35 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 06/18] dm: lcd: Avoid using the lcd.h header file with driver model Simon Glass
2016-01-17 18:40 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 07/18] dm: video: Add a uclass for the text console Simon Glass
2016-01-17 19:09 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 08/18] dm: video: Add a 'normal' text console driver Simon Glass
2016-01-17 20:04 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 09/18] dm: video: Add a driver for a rotated text console Simon Glass
2016-01-17 20:07 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 10/18] common: Move LCD and video memory reservation together Simon Glass
2016-01-17 23:37 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 11/18] dm: common: Add memory reservation for the video uclass Simon Glass
2016-01-17 23:39 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 12/18] dm: video: Implement the bmp command for driver model Simon Glass
2016-01-18 0:01 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 13/18] dm: stdio: video: Plumb the video uclass into stdio Simon Glass
2016-01-18 0:04 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 14/18] sandbox: Move CONFIG_VIDEO_SANDBOX_SDL to Kconfig Simon Glass
2016-01-18 0:05 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 15/18] dm: video: sandbox: Convert sandbox to use driver model for video Simon Glass
2016-01-18 0:10 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 16/18] dm: video: test: Add tests for the video uclass Simon Glass
2016-01-18 0:18 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 17/18] dm: video: test: Add tests for rotated consoles Simon Glass
2016-01-18 0:20 ` Anatolij Gustschin
2016-01-05 16:31 ` [U-Boot] [PATCH 18/18] dm: video: test: Test that bitmap display works correctly Simon Glass
2016-01-18 0:26 ` Anatolij Gustschin
2016-01-13 17:58 ` [U-Boot] [PATCH 00/18] dm: video: Introduce initial driver-model video support Hannes Schmelzer
2016-01-13 20:10 ` 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=568BF882.6050009@denx.de \
--to=sr@denx.de \
--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