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 all K3
devices since the size impact is less than 100 bytes.

Signed-off-by: Anshul Dalal <[email protected]>
---
Changes in v3:
- Handle setting the config in Kconfig instead of the defconfig
- Link to v2: 
https://lore.kernel.org/r/[email protected]

Changes in v2:
- Remove unrelated changes
- Link to v1: 
https://lore.kernel.org/r/[email protected]
---
 arch/arm/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 3db5474a05b..7756ed000b1 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -830,6 +830,7 @@ config ARCH_K3
        bool "Texas Instruments' K3 Architecture"
        select SPL
        select SUPPORT_SPL
+       select SPL_USE_TINY_PRINTF_POINTER_SUPPORT
        select FIT
        select REGEX
        select FIT_SIGNATURE if ARM64

---
base-commit: dca19206acf2af2d339087bb62aa0b8ee1b0e326
change-id: 20251105-am65_dfu_fix-1144eec06b19

Best regards,
-- 
Anshul Dalal <[email protected]>

Reply via email to