* [U-Boot] [PATCH] armv8: ls1043a/ls1046aqds: fix the offsets of MTD partitions on Nor flash
@ 2017-03-14 5:52 Wenbin song
2017-03-15 17:13 ` york sun
0 siblings, 1 reply; 4+ messages in thread
From: Wenbin song @ 2017-03-14 5:52 UTC (permalink / raw)
To: u-boot
Fix the offsets of MTD partitions on Nor flash on ls1043ardb, ls1043aqds
and ls1046aqds boards.
Signed-off-by: Wenbin Song <wenbin.song@nxp.com>
---
include/configs/ls1043a_common.h | 7 ++++---
include/configs/ls1046aqds.h | 7 ++++---
2 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/include/configs/ls1043a_common.h b/include/configs/ls1043a_common.h
index c4b05e0..ca845b6 100644
--- a/include/configs/ls1043a_common.h
+++ b/include/configs/ls1043a_common.h
@@ -192,10 +192,11 @@
#else
#define MTDPARTS_DEFAULT "mtdparts=60000000.nor:1m(nor_bank0_rcw)," \
"1m(nor_bank0_uboot),1m(nor_bank0_uboot_env)," \
- "1m(nor_bank0_fman_uconde),40m(nor_bank0_fit)," \
- "1m(nor_bank4_rcw),1m(nor_bank4_uboot)," \
+ "1m(nor_bank0_fman_uconde),"\
+ "40m at 1100000(nor_bank0_fit)," \
+ "1m at 4000000(nor_bank4_rcw),1m(nor_bank4_uboot)," \
"1m(nor_bank4_uboot_env),1m(nor_bank4_fman_ucode)," \
- "40m(nor_bank4_fit);7e800000.flash:" \
+ "40m at 5100000(nor_bank4_fit);7e800000.flash:" \
"1m(nand_uboot),1m(nand_uboot_env)," \
"20m(nand_fit);spi0.0:1m(uboot)," \
"5m(kernel),1m(dtb),9m(file_system)"
diff --git a/include/configs/ls1046aqds.h b/include/configs/ls1046aqds.h
index 4b3b21e..e045249 100644
--- a/include/configs/ls1046aqds.h
+++ b/include/configs/ls1046aqds.h
@@ -487,10 +487,11 @@ unsigned long get_board_ddr_clk(void);
#else
#define MTDPARTS_DEFAULT "mtdparts=60000000.nor:1m(nor_bank0_rcw)," \
"1m(nor_bank0_uboot),1m(nor_bank0_uboot_env)," \
- "1m(nor_bank0_fman_uconde),40m(nor_bank0_fit)," \
- "1m(nor_bank4_rcw),1m(nor_bank4_uboot)," \
+ "1m(nor_bank0_fman_uconde),"\
+ "40m at 1100000(nor_bank0_fit)," \
+ "1m at 4000000(nor_bank4_rcw),1m(nor_bank4_uboot)," \
"1m(nor_bank4_uboot_env),1m(nor_bank4_fman_ucode)," \
- "40m(nor_bank4_fit);7e800000.flash:" \
+ "40m at 5100000(nor_bank4_fit);7e800000.flash:" \
"4m(nand_uboot),36m(nand_kernel)," \
"472m(nand_free);spi0.0:2m(uboot)," \
"14m(free)"
--
2.1.0.27.g96db324
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] armv8: ls1043a/ls1046aqds: fix the offsets of MTD partitions on Nor flash
2017-03-14 5:52 [U-Boot] [PATCH] armv8: ls1043a/ls1046aqds: fix the offsets of MTD partitions on Nor flash Wenbin song
@ 2017-03-15 17:13 ` york sun
2017-03-17 1:05 ` Wenbin Song
0 siblings, 1 reply; 4+ messages in thread
From: york sun @ 2017-03-15 17:13 UTC (permalink / raw)
To: u-boot
On 03/13/2017 11:06 PM, Wenbin song wrote:
> Fix the offsets of MTD partitions on Nor flash on ls1043ardb, ls1043aqds
> and ls1046aqds boards.
>
> Signed-off-by: Wenbin Song <wenbin.song@nxp.com>
> ---
> include/configs/ls1043a_common.h | 7 ++++---
> include/configs/ls1046aqds.h | 7 ++++---
> 2 files changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/include/configs/ls1043a_common.h b/include/configs/ls1043a_common.h
> index c4b05e0..ca845b6 100644
> --- a/include/configs/ls1043a_common.h
> +++ b/include/configs/ls1043a_common.h
> @@ -192,10 +192,11 @@
> #else
> #define MTDPARTS_DEFAULT "mtdparts=60000000.nor:1m(nor_bank0_rcw)," \
> "1m(nor_bank0_uboot),1m(nor_bank0_uboot_env)," \
> - "1m(nor_bank0_fman_uconde),40m(nor_bank0_fit)," \
> - "1m(nor_bank4_rcw),1m(nor_bank4_uboot)," \
> + "1m(nor_bank0_fman_uconde),"\
> + "40m at 1100000(nor_bank0_fit)," \
> + "1m at 4000000(nor_bank4_rcw),1m(nor_bank4_uboot)," \
> "1m(nor_bank4_uboot_env),1m(nor_bank4_fman_ucode)," \
> - "40m(nor_bank4_fit);7e800000.flash:" \
> + "40m at 5100000(nor_bank4_fit);7e800000.flash:" \
> "1m(nand_uboot),1m(nand_uboot_env)," \
> "20m(nand_fit);spi0.0:1m(uboot)," \
> "5m(kernel),1m(dtb),9m(file_system)"
> diff --git a/include/configs/ls1046aqds.h b/include/configs/ls1046aqds.h
> index 4b3b21e..e045249 100644
> --- a/include/configs/ls1046aqds.h
> +++ b/include/configs/ls1046aqds.h
> @@ -487,10 +487,11 @@ unsigned long get_board_ddr_clk(void);
> #else
> #define MTDPARTS_DEFAULT "mtdparts=60000000.nor:1m(nor_bank0_rcw)," \
> "1m(nor_bank0_uboot),1m(nor_bank0_uboot_env)," \
> - "1m(nor_bank0_fman_uconde),40m(nor_bank0_fit)," \
> - "1m(nor_bank4_rcw),1m(nor_bank4_uboot)," \
> + "1m(nor_bank0_fman_uconde),"\
> + "40m at 1100000(nor_bank0_fit)," \
> + "1m at 4000000(nor_bank4_rcw),1m(nor_bank4_uboot)," \
> "1m(nor_bank4_uboot_env),1m(nor_bank4_fman_ucode)," \
> - "40m(nor_bank4_fit);7e800000.flash:" \
> + "40m at 5100000(nor_bank4_fit);7e800000.flash:" \
> "4m(nand_uboot),36m(nand_kernel)," \
> "472m(nand_free);spi0.0:2m(uboot)," \
> "14m(free)"
>
Wenbin,
Maybe you can use more generic partitions for the NOR and put more
detail in comment. I believe the RCW, U-Boot, Env, Fman ucode, PPA are
at fixed locations. But the rest is up to user. We cannot assume the
layout is identical to SDK. Same goes to NAND partitions.
York
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] armv8: ls1043a/ls1046aqds: fix the offsets of MTD partitions on Nor flash
2017-03-15 17:13 ` york sun
@ 2017-03-17 1:05 ` Wenbin Song
2017-03-17 3:35 ` york sun
0 siblings, 1 reply; 4+ messages in thread
From: Wenbin Song @ 2017-03-17 1:05 UTC (permalink / raw)
To: u-boot
Hi york,
Do you mean it is not necessary to supply so much partitions for user, just supplying two partitions, one is reserved for firmware(including RCW, U-boot, Env, PPA, kernel.itb and so on), the other is the rest of flashes, if not, Could you give me more details?
Best Regards
Wenbin Song
-----Original Message-----
From: york sun
Sent: Thursday, March 16, 2017 1:14 AM
To: Wenbin Song <wenbin.song@nxp.com>; Mingkai Hu <mingkai.hu@nxp.com>; sjg at chromium.org; Z.Q. Hou <zhiqiang.hou@nxp.com>; M.H. Lian <minghuan.lian@nxp.com>; u-boot at lists.denx.de
Subject: Re: [PATCH] armv8: ls1043a/ls1046aqds: fix the offsets of MTD partitions on Nor flash
On 03/13/2017 11:06 PM, Wenbin song wrote:
> Fix the offsets of MTD partitions on Nor flash on ls1043ardb,
> ls1043aqds and ls1046aqds boards.
>
> Signed-off-by: Wenbin Song <wenbin.song@nxp.com>
> ---
> include/configs/ls1043a_common.h | 7 ++++---
> include/configs/ls1046aqds.h | 7 ++++---
> 2 files changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/include/configs/ls1043a_common.h
> b/include/configs/ls1043a_common.h
> index c4b05e0..ca845b6 100644
> --- a/include/configs/ls1043a_common.h
> +++ b/include/configs/ls1043a_common.h
> @@ -192,10 +192,11 @@
> #else
> #define MTDPARTS_DEFAULT "mtdparts=60000000.nor:1m(nor_bank0_rcw)," \
> "1m(nor_bank0_uboot),1m(nor_bank0_uboot_env)," \
> - "1m(nor_bank0_fman_uconde),40m(nor_bank0_fit)," \
> - "1m(nor_bank4_rcw),1m(nor_bank4_uboot)," \
> + "1m(nor_bank0_fman_uconde),"\
> + "40m at 1100000(nor_bank0_fit)," \
> + "1m at 4000000(nor_bank4_rcw),1m(nor_bank4_uboot)," \
> "1m(nor_bank4_uboot_env),1m(nor_bank4_fman_ucode)," \
> - "40m(nor_bank4_fit);7e800000.flash:" \
> + "40m at 5100000(nor_bank4_fit);7e800000.flash:" \
> "1m(nand_uboot),1m(nand_uboot_env)," \
> "20m(nand_fit);spi0.0:1m(uboot)," \
> "5m(kernel),1m(dtb),9m(file_system)"
> diff --git a/include/configs/ls1046aqds.h
> b/include/configs/ls1046aqds.h index 4b3b21e..e045249 100644
> --- a/include/configs/ls1046aqds.h
> +++ b/include/configs/ls1046aqds.h
> @@ -487,10 +487,11 @@ unsigned long get_board_ddr_clk(void); #else
> #define MTDPARTS_DEFAULT "mtdparts=60000000.nor:1m(nor_bank0_rcw)," \
> "1m(nor_bank0_uboot),1m(nor_bank0_uboot_env)," \
> - "1m(nor_bank0_fman_uconde),40m(nor_bank0_fit)," \
> - "1m(nor_bank4_rcw),1m(nor_bank4_uboot)," \
> + "1m(nor_bank0_fman_uconde),"\
> + "40m at 1100000(nor_bank0_fit)," \
> + "1m at 4000000(nor_bank4_rcw),1m(nor_bank4_uboot)," \
> "1m(nor_bank4_uboot_env),1m(nor_bank4_fman_ucode)," \
> - "40m(nor_bank4_fit);7e800000.flash:" \
> + "40m at 5100000(nor_bank4_fit);7e800000.flash:" \
> "4m(nand_uboot),36m(nand_kernel)," \
> "472m(nand_free);spi0.0:2m(uboot)," \
> "14m(free)"
>
Wenbin,
Maybe you can use more generic partitions for the NOR and put more detail in comment. I believe the RCW, U-Boot, Env, Fman ucode, PPA are at fixed locations. But the rest is up to user. We cannot assume the layout is identical to SDK. Same goes to NAND partitions.
York
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] armv8: ls1043a/ls1046aqds: fix the offsets of MTD partitions on Nor flash
2017-03-17 1:05 ` Wenbin Song
@ 2017-03-17 3:35 ` york sun
0 siblings, 0 replies; 4+ messages in thread
From: york sun @ 2017-03-17 3:35 UTC (permalink / raw)
To: u-boot
On 03/16/2017 06:05 PM, Wenbin Song wrote:
> Hi york,
>
> Do you mean it is not necessary to supply so much partitions for user, just supplying two partitions, one is reserved for firmware(including RCW, U-boot, Env, PPA, kernel.itb and so on), the other is the rest of flashes, if not, Could you give me more details?
Wenbin,
I think it depends on what you expect users to do with those partitions.
That's why I said you don't have use SDK layout.
If you don't expect users to modify (eg. RCW and/or U-Boot), you can
mark them as "ro". For some space, if you are not sure if the FIT image
will be there, you can skip it. That's the benefit of using offsets.
York
>
> Best Regards
> Wenbin Song
>
>
>
> -----Original Message-----
> From: york sun
> Sent: Thursday, March 16, 2017 1:14 AM
> To: Wenbin Song <wenbin.song@nxp.com>; Mingkai Hu <mingkai.hu@nxp.com>; sjg at chromium.org; Z.Q. Hou <zhiqiang.hou@nxp.com>; M.H. Lian <minghuan.lian@nxp.com>; u-boot at lists.denx.de
> Subject: Re: [PATCH] armv8: ls1043a/ls1046aqds: fix the offsets of MTD partitions on Nor flash
>
> On 03/13/2017 11:06 PM, Wenbin song wrote:
>> Fix the offsets of MTD partitions on Nor flash on ls1043ardb,
>> ls1043aqds and ls1046aqds boards.
>>
>> Signed-off-by: Wenbin Song <wenbin.song@nxp.com>
>> ---
>> include/configs/ls1043a_common.h | 7 ++++---
>> include/configs/ls1046aqds.h | 7 ++++---
>> 2 files changed, 8 insertions(+), 6 deletions(-)
>>
>> diff --git a/include/configs/ls1043a_common.h
>> b/include/configs/ls1043a_common.h
>> index c4b05e0..ca845b6 100644
>> --- a/include/configs/ls1043a_common.h
>> +++ b/include/configs/ls1043a_common.h
>> @@ -192,10 +192,11 @@
>> #else
>> #define MTDPARTS_DEFAULT "mtdparts=60000000.nor:1m(nor_bank0_rcw)," \
>> "1m(nor_bank0_uboot),1m(nor_bank0_uboot_env)," \
>> - "1m(nor_bank0_fman_uconde),40m(nor_bank0_fit)," \
>> - "1m(nor_bank4_rcw),1m(nor_bank4_uboot)," \
>> + "1m(nor_bank0_fman_uconde),"\
>> + "40m at 1100000(nor_bank0_fit)," \
>> + "1m at 4000000(nor_bank4_rcw),1m(nor_bank4_uboot)," \
>> "1m(nor_bank4_uboot_env),1m(nor_bank4_fman_ucode)," \
>> - "40m(nor_bank4_fit);7e800000.flash:" \
>> + "40m at 5100000(nor_bank4_fit);7e800000.flash:" \
>> "1m(nand_uboot),1m(nand_uboot_env)," \
>> "20m(nand_fit);spi0.0:1m(uboot)," \
>> "5m(kernel),1m(dtb),9m(file_system)"
>> diff --git a/include/configs/ls1046aqds.h
>> b/include/configs/ls1046aqds.h index 4b3b21e..e045249 100644
>> --- a/include/configs/ls1046aqds.h
>> +++ b/include/configs/ls1046aqds.h
>> @@ -487,10 +487,11 @@ unsigned long get_board_ddr_clk(void); #else
>> #define MTDPARTS_DEFAULT "mtdparts=60000000.nor:1m(nor_bank0_rcw)," \
>> "1m(nor_bank0_uboot),1m(nor_bank0_uboot_env)," \
>> - "1m(nor_bank0_fman_uconde),40m(nor_bank0_fit)," \
>> - "1m(nor_bank4_rcw),1m(nor_bank4_uboot)," \
>> + "1m(nor_bank0_fman_uconde),"\
>> + "40m at 1100000(nor_bank0_fit)," \
>> + "1m at 4000000(nor_bank4_rcw),1m(nor_bank4_uboot)," \
>> "1m(nor_bank4_uboot_env),1m(nor_bank4_fman_ucode)," \
>> - "40m(nor_bank4_fit);7e800000.flash:" \
>> + "40m at 5100000(nor_bank4_fit);7e800000.flash:" \
>> "4m(nand_uboot),36m(nand_kernel)," \
>> "472m(nand_free);spi0.0:2m(uboot)," \
>> "14m(free)"
>>
>
> Wenbin,
>
> Maybe you can use more generic partitions for the NOR and put more detail in comment. I believe the RCW, U-Boot, Env, Fman ucode, PPA are at fixed locations. But the rest is up to user. We cannot assume the layout is identical to SDK. Same goes to NAND partitions.
>
> York
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-03-17 3:35 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-14 5:52 [U-Boot] [PATCH] armv8: ls1043a/ls1046aqds: fix the offsets of MTD partitions on Nor flash Wenbin song
2017-03-15 17:13 ` york sun
2017-03-17 1:05 ` Wenbin Song
2017-03-17 3:35 ` york sun
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.