On 1/8/25 19:48, Patrick DELAUNAY wrote:
> Hi,
>
> On 11/29/24 13:27, Patrice Chotard wrote:
>> If CONFIG_SYS_64BIT_LBA flag is enable, following warning is triggered:
>>
>> ../arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c: In function
>> 'init_device':
>> ../arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c:793:27: warning: format
>> '%ld' expects argument of type 'long int', but argument 8 has type
>> 'lbaint_t' {aka 'long long unsigned int'} [-Wformat=]
>> 793 | log_debug("MMC %d: lba=%ld blksz=%ld\n",
>> dev->dev_id,
>> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> ../include/log.h:157:21: note: in definition of macro 'pr_fmt'
>> 157 | #define pr_fmt(fmt) fmt
>> | ^~~
>> ../include/log.h:182:33: note: in expansion of macro 'log'
>> 182 | #define log_debug(_fmt...) log(LOG_CATEGORY, LOGL_DEBUG,
>> ##_fmt)
>> | ^~~
>> ../arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c:793:17: note: in expansion
>> of macro 'log_debug'
>> 793 | log_debug("MMC %d: lba=%ld blksz=%ld\n",
>> dev->dev_id,
>> | ^~~~~~~~~
>> ../arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c:793:42: note: format
>> string
>> is defined here
>> 793 | log_debug("MMC %d: lba=%ld blksz=%ld\n",
>> dev->dev_id,
>> | ~~^
>> | |
>> | long int
>> | %lld
>>
>> Cast block_dev->lba to u64 and set the length specifier to %lld which
>> is ok with or without CONFIG_SYS_64BIT_LBA flag.
>>
>> Signed-off-by: Patrice Chotard <[email protected]>
>> ---
>>
>> arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>
>
> Reviewed-by: Patrick Delaunay <[email protected]>
>
> Thanks
> Patrick
>
Applied to u-boot-stm32/master
Thanks
Patrice
