Hi Tom,
Thank you for the patch.
On Tue, Jul 01, 2025 at 19:06, Tom Rini <tr...@konsulko.com> wrote:
> When printing the contents of an lbaint_t variable we need to use LBAF
> to print it in order to get the correct format type depending on 32 or
> 64bit-ness. Furthermore, printed message should not be split as that
> makes finding them harder, so bring this back to a single line.
>
> Signed-off-by: Tom Rini <tr...@konsulko.com>
Reviewed-by: Mattijs Korpershoek <mkorpersh...@kernel.org>
> ---
> Cc: Mattijs Korpershoek <mkorpersh...@kernel.org>
> Cc: Igor Opaniuk <igor.opan...@gmail.com>
> Cc: Sam Protsenko <semen.protse...@linaro.org>
> ---
> boot/android_ab.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/boot/android_ab.c b/boot/android_ab.c
> index a287eac04fe8..13e82dbcb7fe 100644
> --- a/boot/android_ab.c
> +++ b/boot/android_ab.c
> @@ -101,8 +101,7 @@ static int ab_control_create_from_disk(struct blk_desc
> *dev_desc,
> abc_blocks = DIV_ROUND_UP(sizeof(struct bootloader_control),
> part_info->blksz);
> if (abc_offset + abc_blocks > part_info->size) {
> - log_err("ANDROID: boot control partition too small. Need at");
> - log_err(" least %lu blocks but have %lu blocks.\n",
> + log_err("ANDROID: boot control partition too small. Need at
> least %lu blocks but have " LBAF " blocks.\n",
> abc_offset + abc_blocks, part_info->size);
> return -EINVAL;
> }
> --
> 2.43.0
