[PATCH v1 2/6] arm: stm32mp: stm32prog: fix warning when CONFIG_SYS_64BIT_LBA is enable

[Patrice CHOTARD]

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 <patrice.chotard at foss.st.com>
>> ---
>>
>>   arch/arm/mach-stm32mp/cmd_stm32prog/stm32prog.c | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
> 
> 
> Reviewed-by: Patrick Delaunay <patrick.delaunay at foss.st.com>
> 
> Thanks
> Patrick
> 
Applied to u-boot-stm32/master

Thanks
Patrice