Re: [PATCH 1/2] imx8qxp: Do not use CONFIG_XPL_BUILD in device tree

2025-03-17 Thread Hendrik Donner
Hello, On 17.03.25 12:10, Quentin Schulz wrote: Hi Fabio, On 3/15/25 12:58 PM, Fabio Estevam wrote: Hi Hendrik, On Fri, Mar 14, 2025 at 5:09 PM Hendrik Donner wrote: In c9713c155127 the device tree was moved from CONFIG_SPL to CONFIG_SPL_BUILD and later to CONFIG_XPL_BUILD, but the

[PATCH 2/2] imx8qm: Do not use CONFIG_XPL_BUILD in device tree

2025-03-14 Thread Hendrik Donner
build issue. Fixes: c9713c15512 ("imx8-u-boot: Fix SPL guard option") Signed-off-by: Hendrik Donner --- arch/arm/dts/imx8qm-u-boot.dtsi | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/arm/dts/imx8qm-u-boot.dtsi b/arch/arm/dts/imx8qm-u-boot.dtsi index a

[PATCH 1/2] imx8qxp: Do not use CONFIG_XPL_BUILD in device tree

2025-03-14 Thread Hendrik Donner
t;) Signed-off-by: Hendrik Donner --- arch/arm/dts/imx8qxp-u-boot.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/dts/imx8qxp-u-boot.dtsi b/arch/arm/dts/imx8qxp-u-boot.dtsi index 8058caae9ba..db6a3dedb95 100644 --- a/arch/arm/dts/imx8qxp-u-boot.dtsi +++ b/ar

[PATCH 0/2] Do not use CONFIG_XPL_BUILD in device tree

2025-03-14 Thread Hendrik Donner
As discussed on the mailing list, move instances of CONFIG_XPL_BUILD back to the initially used CONFIG_SPL in device trees. Hendrik Donner (2): imx8qxp: Do not use CONFIG_XPL_BUILD in device tree imx8qm: Do not use CONFIG_XPL_BUILD in device tree arch/arm/dts/imx8qm-u-boot.dtsi | 4

[PATCH] checkpatch.pl: Report an error for CONFIG_xPL_BUILD in device trees

2025-03-06 Thread Hendrik Donner
The defines for the different build phases shouldn't be used in device trees, currently they aren't even defined for device tree pre processing, resulting in build errors. Link: https://lists.denx.de/pipermail/u-boot/2025-March/582787.html Signed-off-by: Hendrik Donner --- scripts/che

Re: [PATCH 2/2] Makefile.lib: Set xPL build related defines for DTB build

2025-03-06 Thread Hendrik Donner
Hello, On 06.03.25 11:18, Quentin Schulz wrote: Hi Hendrik, On 3/5/25 7:35 PM, Hendrik Donner wrote: [You don't often get email from h...@os-cillation.de. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ] The CONFIG_*PL_BUILD defines are currently not de

[PATCH 2/2] Makefile.lib: Set xPL build related defines for DTB build

2025-03-05 Thread Hendrik Donner
The CONFIG_*PL_BUILD defines are currently not defined when preprocessing the dts files, leading to build problems with binman. Set the defines based on the related CONFIG_*PL values. Tested-by: Oliver Graute Signed-off-by: Hendrik Donner --- scripts/Makefile.lib | 10 ++ 1 file

[PATCH 1/2] imx8qm: Fix build when using SPL

2025-03-05 Thread Hendrik Donner
nal blobs and is non-functional: spl.bin Some images are invalid Guard creation of flash.bin with CONFIG_XPL_BUILD option. Fixes: c9713c155127 ("imx8-u-boot: Fix SPL guard option") Tested-by: Oliver Graute Signed-off-by: Hendrik Donner --- arch/arm/dts/imx8qm-u-boot.dtsi | 2 ++ 1 file change

[PATCH 0/2] Build fixes for CONFIG_XPL_BUILD in dts files

2025-03-05 Thread Hendrik Donner
Just 2 fixes related to CONFIG_XPL_BUILD usage in dts files with binman. The second patch works for me, not sure if it is exactly the correct approach. Hendrik Donner (2): imx8qm: Fix build when using SPL Makefile.lib: Set xPL build related defines for DTB build arch/arm/dts/imx8qm-u