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

[Michal Simek <michal.simek@amd.com>]

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