| Message ID | 20200824211948.12852-5-rentao.bupt@gmail.com |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | ARM: dts: aspeed: Add Facebook Wedge400 (AST2500) BMC | expand |
On Mon, Aug 24, 2020 at 02:19:47PM -0700, rentao.bupt@gmail.com wrote: > From: Tao Ren <rentao.bupt@gmail.com> > > Set 64Mb FMC flash layout in Minipack device tree explicitly because the > flash layout was removed from "ast2500-facebook-netbmc-common.dtsi". > > Please note "data0" partition' size is updated to 4MB to be consistent > with other Facebook OpenBMC platforms. > > Signed-off-by: Tao Ren <rentao.bupt@gmail.com> > --- > .../boot/dts/aspeed-bmc-facebook-minipack.dts | 47 ++++++++++++++++++- > 1 file changed, 45 insertions(+), 2 deletions(-) > Reviewed-by: Patrick Williams <patrick@stwcx.xyz>
On Tue, Aug 25, 2020 at 09:18:08AM -0500, Patrick Williams wrote: > On Mon, Aug 24, 2020 at 02:19:47PM -0700, rentao.bupt@gmail.com wrote: > > From: Tao Ren <rentao.bupt@gmail.com> > > > > Set 64Mb FMC flash layout in Minipack device tree explicitly because the > > flash layout was removed from "ast2500-facebook-netbmc-common.dtsi". > > > > Please note "data0" partition' size is updated to 4MB to be consistent > > with other Facebook OpenBMC platforms. > > > > Signed-off-by: Tao Ren <rentao.bupt@gmail.com> > > --- > > .../boot/dts/aspeed-bmc-facebook-minipack.dts | 47 ++++++++++++++++++- > > 1 file changed, 45 insertions(+), 2 deletions(-) > > > > Reviewed-by: Patrick Williams <patrick@stwcx.xyz> Thanks for the review, Patrick. Cheers, Tao
diff --git a/arch/arm/boot/dts/aspeed-bmc-facebook-minipack.dts b/arch/arm/boot/dts/aspeed-bmc-facebook-minipack.dts index 88ce4ff9f47e..c34741dbd268 100644 --- a/arch/arm/boot/dts/aspeed-bmc-facebook-minipack.dts +++ b/arch/arm/boot/dts/aspeed-bmc-facebook-minipack.dts @@ -88,17 +88,60 @@ */ &fmc_flash0 { partitions { - data0@1c00000 { - reg = <0x1c00000 0x2400000>; + compatible = "fixed-partitions"; + #address-cells = <1>; + #size-cells = <1>; + + /* + * u-boot partition: 384KB. + */ + u-boot@0 { + reg = <0x0 0x60000>; + label = "u-boot"; }; + + /* + * u-boot environment variables: 128KB. + */ + u-boot-env@60000 { + reg = <0x60000 0x20000>; + label = "env"; + }; + + /* + * FIT image: 59.5 MB. + */ + fit@80000 { + reg = <0x80000 0x3b80000>; + label = "fit"; + }; + + /* + * "data0" partition (4MB) is reserved for persistent + * data store. + */ + data0@3800000 { + reg = <0x3c00000 0x400000>; + label = "data0"; + }; + + /* + * "flash0" partition (covering the entire flash) is + * explicitly created to avoid breaking legacy applications. + */ flash0@0 { reg = <0x0 0x4000000>; + label = "flash0"; }; }; }; &fmc_flash1 { partitions { + compatible = "fixed-partitions"; + #address-cells = <1>; + #size-cells = <1>; + flash1@0 { reg = <0x0 0x4000000>; };