From: Chai Wen <chaiw.fnst@cn.fujitsu.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] ARM: shmobile: koelsch: Fix flash partition label and size in device tree
Date: Thu, 30 Oct 2014 10:50:49 +0000 [thread overview]
Message-ID: <54521809.4000208@cn.fujitsu.com> (raw)
In-Reply-To: <CAMuHMdUAYJfLtMwzwOvsqaoDtXPW=q2yQLUH9+jQDB+A2JHX9Q@mail.gmail.com>
On 10/30/2014 06:23 PM, Geert Uytterhoeven wrote:
> On Thu, Oct 30, 2014 at 6:54 AM, Simon Horman
> <horms+renesas@verge.net.au> wrote:
>> From: Hisashi Nakamura <hisashi.nakamura.ak@renesas.com>
>>
>> The size and the name of flash partition are consistent
>> across use the R-Car Gen2 series.
>>
>> This patch makes the koelsch board consistent with the lager board.
>
> Does this match the actual data in the FLASH?
>
> On my board, the SPI boot loader is at 0x00000000.
> There's also a small unidentified chunk at 0x00040000:
>
> 00040000 00 40 30 e6 5c 3a 03 00 ff ff ff ff ff ff ff ff
> |.@0.\:..........|
>
> u-Boot seems to be at 0x00080000.
> u-Boot environment variables are saved at 0x000c0000.
> This does match the old "bootenv" partition.
>
> After that it's all-zeroes until 0x00100000, where I did my SPI tests,
> based on the old layout.
>
> Do I have a non-standard Koelsch FLASH layout on my board?
FYI, I have a koelsch board, and the layout out the flash is like:
0x0 0x10000 0x50000
|-- loader.bin --|-- uboot.bin --|-- something else --|
0 64K 320K 16M
But, I have no idea whether this is a standard layout or not.
thanks
chai wen
>
>> --- a/arch/arm/boot/dts/r8a7791-koelsch.dts
>> +++ b/arch/arm/boot/dts/r8a7791-koelsch.dts
>> @@ -390,17 +390,17 @@
>>
>> partition@0 {
>> label = "loader";
>> - reg = <0x00000000 0x00080000>;
>> + reg = <0x00000000 0x00040000>;
>> read-only;
>> };
>> - partition@80000 {
>> - label = "bootenv";
>> - reg = <0x00080000 0x00080000>;
>> + partition@40000 {
>> + label = "user";
>> + reg = <0x00040000 0x00400000>;
>> read-only;
>> };
>> - partition@100000 {
>> - label = "data";
>> - reg = <0x00100000 0x03f00000>;
>> + partition@440000 {
>> + label = "flash";
>> + reg = <0x00440000 0x03bc0000>;
>> };
>> };
>> };
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> .
>
--
Regards
Chai Wen
WARNING: multiple messages have this Message-ID (diff)
From: chaiw.fnst@cn.fujitsu.com (Chai Wen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: shmobile: koelsch: Fix flash partition label and size in device tree
Date: Thu, 30 Oct 2014 18:50:49 +0800 [thread overview]
Message-ID: <54521809.4000208@cn.fujitsu.com> (raw)
In-Reply-To: <CAMuHMdUAYJfLtMwzwOvsqaoDtXPW=q2yQLUH9+jQDB+A2JHX9Q@mail.gmail.com>
On 10/30/2014 06:23 PM, Geert Uytterhoeven wrote:
> On Thu, Oct 30, 2014 at 6:54 AM, Simon Horman
> <horms+renesas@verge.net.au> wrote:
>> From: Hisashi Nakamura <hisashi.nakamura.ak@renesas.com>
>>
>> The size and the name of flash partition are consistent
>> across use the R-Car Gen2 series.
>>
>> This patch makes the koelsch board consistent with the lager board.
>
> Does this match the actual data in the FLASH?
>
> On my board, the SPI boot loader is at 0x00000000.
> There's also a small unidentified chunk at 0x00040000:
>
> 00040000 00 40 30 e6 5c 3a 03 00 ff ff ff ff ff ff ff ff
> |. at 0.\:..........|
>
> u-Boot seems to be at 0x00080000.
> u-Boot environment variables are saved at 0x000c0000.
> This does match the old "bootenv" partition.
>
> After that it's all-zeroes until 0x00100000, where I did my SPI tests,
> based on the old layout.
>
> Do I have a non-standard Koelsch FLASH layout on my board?
FYI, I have a koelsch board, and the layout out the flash is like:
0x0 0x10000 0x50000
|-- loader.bin --|-- uboot.bin --|-- something else --|
0 64K 320K 16M
But, I have no idea whether this is a standard layout or not.
thanks
chai wen
>
>> --- a/arch/arm/boot/dts/r8a7791-koelsch.dts
>> +++ b/arch/arm/boot/dts/r8a7791-koelsch.dts
>> @@ -390,17 +390,17 @@
>>
>> partition at 0 {
>> label = "loader";
>> - reg = <0x00000000 0x00080000>;
>> + reg = <0x00000000 0x00040000>;
>> read-only;
>> };
>> - partition at 80000 {
>> - label = "bootenv";
>> - reg = <0x00080000 0x00080000>;
>> + partition at 40000 {
>> + label = "user";
>> + reg = <0x00040000 0x00400000>;
>> read-only;
>> };
>> - partition at 100000 {
>> - label = "data";
>> - reg = <0x00100000 0x03f00000>;
>> + partition at 440000 {
>> + label = "flash";
>> + reg = <0x00440000 0x03bc0000>;
>> };
>> };
>> };
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> .
>
--
Regards
Chai Wen
next prev parent reply other threads:[~2014-10-30 10:50 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-30 5:54 [PATCH] ARM: shmobile: koelsch: Fix flash partition label and size in device tree Simon Horman
2014-10-30 5:54 ` Simon Horman
2014-10-30 10:23 ` Geert Uytterhoeven
2014-10-30 10:23 ` Geert Uytterhoeven
2014-10-30 10:50 ` Chai Wen [this message]
2014-10-30 10:50 ` Chai Wen
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=54521809.4000208@cn.fujitsu.com \
--to=chaiw.fnst@cn.fujitsu.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.