[PATCH] autoboot: Fix inconsistent countdown output

[Tom Rini]

On Sun, Oct 26, 2025 at 07:24:58PM -0500, Sam Protsenko wrote:

> Commit 5f70be08b015 ("Fix autoboot countdown printing wrong") introduces
> inconsistency in how the countdown is displayed. For example, in case
> when BOOTDELAY=5, the next output is observed during the boot:
> 
>     Hit any key to stop autoboot:  5
>     Hit any key to stop autoboot: 4
>     Hit any key to stop autoboot: 3
> 
> That happens due to different printf format (%2d vs %1d). Moreover, the
> mentioned commit fails to handle the case when the user is holding some
> key before the countdown is shown. E.g. if BOOTDELAY=101, the next
> malformed output is being produced:
> 
>     Hit any key to stop autoboot: 1 0
> 
> That's because the fast path code wasn't modified accordingly, and still
> tries to erase the number using '\b\b\b' format.
> 
> Fix both issues by introducing a dedicated routine for printing the
> whole countdown line.
> 
> Fixes: 5f70be08b015 ("Fix autoboot countdown printing wrong")
> Signed-off-by: Sam Protsenko <semen.protsenko at linaro.org>

This seems the reasonable path, given everything in the following parts
of the thread, thanks.

Reviewed-by: Tom Rini <trini at konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20251028/859c8980/attachment.sig>