| Message ID | 20230109010702.578523-1-jaswinder.singh@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | FWU: Add support for mtd backed feature on DeveloperBox | expand |
On 1/9/23 02:07, Jassi Brar wrote: > Specify Bank-0/1 and fwu metadata mtd regions. > > Signed-off-by: Jassi Brar <jaswinder.singh@linaro.org> > Acked-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> > --- > .../synquacer-sc2a11-developerbox-u-boot.dtsi | 22 ++++++++++++++++++- > 1 file changed, 21 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi > index 7a56116d6f..62eee0aaf0 100644 > --- a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi > +++ b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi > @@ -23,7 +23,7 @@ > active_clk_edges; > chipselect_num = <1>; > > - spi-flash@0 { > + spi_flash: spi-flash@0 { > #address-cells = <1>; > #size-cells = <1>; > compatible = "jedec,spi-nor"; > @@ -36,6 +36,7 @@ > compatible = "fixed-partitions"; > #address-cells = <1>; > #size-cells = <1>; > + uuid = "17e86d77-41f9-4fd7-87ec-a55df9842de5"; I looked at dt-schema and also to linus tree and linux-next and I can't see any record for uuid in bindings. That's why this is something what it is not documented that's why you shouldn't really use it. > > partition@0 { > label = "BootStrap-BL1"; > @@ -79,6 +80,19 @@ > label = "Ex-OPTEE"; > reg = <0x500000 0x200000>; > }; > + > + /* FWU Multi bank update partitions */ > + partition@600000 { > + label = "FIP-Bank0"; > + reg = <0x600000 0x400000>; > + uuid = "5a66a702-99fd-4fef-a392-c26e261a2828"; > + }; > + > + partition@a00000 { > + label = "FIP-Bank1"; > + reg = <0xa00000 0x400000>; > + uuid = "a8f868a1-6e5c-4757-878d-ce63375ef2c0"; > + }; > }; > }; > }; > @@ -104,6 +118,12 @@ > optee { > status = "okay"; > }; > + > + fwu-mdata { > + compatible = "u-boot,fwu-mdata-mtd"; > + fwu-mdata-store = <&spi_flash>; > + mdata-offsets = <0x500000 0x530000>; > + }; SR IR 2.0 has to pass binding check and I can't see any acked binding for it. It means please get this approve first. Thanks, Michal
diff --git a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi index 7a56116d6f..62eee0aaf0 100644 --- a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi +++ b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi @@ -23,7 +23,7 @@ active_clk_edges; chipselect_num = <1>; - spi-flash@0 { + spi_flash: spi-flash@0 { #address-cells = <1>; #size-cells = <1>; compatible = "jedec,spi-nor"; @@ -36,6 +36,7 @@ compatible = "fixed-partitions"; #address-cells = <1>; #size-cells = <1>; + uuid = "17e86d77-41f9-4fd7-87ec-a55df9842de5"; partition@0 { label = "BootStrap-BL1"; @@ -79,6 +80,19 @@ label = "Ex-OPTEE"; reg = <0x500000 0x200000>; }; + + /* FWU Multi bank update partitions */ + partition@600000 { + label = "FIP-Bank0"; + reg = <0x600000 0x400000>; + uuid = "5a66a702-99fd-4fef-a392-c26e261a2828"; + }; + + partition@a00000 { + label = "FIP-Bank1"; + reg = <0xa00000 0x400000>; + uuid = "a8f868a1-6e5c-4757-878d-ce63375ef2c0"; + }; }; }; }; @@ -104,6 +118,12 @@ optee { status = "okay"; }; + + fwu-mdata { + compatible = "u-boot,fwu-mdata-mtd"; + fwu-mdata-store = <&spi_flash>; + mdata-offsets = <0x500000 0x530000>; + }; }; };