2021-03-08 17:13:12

by Anton Kachalov

Subject: [PATCH v2] ARM: dts: nuvoton: Fix flash layout

From: "Anton D. Kachalov" <[email protected]>

This change follows OpenBMC partitions' naming layout.

Signed-off-by: Anton D. Kachalov <[email protected]>
---
arch/arm/boot/dts/nuvoton-npcm750-evb.dts | 38 +++++++----------------
1 file changed, 11 insertions(+), 27 deletions(-)

diff --git a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
index 9f13d08f5804..55c5a89592d7 100644
--- a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
+++ b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
@@ -78,8 +78,8 @@ partitions@80000000 {
compatible = "fixed-partitions";
#address-cells = <1>;
#size-cells = <1>;
- bbuboot1@0 {
- label = "bb-uboot-1";
+ u-boot@0 {
+ label = "u-boot";
reg = <0x0000000 0x80000>;
read-only;
};
@@ -88,38 +88,22 @@ bbuboot2@80000 {
reg = <0x0080000 0x80000>;
read-only;
};
- envparam@100000 {
- label = "env-param";
+ u-boot-env@100000 {
+ label = "u-boot-env";
reg = <0x0100000 0x40000>;
read-only;
};
- spare@140000 {
- label = "spare";
- reg = <0x0140000 0xC0000>;
- };
kernel@200000 {
label = "kernel";
- reg = <0x0200000 0x400000>;
- };
- rootfs@600000 {
- label = "rootfs";
- reg = <0x0600000 0x700000>;
- };
- spare1@D00000 {
- label = "spare1";
- reg = <0x0D00000 0x200000>;
- };
- spare2@0F00000 {
- label = "spare2";
- reg = <0x0F00000 0x200000>;
+ reg = <0x0200000 0x580000>;
};
- spare3@1100000 {
- label = "spare3";
- reg = <0x1100000 0x200000>;
+ rofs@780000 {
+ label = "rofs";
+ reg = <0x0780000 0x1680000>;
};
- spare4@1300000 {
- label = "spare4";
- reg = <0x1300000 0x0>;
+ rwfs@1e00000 {
+ label = "rwfs";
+ reg = <0x1e00000 0x200000>;
};
};
};
--
2.30.1.766.gb4fecdf3b7-goog

2021-03-25 01:27:34

by Anton Kachalov

Subject: Re: [PATCH v2] ARM: dts: nuvoton: Fix flash layout

Gently ping

On Mon, 8 Mar 2021 at 18:11, <[email protected]> wrote:
>
> From: "Anton D. Kachalov" <[email protected]>
>
> This change follows OpenBMC partitions' naming layout.
>
> Signed-off-by: Anton D. Kachalov <[email protected]>
> ---
> arch/arm/boot/dts/nuvoton-npcm750-evb.dts | 38 +++++++----------------
> 1 file changed, 11 insertions(+), 27 deletions(-)
>
> diff --git a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> index 9f13d08f5804..55c5a89592d7 100644
> --- a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> +++ b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> @@ -78,8 +78,8 @@ partitions@80000000 {
> compatible = "fixed-partitions";
> #address-cells = <1>;
> #size-cells = <1>;
> - bbuboot1@0 {
> - label = "bb-uboot-1";
> + u-boot@0 {
> + label = "u-boot";
> reg = <0x0000000 0x80000>;
> read-only;
> };
> @@ -88,38 +88,22 @@ bbuboot2@80000 {
> reg = <0x0080000 0x80000>;
> read-only;
> };
> - envparam@100000 {
> - label = "env-param";
> + u-boot-env@100000 {
> + label = "u-boot-env";
> reg = <0x0100000 0x40000>;
> read-only;
> };
> - spare@140000 {
> - label = "spare";
> - reg = <0x0140000 0xC0000>;
> - };
> kernel@200000 {
> label = "kernel";
> - reg = <0x0200000 0x400000>;
> - };
> - rootfs@600000 {
> - label = "rootfs";
> - reg = <0x0600000 0x700000>;
> - };
> - spare1@D00000 {
> - label = "spare1";
> - reg = <0x0D00000 0x200000>;
> - };
> - spare2@0F00000 {
> - label = "spare2";
> - reg = <0x0F00000 0x200000>;
> + reg = <0x0200000 0x580000>;
> };
> - spare3@1100000 {
> - label = "spare3";
> - reg = <0x1100000 0x200000>;
> + rofs@780000 {
> + label = "rofs";
> + reg = <0x0780000 0x1680000>;
> };
> - spare4@1300000 {
> - label = "spare4";
> - reg = <0x1300000 0x0>;
> + rwfs@1e00000 {
> + label = "rwfs";
> + reg = <0x1e00000 0x200000>;
> };
> };
> };
> --
> 2.30.1.766.gb4fecdf3b7-goog
>

2021-03-31 08:07:01

by Joel Stanley

Subject: Re: [PATCH v2] ARM: dts: nuvoton: Fix flash layout

Hi Anton,

On Thu, 25 Mar 2021 at 01:28, Anton Kachalov <[email protected]> wrote:
>
> Gently ping
>
> On Mon, 8 Mar 2021 at 18:11, <[email protected]> wrote:
> >
> > From: "Anton D. Kachalov" <[email protected]>
> >
> > This change follows OpenBMC partitions' naming layout.
> >
> > Signed-off-by: Anton D. Kachalov <[email protected]>

I believe you discussed what approach to take and this was agreed
upon. Can I get an ack from some other nuvoton people before I apply?

Cheers,

Joel

> > ---
> > arch/arm/boot/dts/nuvoton-npcm750-evb.dts | 38 +++++++----------------
> > 1 file changed, 11 insertions(+), 27 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> > index 9f13d08f5804..55c5a89592d7 100644
> > --- a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> > +++ b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> > @@ -78,8 +78,8 @@ partitions@80000000 {
> > compatible = "fixed-partitions";
> > #address-cells = <1>;
> > #size-cells = <1>;
> > - bbuboot1@0 {
> > - label = "bb-uboot-1";
> > + u-boot@0 {
> > + label = "u-boot";
> > reg = <0x0000000 0x80000>;
> > read-only;
> > };
> > @@ -88,38 +88,22 @@ bbuboot2@80000 {
> > reg = <0x0080000 0x80000>;
> > read-only;
> > };
> > - envparam@100000 {
> > - label = "env-param";
> > + u-boot-env@100000 {
> > + label = "u-boot-env";
> > reg = <0x0100000 0x40000>;
> > read-only;
> > };
> > - spare@140000 {
> > - label = "spare";
> > - reg = <0x0140000 0xC0000>;
> > - };
> > kernel@200000 {
> > label = "kernel";
> > - reg = <0x0200000 0x400000>;
> > - };
> > - rootfs@600000 {
> > - label = "rootfs";
> > - reg = <0x0600000 0x700000>;
> > - };
> > - spare1@D00000 {
> > - label = "spare1";
> > - reg = <0x0D00000 0x200000>;
> > - };
> > - spare2@0F00000 {
> > - label = "spare2";
> > - reg = <0x0F00000 0x200000>;
> > + reg = <0x0200000 0x580000>;
> > };
> > - spare3@1100000 {
> > - label = "spare3";
> > - reg = <0x1100000 0x200000>;
> > + rofs@780000 {
> > + label = "rofs";
> > + reg = <0x0780000 0x1680000>;
> > };
> > - spare4@1300000 {
> > - label = "spare4";
> > - reg = <0x1300000 0x0>;
> > + rwfs@1e00000 {
> > + label = "rwfs";
> > + reg = <0x1e00000 0x200000>;
> > };
> > };
> > };
> > --
> > 2.30.1.766.gb4fecdf3b7-goog
> >