From: "Marek Behún" <marek.behun@nic.cz>
To: Masami Hiramatsu <masami.hiramatsu@linaro.org>
Cc: Simon Glass <sjg@chromium.org>, Tom Rini <trini@konsulko.com>,
Heinrich Schuchardt <xypron.glpk@gmx.de>,
Jassi Brar <jaswinder.singh@linaro.org>,
Ilias Apalodimas <ilias.apalodimas@linaro.org>,
Masahisa Kojima <masahisa.kojima@linaro.org>,
Takahiro Akashi <takahiro.akashi@linaro.org>,
U-Boot Mailing List <u-boot@lists.denx.de>
Subject: Re: [PATCH 3/6] dts: synquacer: Add partition information to the spi-nor
Date: Fri, 9 Jul 2021 14:06:15 +0200 [thread overview]
Message-ID: <20210709140615.2c3a0993@dellmb> (raw)
In-Reply-To: <CAA93ih3gJ=7UF9-qHJjc0q_1Ea3+R4PwH_-XBmuoEXxBe+y1sg@mail.gmail.com>
On Fri, 9 Jul 2021 20:27:18 +0900
Masami Hiramatsu <masami.hiramatsu@linaro.org> wrote:
> Hi Marek,
>
> 2021年7月9日(金) 19:54 Marek Behún <marek.behun@nic.cz>:
> >
> > On Fri, 9 Jul 2021 19:50:27 +0900
> > Masami Hiramatsu <masami.hiramatsu@linaro.org> wrote:
> >
> > > Add partition information to the spi-nor flash.
> > > This is required for accessing NOR flash via mtdparts.
> > >
> > > Signed-off-by: Masami Hiramatsu <masami.hiramatsu@linaro.org>
> > > ---
> > > .../dts/synquacer-sc2a11-developerbox-u-boot.dtsi | 42
> > > ++++++++++++++++++++ 1 file changed, 42 insertions(+)
> > >
> > > diff --git
> > > a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi
> > > b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi index
> > > 2f13a42235..245ebcda01 100644 ---
> > > a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi +++
> > > b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi @@ -31,6
> > > +31,48 @@ spi-max-frequency = <31250000>; spi-rx-bus-width =
> > > <0x1>; spi-tx-bus-width = <0x1>;
> > > +
> > > + partitions {
> > > + compatible = "fixed-partitions";
> > > + #address-cells = <1>;
> > > + #size-cells = <1>;
> > > +
> > > + partition@0 {
> > > + label = "BootStrap-BL1";
> > > + reg = <0x0 0x70000>;
> > > + read-only;
> > > + };
> > > + partition@70000 {
> > > + label = "Flash-Writer";
> > > + reg = <0x70000 0x90000>;
> > > + read-only;
> > > + };
> > > + partition@100000 {
> > > + label = "SCP-BL2";
> > > + reg = <0x100000 0x80000>;
> > > + read-only;
> > > + };
> > > + partition@180000 {
> > > + label = "FIP-TFA";
> > > + reg = <0x180000 0x78000>;
> > > + };
> > > + partition@1f8000 {
> > > + label = "Stage2Tables";
> > > + reg = <0x1f8000 0x8000>;
> > > + };
> > > + partition@200000 {
> > > + label = "U-Boot";
> > > + reg = <0x200000 0x100000>;
> > > + };
> > > + partition@300000 {
> > > + label = "UBoot-Env";
> > > + reg = <0x300000 0x100000>;
> > > + };
> > > + partition@500000 {
> > > + label = "Ex-OPTEE";
> > > + reg = <0x500000 0x200000>;
> > > + };
> > > + };
> > > };
> > > };
> > >
> > >
> >
> > Just a style hint: the individual partition nodes should be IMO
> > separated by an additional newline character, i.e.:
>
> Ah, OK.
>
> >
> > partition@0 {
> > label = "BootStrap-BL1";
> > reg = <0x0 0x70000>;
> > read-only;
> > };
> >
> > partition@70000 {
> > label = "Flash-Writer";
> > reg = <0x70000 0x90000>;
> > read-only;
> > };
> >
> > partition@100000 {
> > label = "SCP-BL2";
> > reg = <0x100000 0x80000>;
> > read-only;
> > };
> >
> > Other than that:
> >
> > Reviewed-by: Marek Behún <marek.behun@nic.cz>
> >
> > Btw, did you test this? Does this work correctly with mtd command?
>
> Yes, I've tested that, and mtd list works. But it seems "mtd erase"
> commands don't work well.
> See the log below;
>
> -------
> => log level 7
> => mtd list
> List of MTD devices:
> * mx66u51235f
> - device: spi-flash@0
> - parent: spi@54800000
> - driver: jedec_spi_nor
> - path: /spi@54800000/spi-flash@0
> - type: NOR flash
> - block size: 0x1000 bytes
> - min I/O: 0x1 bytes
> - 0x000000000000-0x000004000000 : "mx66u51235f"
> - 0x000000000000-0x000000070000 : "BootStrap-BL1"
> - 0x000000070000-0x000000100000 : "Flash-Writer"
> - 0x000000100000-0x000000180000 : "SCP-BL2"
> - 0x000000180000-0x0000001f8000 : "FIP-TFA"
> - 0x0000001f8000-0x000000200000 : "Stage2Tables"
> - 0x000000200000-0x000000400000 : "EDK2"
> - 0x000000400000-0x000000500000 : "EDK2-Env"
> - 0x000000500000-0x000000700000 : "Ex-OPTEE"
> - 0x000000700000-0x000000800000 : "UBoot-Env"
> - 0x000000800000-0x000000900000 : "U-Boot"
> => mtd erase Ex-OPTEE
> Erasing 0x00000000 ... 0x001fffff (512 eraseblock(s))
> jedec_spi_nor spi-flash@0: at 0x500000, len 4096
> => mtd dump E
> EDK2 EDK2-Env Ex-OPTEE
> => mtd dump Ex-OPTEE 0 10
> Reading 16 byte(s) at offset 0x00000000
> jedec_spi_nor spi-flash@0: from 0x00500000, len 16
>
> Dump 16 data bytes from 0x0:
> 0x00000000: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
> => mtd dump Ex-OPTEE 1000 10
> Reading 16 byte(s) at offset 0x00001000
> jedec_spi_nor spi-flash@0: from 0x00501000, len 16
>
> Dump 16 data bytes from 0x1000:
> 0x00001000: 00 00 00 14 1f 20 03 d5 1f 20 03 d5 1f 20 03 d5
> =>
> -------
>
> So it seems first 4k block is cleared, but other blocks are not
> erased. I think that is not a devicetree issue, but mtd subsystem
> issue.
Can reproduce, going to look into this.
Marek
next prev parent reply other threads:[~2021-07-09 12:06 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-09 10:49 [PATCH 0/6] arm64: synquacer: DeveloperBox updates Masami Hiramatsu
2021-07-09 10:50 ` [PATCH 1/6] board: synquacer: Initialize SCBM SMMU at board_init() Masami Hiramatsu
2021-07-09 10:50 ` [PATCH 2/6] configs: synquacer: Make U-Boot binary position independent Masami Hiramatsu
2021-07-09 10:52 ` Marek Behún
2021-07-10 10:12 ` Masami Hiramatsu
2021-07-09 10:50 ` [PATCH 3/6] dts: synquacer: Add partition information to the spi-nor Masami Hiramatsu
2021-07-09 10:54 ` Marek Behún
2021-07-09 11:27 ` Masami Hiramatsu
2021-07-09 12:06 ` Marek Behún [this message]
2021-07-09 10:50 ` [PATCH 4/6] configs: synquacer: Remove mtdparts settings and update DFU setting Masami Hiramatsu
2021-07-09 10:50 ` [PATCH 5/6] configs: synquacer: Drop Ext2/4 support by default Masami Hiramatsu
2021-07-09 10:50 ` [PATCH 6/6] configs: synquacer: Enable UEFI secure boot Masami Hiramatsu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210709140615.2c3a0993@dellmb \
--to=marek.behun@nic.cz \
--cc=ilias.apalodimas@linaro.org \
--cc=jaswinder.singh@linaro.org \
--cc=masahisa.kojima@linaro.org \
--cc=masami.hiramatsu@linaro.org \
--cc=sjg@chromium.org \
--cc=takahiro.akashi@linaro.org \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
--cc=xypron.glpk@gmx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox