[PATCH] imx8mp: binman: rename spl and u-boot nodes

2023-10-16 Thread sbabic 
> The hab signing script doc/imx/habv4/csf_examples/mx8m/csf.sh does
>   fdtget -t x u-boot.dtb /binman/imx-boot/uboot offset
> to figure out the offset of u-boot.itb inside flash.bin. That works
> fine for imx8mm, imx8mn, imx8mq, but fails for imx8mp because in that
> case 'uboot' is merely a label and not actually the node name.
> Homogenize these cases and make imx8mp the same as the other imx8m*
> variants. The binman type is explicitly given and no longer derived
> from the node name, and the csf.sh script will work for all four SOCs.
> Signed-off-by: Rasmus Villemoes 
> Reviewed-by: Marek Vasut 
> Reviewed-by: Fabio Estevam 
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: sba...@denx.de
=

Re: [PATCH] imx8mp: binman: rename spl and u-boot nodes

2023-09-20 Thread Fabio Estevam 
On Tue, Sep 19, 2023 at 6:29 AM Rasmus Villemoes
 wrote:
>
> The hab signing script doc/imx/habv4/csf_examples/mx8m/csf.sh does
>
>   fdtget -t x u-boot.dtb /binman/imx-boot/uboot offset
>
> to figure out the offset of u-boot.itb inside flash.bin. That works
> fine for imx8mm, imx8mn, imx8mq, but fails for imx8mp because in that
> case 'uboot' is merely a label and not actually the node name.
>
> Homogenize these cases and make imx8mp the same as the other imx8m*
> variants. The binman type is explicitly given and no longer derived
> from the node name, and the csf.sh script will work for all four SOCs.
>
> Signed-off-by: Rasmus Villemoes 

Reviewed-by: Fabio Estevam

Re: [PATCH] imx8mp: binman: rename spl and u-boot nodes

2023-09-19 Thread Marek Vasut 

On 9/19/23 11:28, Rasmus Villemoes wrote:

The hab signing script doc/imx/habv4/csf_examples/mx8m/csf.sh does

   fdtget -t x u-boot.dtb /binman/imx-boot/uboot offset

to figure out the offset of u-boot.itb inside flash.bin. That works
fine for imx8mm, imx8mn, imx8mq, but fails for imx8mp because in that
case 'uboot' is merely a label and not actually the node name.

Homogenize these cases and make imx8mp the same as the other imx8m*
variants. The binman type is explicitly given and no longer derived
from the node name, and the csf.sh script will work for all four SOCs.

Signed-off-by: Rasmus Villemoes 
---
  arch/arm/dts/imx8mp-u-boot.dtsi | 6 --
  1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/arch/arm/dts/imx8mp-u-boot.dtsi b/arch/arm/dts/imx8mp-u-boot.dtsi
index 200938a980..355af16cf3 100644
--- a/arch/arm/dts/imx8mp-u-boot.dtsi
+++ b/arch/arm/dts/imx8mp-u-boot.dtsi
@@ -171,14 +171,16 @@
filename = "flash.bin";
pad-byte = <0x00>;
  
-		spl: blob-ext@1 {

+   spl {
filename = "spl.bin";
offset = <0x0>;
+   type = "blob-ext";
};
  
-		uboot: blob-ext@2 {

+   binman_uboot: uboot {
filename = "u-boot.itb";
offset = <0x58000>;
+   type = "blob-ext";
};
};
  };


Reviewed-by: Marek Vasut

[PATCH] imx8mp: binman: rename spl and u-boot nodes

2023-09-19 Thread Rasmus Villemoes 
The hab signing script doc/imx/habv4/csf_examples/mx8m/csf.sh does

  fdtget -t x u-boot.dtb /binman/imx-boot/uboot offset

to figure out the offset of u-boot.itb inside flash.bin. That works
fine for imx8mm, imx8mn, imx8mq, but fails for imx8mp because in that
case 'uboot' is merely a label and not actually the node name.

Homogenize these cases and make imx8mp the same as the other imx8m*
variants. The binman type is explicitly given and no longer derived
from the node name, and the csf.sh script will work for all four SOCs.

Signed-off-by: Rasmus Villemoes 
---
 arch/arm/dts/imx8mp-u-boot.dtsi | 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/arch/arm/dts/imx8mp-u-boot.dtsi b/arch/arm/dts/imx8mp-u-boot.dtsi
index 200938a980..355af16cf3 100644
--- a/arch/arm/dts/imx8mp-u-boot.dtsi
+++ b/arch/arm/dts/imx8mp-u-boot.dtsi
@@ -171,14 +171,16 @@
filename = "flash.bin";
pad-byte = <0x00>;
 
-   spl: blob-ext@1 {
+   spl {
filename = "spl.bin";
offset = <0x0>;
+   type = "blob-ext";
};
 
-   uboot: blob-ext@2 {
+   binman_uboot: uboot {
filename = "u-boot.itb";
offset = <0x58000>;
+   type = "blob-ext";
};
};
 };
-- 
2.37.2