Since the commit 1e24e84db41a ("tiny-printf: Handle formatting of %p
with an extra Kconfig"), SPL_USE_TINY_PRINTF_POINTER_SUPPORT has been
made mandatory in order to use %p which would earlier have defaulted to
a 'long' print.Without this config symbol, k3_sysfw_dfu_download fails to set the correct value for the DFU string with: sprintf(dfu_str, "sysfw.itb ram 0x%p 0x%x", addr, CONFIG_K3_SYSFW_IMAGE_SIZE_MAX); The value we get "sysfw.itb ram 0x? 0x41c29d40" causes a boot failure. Therefore this patch sets SPL_USE_TINY_PRINTF_POINTER_SUPPORT for the usbdfu defconfig. Signed-off-by: Anshul Dalal <[email protected]> --- Not sure if a 'Fixes' tag is appropriate here since the commit 1e24e84db41a ("tiny-printf: Handle formatting of %p with an extra Kconfig") did cause a regression though I think the problem existed with the usage of "%p" with TINY_PRINTF set to begin with. --- Changes in v2: - Remove unrelated changes - Link to v1: https://lore.kernel.org/r/[email protected] --- configs/am65x_evm_r5_usbdfu_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/am65x_evm_r5_usbdfu_defconfig b/configs/am65x_evm_r5_usbdfu_defconfig index f9161f1fe111ff97f7aa79f10b21c3b0ab1dad32..add03dce002b2abd913770cd836d2cd8c852ed5e 100644 --- a/configs/am65x_evm_r5_usbdfu_defconfig +++ b/configs/am65x_evm_r5_usbdfu_defconfig @@ -6,6 +6,7 @@ CONFIG_SYS_MALLOC_F_LEN=0x57000 CONFIG_SPL_GPIO=y CONFIG_SPL_LIBCOMMON_SUPPORT=y CONFIG_SPL_LIBGENERIC_SUPPORT=y +CONFIG_SPL_USE_TINY_PRINTF_POINTER_SUPPORT=y CONFIG_SOC_K3_AM654=y CONFIG_K3_EARLY_CONS=y CONFIG_TARGET_AM654_R5_EVM=y --- base-commit: 1c250e444ad3b15315ee8b0fcb3fc3acc26449e2 change-id: 20251105-am65_dfu_fix-1144eec06b19 Best regards, -- Anshul Dalal <[email protected]>
