Re: [PATCH v4 4/6] dt: fwu: developerbox: enable fwu banks and mdata regions

On 3/27/23 23:16, jassisinghbrar@gmail.com wrote:

From: Jassi Brar jaswinder.singh@linaro.org

Specify Bank-0/1 and fwu metadata mtd regions.

Signed-off-by: Jassi Brar jaswinder.singh@linaro.org

.../synquacer-sc2a11-developerbox-u-boot.dtsi | 49 +++++++++++++++++-- 1 file changed, 46 insertions(+), 3 deletions(-)

diff --git a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi index 9f9837b33b..9957646a46 100644 --- a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi +++ b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi @@ -21,7 +21,7 @@ #size-cells = <0>; status = "okay";

• flash@0 {
  

• flash0: flash@0 {
  #address-cells = <1>;
  #size-cells = <1>;
  compatible = "jedec,spi-nor";
  

@@ -74,8 +74,24 @@ };

		partition@500000 {

• 			label = "Ex-OPTEE";
  

• 			reg = <0x500000 0x200000>;
  

• 			label = "MDATA-Pri";
  

• 			reg = <0x500000 0x1000>;
  

• 		};
  

• 		partition@530000 {
  

• 			label = "MDATA-Sec";
  

• 			reg = <0x530000 0x1000>;
  

• 		};
  

• 		/* FWU Multi bank update partitions */
  

• 		partition@600000 {
  

• 			label = "FIP-Bank0";
  

• 			reg = <0x600000 0x400000>;
  

• 		};
  

• 		partition@a00000 {
  

• 			label = "FIP-Bank1";
  

• 			reg = <0xa00000 0x400000>;
  	};
  };
  

  };

@@ -102,6 +118,33 @@ optee { status = "okay"; };

• fwu-mdata {
  

• 	compatible = "u-boot,fwu-mdata-mtd";
  

• 	fwu-mdata-store = <&flash0>;
  

• 	mdata-parts = "MDATA-Pri", "MDATA-Sec";
  

As I discussed this with Ilias today. This should be approved or this DT won't pass yaml checking for SR certification. That's why this should get to schema to be able to widely use.

Thanks, Michal