[PATCH v4 0/2] dts: imx8mp-rsb3720: modify configrations to load fip into memory

[PATCH v4 0/2] dts: imx8mp-rsb3720: modify configrations to load fip into memory

[Ying-Chun Liu (PaulLiu)]

The changes of commit 6a21c695213b ("arm: dts: imx8mp: add of-list
support to common imx8mp-u-boot.dtsi") breaks the loading of the fip.
This commit fixes the break by modify the configuration properly.

v2: just rebase to the latest master branch.
v3: rebase to the latest master branch.
v4: fix subject and commit log: node name -> label

Ying-Chun Liu (PaulLiu) (2):
  dts: imx8mp: assign binman_configuration label to config-SEQ
  dts: imx8mp-rsb3720: modify configrations to load fip into memory

 arch/arm/dts/imx8mp-rsb3720-a1-u-boot.dtsi | 10 ++++------
 arch/arm/dts/imx8mp-u-boot.dtsi            |  2 +-
 2 files changed, 5 insertions(+), 7 deletions(-)

-- 
2.39.0

[PATCH v4 1/2] dts: imx8mp: assign binman_configuration label to config-SEQ

[Ying-Chun Liu (PaulLiu)]

assign a label for config-SEQ so that the board dts can modify
the configuration more easily.

Signed-off-by: Ying-Chun Liu (PaulLiu) <paul.liu@linaro.org>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: NXP i.MX U-Boot Team <uboot-imx@nxp.com>
---
v2: just rebase to the latest master branch.
v3: rebase to the latest master branch.
v4: fix subject and commit log: node name -> label
---
 arch/arm/dts/imx8mp-u-boot.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/imx8mp-u-boot.dtsi b/arch/arm/dts/imx8mp-u-boot.dtsi
index f9883aa133..cb9b5b649a 100644
--- a/arch/arm/dts/imx8mp-u-boot.dtsi
+++ b/arch/arm/dts/imx8mp-u-boot.dtsi
@@ -148,7 +148,7 @@
 			configurations {
 				default = "@config-DEFAULT-SEQ";
 
-				@config-SEQ {
+				binman_configuration: @config-SEQ {
 					description = "NAME";
 					fdt = "fdt-SEQ";
 					firmware = "uboot";
-- 
2.39.0

Re: [PATCH v4 1/2] dts: imx8mp: assign binman_configuration label to config-SEQ

[Fabio Estevam]

On Tue, Jan 17, 2023 at 4:15 PM Ying-Chun Liu (PaulLiu)
<paul.liu@linaro.org> wrote:
>
> assign a label for config-SEQ so that the board dts can modify
> the configuration more easily.
>
> Signed-off-by: Ying-Chun Liu (PaulLiu) <paul.liu@linaro.org>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: NXP i.MX U-Boot Team <uboot-imx@nxp.com>

Reviewed-by: Fabio Estevam <festevam@denx.de>

[PATCH v4 1/2] dts: imx8mp: assign binman_configuration label to config-SEQ

[sbabic]

> assign a label for config-SEQ so that the board dts can modify
> the configuration more easily.
> Signed-off-by: Ying-Chun Liu (PaulLiu) <paul.liu@linaro.org>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: NXP i.MX U-Boot Team <uboot-imx@nxp.com>
> Reviewed-by: Fabio Estevam <festevam@denx.de>
Applied to u-boot-imx, master, thanks !

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,        Managing Director: Erika Unter  
HRB 165235 Munich,   Office: Kirchenstr.5, 82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de
=====================================================================

[PATCH v4 2/2] dts: imx8mp-rsb3720: modify configrations to load fip into memory

[Ying-Chun Liu (PaulLiu)]

The changes of commit 6a21c695213b ("arm: dts: imx8mp: add of-list
support to common imx8mp-u-boot.dtsi") breaks the loading of the fip.
This commit fixes the break by modify the configuration properly.

Signed-off-by: Ying-Chun Liu (PaulLiu) <paul.liu@linaro.org>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: NXP i.MX U-Boot Team <uboot-imx@nxp.com>
---
v2: just rebase to the latest master branch.
v3: rebase to the latest master branch.
---
 arch/arm/dts/imx8mp-rsb3720-a1-u-boot.dtsi | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/arch/arm/dts/imx8mp-rsb3720-a1-u-boot.dtsi b/arch/arm/dts/imx8mp-rsb3720-a1-u-boot.dtsi
index 4419967ee4..32d9fbc886 100644
--- a/arch/arm/dts/imx8mp-rsb3720-a1-u-boot.dtsi
+++ b/arch/arm/dts/imx8mp-rsb3720-a1-u-boot.dtsi
@@ -158,12 +158,10 @@
 					};
 				};
 			};
-
-			configurations {
-				conf {
-					loadables = "atf", "fip";
-				};
-			};
 		};
 	};
 };
+
+&binman_configuration {
+	loadables = "atf", "fip";
+};
-- 
2.39.0

Re: [PATCH v4 2/2] dts: imx8mp-rsb3720: modify configrations to load fip into memory

[Fabio Estevam]

On Tue, Jan 17, 2023 at 4:15 PM Ying-Chun Liu (PaulLiu)
<paul.liu@linaro.org> wrote:
>
> The changes of commit 6a21c695213b ("arm: dts: imx8mp: add of-list
> support to common imx8mp-u-boot.dtsi") breaks the loading of the fip.
> This commit fixes the break by modify the configuration properly.
>
> Signed-off-by: Ying-Chun Liu (PaulLiu) <paul.liu@linaro.org>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: NXP i.MX U-Boot Team <uboot-imx@nxp.com>

Reviewed-by: Fabio Estevam <festevam@denx.de>

[PATCH v4 2/2] dts: imx8mp-rsb3720: modify configrations to load fip into memory

[sbabic]

> The changes of commit 6a21c695213b ("arm: dts: imx8mp: add of-list
> support to common imx8mp-u-boot.dtsi") breaks the loading of the fip.
> This commit fixes the break by modify the configuration properly.
> Signed-off-by: Ying-Chun Liu (PaulLiu) <paul.liu@linaro.org>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: NXP i.MX U-Boot Team <uboot-imx@nxp.com>
> Reviewed-by: Fabio Estevam <festevam@denx.de>
Applied to u-boot-imx, master, thanks !

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,        Managing Director: Erika Unter  
HRB 165235 Munich,   Office: Kirchenstr.5, 82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de
=====================================================================