Skip to contentSkip to conversation

powerpc/p5040: enable NAND boot support

PATCH Superseded Shaohui Xie 5 participants 12 messages Jan 16, 2013 – Jan 18, 2013

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/3 powerpc/p5040: enable NAND boot support

---
 boards.cfg |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
boards.cfg +1 -0
index e4b0d44..8cf4936 100644
@@ -855,6 +855,7 @@ P5020DS_SECURE_BOOT          powerpc     mpc85xx     corenet_ds          freesca
 P5020DS_SPIFLASH	     powerpc     mpc85xx     corenet_ds          freescale      -           P5020DS:RAMBOOT_PBL,SPIFLASH,SYS_TEXT_BASE=0xFFF80000
 P5020DS_SRIO_PCIE_BOOT          powerpc     mpc85xx     corenet_ds          freescale      -           P5020DS:SRIO_PCIE_BOOT_SLAVE,SYS_TEXT_BASE=0xFFF80000
 P5040DS                      powerpc     mpc85xx     corenet_ds          freescale
+P5040DS_NAND		     powerpc     mpc85xx     corenet_ds          freescale      -           P5040DS:RAMBOOT_PBL,NAND,SYS_TEXT_BASE=0xFFF80000
 BSC9131RDB_SPIFLASH          powerpc     mpc85xx     bsc9131rdb          freescale      -           BSC9131RDB:BSC9131RDB,SPIFLASH
 stxgp3                       powerpc     mpc85xx     stxgp3              stx
 stxssa                       powerpc     mpc85xx     stxssa              stx            -           stxssa
Signature · 1 line
1.6.4
#1 lore raw

PATCH 2/3 powerpc/p5040: enable SD boot support

---
 boards.cfg |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
boards.cfg +1 -0
index 8cf4936..314afa2 100644
@@ -856,6 +856,7 @@ P5020DS_SPIFLASH	     powerpc     mpc85xx     corenet_ds          freescale
 P5020DS_SRIO_PCIE_BOOT          powerpc     mpc85xx     corenet_ds          freescale      -           P5020DS:SRIO_PCIE_BOOT_SLAVE,SYS_TEXT_BASE=0xFFF80000
 P5040DS                      powerpc     mpc85xx     corenet_ds          freescale
 P5040DS_NAND		     powerpc     mpc85xx     corenet_ds          freescale      -           P5040DS:RAMBOOT_PBL,NAND,SYS_TEXT_BASE=0xFFF80000
+P5040DS_SDCARD		     powerpc     mpc85xx     corenet_ds          freescale      -           P5040DS:RAMBOOT_PBL,SDCARD,SYS_TEXT_BASE=0xFFF80000
 BSC9131RDB_SPIFLASH          powerpc     mpc85xx     bsc9131rdb          freescale      -           BSC9131RDB:BSC9131RDB,SPIFLASH
 stxgp3                       powerpc     mpc85xx     stxgp3              stx
 stxssa                       powerpc     mpc85xx     stxssa              stx            -           stxssa
Signature · 1 line
1.6.4
#2 lore raw

PATCH 3/3 powerpc/p5040: enable SPI boot support

---
 boards.cfg |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
boards.cfg +1 -0
index 314afa2..710942f 100644
@@ -857,6 +857,7 @@ P5020DS_SRIO_PCIE_BOOT          powerpc     mpc85xx     corenet_ds          free
 P5040DS                      powerpc     mpc85xx     corenet_ds          freescale
 P5040DS_NAND		     powerpc     mpc85xx     corenet_ds          freescale      -           P5040DS:RAMBOOT_PBL,NAND,SYS_TEXT_BASE=0xFFF80000
 P5040DS_SDCARD		     powerpc     mpc85xx     corenet_ds          freescale      -           P5040DS:RAMBOOT_PBL,SDCARD,SYS_TEXT_BASE=0xFFF80000
+P5040DS_SPIFLASH	     powerpc     mpc85xx     corenet_ds          freescale      -           P5040DS:RAMBOOT_PBL,SPIFLASH,SYS_TEXT_BASE=0xFFF80000
 BSC9131RDB_SPIFLASH          powerpc     mpc85xx     bsc9131rdb          freescale      -           BSC9131RDB:BSC9131RDB,SPIFLASH
 stxgp3                       powerpc     mpc85xx     stxgp3              stx
 stxssa                       powerpc     mpc85xx     stxssa              stx            -           stxssa
Signature · 1 line
1.6.4
#3 lore raw

PATCH 1/3 powerpc/p5040: enable NAND boot support

In reply to Shaohui Xie
Quoted 2 lines · Shaohui Xie

---

These patches only add one line each, and each depends on the previous anyway. They should be merged into one patch.

Quoted 16 lines
 boards.cfg |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
boards.cfg +1 -0
index e4b0d44..8cf4936 100644
@@ -855,6 +855,7 @@ P5020DS_SECURE_BOOT          powerpc     mpc85xx     corenet_ds          freesca
 P5020DS_SPIFLASH	     powerpc     mpc85xx     corenet_ds          freescale      -           P5020DS:RAMBOOT_PBL,SPIFLASH,SYS_TEXT_BASE=0xFFF80000
 P5020DS_SRIO_PCIE_BOOT          powerpc     mpc85xx     corenet_ds          freescale      -           P5020DS:SRIO_PCIE_BOOT_SLAVE,SYS_TEXT_BASE=0xFFF80000
 P5040DS                      powerpc     mpc85xx     corenet_ds          freescale
+P5040DS_NAND		     powerpc     mpc85xx     corenet_ds          freescale      -           P5040DS:RAMBOOT_PBL,NAND,SYS_TEXT_BASE=0xFFF80000
 BSC9131RDB_SPIFLASH          powerpc     mpc85xx     bsc9131rdb          freescale      -           BSC9131RDB:BSC9131RDB,SPIFLASH
 stxgp3                       powerpc     mpc85xx     stxgp3              stx
 stxssa                       powerpc     mpc85xx     stxssa              stx            -           stxssa
Signature · 2 lines
Timur Tabi
Linux kernel developer at Freescale
#4 lore raw

PATCH 1/3 powerpc/p5040: enable NAND boot support

Quoted 8 lines

-----Original Message-----
From: Tabi Timur-B04825
Sent: Wednesday, January 16, 2013 10:42 PM
To: Xie Shaohui-B21989
Cc: u-boot at lists.denx.de
Subject: Re: [u-boot-release] [PATCH 1/3] powerpc/p5040: enable NAND boot support

Quoted 7 lines
Quoted 2 lines · Shaohui Xie

---

These patches only add one line each, and each depends on the previous anyway. They should be merged into one patch.

[S.H] I thought one patch doing one thing even it's simple. I'm fine with the merge if you insist.

Best Regards,
Shaohui Xie

#5 lore raw

PATCH 1/3 powerpc/p5040: enable NAND boot support

Quoted 2 lines · Xie Shaohui-B21989 wrote:

[S.H] I thought one patch doing one thing even it's simple. I'm fine with the merge if you insist.

Well, it is a matter of opinion. I don't insist, but I think it would be easier for everyone if you did merge these patches. Each one is very short, and they are all very similar.

Signature · 2 lines
Timur Tabi
Linux kernel developer at Freescale
#6 lore raw

PATCH 1/3 powerpc/p5040: enable NAND boot support

Quoted 8 lines

-----Original Message-----
From: Tabi Timur-B04825
Sent: Friday, January 18, 2013 10:27 AM
To: Xie Shaohui-B21989
Cc: u-boot at lists.denx.de
Subject: Re: [u-boot-release] [PATCH 1/3] powerpc/p5040: enable NAND boot support

Quoted 7 lines
Quoted 2 lines · Xie Shaohui-B21989 wrote:

[S.H] I thought one patch doing one thing even it's simple. I'm fine with the merge if you insist.

Well, it is a matter of opinion. I don't insist, but I think it would be easier for everyone if you did merge these patches. Each one is very short, and they are all very similar.

[S.H] it's also easier for me. :)
I had to separate the three lines to three patches when sending them, I thought Someone may NAC the merged patch because it added three features.

Best Regards,
Shaohui Xie

#7 lore raw

PATCH 1/3 powerpc/p5040: enable NAND boot support

In reply to Shaohui Xie
Quoted 21 lines · Shaohui Xie
---
 boards.cfg |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
boards.cfg +0 -0
index e4b0d44..8cf4936 100644
@@ -855,6 +855,7 @@ P5020DS_SECURE_BOOT          powerpc      

mpc85xx corenet_ds freesca
P5020DS_SPIFLASH powerpc mpc85xx
corenet_ds freescale -
P5020DS:RAMBOOT_PBL,SPIFLASH,SYS_TEXT_BASE=0xFFF80000
P5020DS_SRIO_PCIE_BOOT powerpc mpc85xx
corenet_ds freescale -
P5020DS:SRIO_PCIE_BOOT_SLAVE,SYS_TEXT_BASE=0xFFF80000
P5040DS powerpc mpc85xx

corenet_ds          freescale
+P5040DS_NAND		     powerpc     mpc85xx      
corenet_ds          freescale      -            
Quoted 8 lines

P5040DS:RAMBOOT_PBL,NAND,SYS_TEXT_BASE=0xFFF80000
BSC9131RDB_SPIFLASH powerpc mpc85xx
bsc9131rdb freescale -
BSC9131RDB:BSC9131RDB,SPIFLASH
stxgp3 powerpc mpc85xx
stxgp3 stx
stxssa powerpc mpc85xx
stxssa stx - stxssa

This needs more explanation. What sort of image am I supposed to get when I build "P5040DS_NAND"? Where is the PBI?

What is the long-term plan for fixing the problem of the environment not being available until after relocation? With SPL we could use CONFIG_NAND_ENV_DST (which has some issues, but they're fixable).

It would also be nice to include instructions for configuring the board to boot from NAND in a README (Freescale's user manuals often do not make this clear, especially for soft configuration). Also please provide a built-in command (or script in the default environment) to soft-boot into NAND (or if it already exists, please document it).

-Scott

#8 lore raw

PATCH 1/3 powerpc/p5040: enable NAND boot support

Quoted 32 lines

-----Original Message-----
From: Wood Scott-B07421
Sent: Thursday, January 17, 2013 4:29 AM
To: Xie Shaohui-B21989
Cc: u-boot at lists.denx.de
Subject: Re: [U-Boot] [PATCH 1/3] powerpc/p5040: enable NAND boot support

Quoted 24 lines · Shaohui Xie
---
 boards.cfg |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
boards.cfg +0 -0
index e4b0d44..8cf4936 100644
@@ -855,6 +855,7 @@ P5020DS_SECURE_BOOT          powerpc

mpc85xx corenet_ds freesca
P5020DS_SPIFLASH powerpc mpc85xx
corenet_ds freescale -
P5020DS:RAMBOOT_PBL,SPIFLASH,SYS_TEXT_BASE=0xFFF80000
P5020DS_SRIO_PCIE_BOOT powerpc mpc85xx
corenet_ds freescale -
P5020DS:SRIO_PCIE_BOOT_SLAVE,SYS_TEXT_BASE=0xFFF80000
P5040DS powerpc mpc85xx

corenet_ds          freescale
+P5040DS_NAND		     powerpc     mpc85xx
corenet_ds          freescale      -

P5040DS:RAMBOOT_PBL,NAND,SYS_TEXT_BASE=0xFFF80000
BSC9131RDB_SPIFLASH powerpc mpc85xx
bsc9131rdb freescale -

Quoted 8 lines
Quoted 5 lines

BSC9131RDB:BSC9131RDB,SPIFLASH
stxgp3 powerpc mpc85xx
stxgp3 stx
stxssa powerpc mpc85xx
stxssa stx - stxssa

This needs more explanation. What sort of image am I supposed to get when I build "P5040DS_NAND"? Where is the PBI?

[S.H] there is already a readme for p3041/p5020/p4080 (doc/README.pblimage), With the PBL tool, we will get a ramboot image "u-boot.pbl".
and also the PBI, it is shared by P3/P4/P5.

Quoted 4 lines

What is the long-term plan for fixing the problem of the environment not being available until after relocation? With SPL we could use CONFIG_NAND_ENV_DST (which has some issues, but they're fixable).

[S.H] this will need export some NAND read APIs (like load ENV stuff from CONFIG_NAND_ENV_DST), then they can be reused before relocation even not using SPL.
Please suggest.

Quoted 3 lines

It would also be nice to include instructions for configuring the board to boot from NAND in a README

[S.H] This exist in the above readme.

(Freescale's user manuals often do not make

Quoted 3 lines

this clear, especially for soft configuration). Also please provide a built-in command (or script in the default environment) to soft-boot into NAND (or if it already exists, please document it).

[S.H] I was told that when doing ramboot, I should not assume the board has a NOR flash, for ex. on customer's board, they may only have NAND. So I did not do this.

Best Regards,
Shaohui Xie

#9 lore raw

PATCH 1/3 powerpc/p5040: enable NAND boot support

Quoted 40 lines · On 01/17/2013 08:45:36 PM, Xie Shaohui-B21989 wrote:
Quoted 6 lines

-----Original Message-----
From: Wood Scott-B07421
Sent: Thursday, January 17, 2013 4:29 AM
To: Xie Shaohui-B21989
Cc: u-boot at lists.denx.de
Subject: Re: [U-Boot] [PATCH 1/3] powerpc/p5040: enable NAND boot

support

Quoted 25 lines · Shaohui Xie
Quoted 23 lines
---
 boards.cfg |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
boards.cfg +0 -0
index e4b0d44..8cf4936 100644
@@ -855,6 +855,7 @@ P5020DS_SECURE_BOOT          powerpc

mpc85xx corenet_ds freesca
P5020DS_SPIFLASH powerpc mpc85xx
corenet_ds freescale -
P5020DS:RAMBOOT_PBL,SPIFLASH,SYS_TEXT_BASE=0xFFF80000
P5020DS_SRIO_PCIE_BOOT powerpc mpc85xx
corenet_ds freescale -
P5020DS:SRIO_PCIE_BOOT_SLAVE,SYS_TEXT_BASE=0xFFF80000
P5040DS powerpc mpc85xx

corenet_ds          freescale
+P5040DS_NAND		     powerpc     mpc85xx
corenet_ds          freescale      -

P5040DS:RAMBOOT_PBL,NAND,SYS_TEXT_BASE=0xFFF80000
BSC9131RDB_SPIFLASH powerpc mpc85xx

Quoted 8 lines
Quoted 6 lines

bsc9131rdb freescale -
BSC9131RDB:BSC9131RDB,SPIFLASH
stxgp3 powerpc mpc85xx
stxgp3 stx
stxssa powerpc mpc85xx
stxssa stx - stxssa

This needs more explanation. What sort of image am I supposed to

Quoted 6 lines

get

Quoted 1 line

when I build "P5040DS_NAND"? Where is the PBI?

[S.H] there is already a readme for p3041/p5020/p4080 (doc/README.pblimage),
With the PBL tool, we will get a ramboot image "u-boot.pbl". and also the PBI, it is shared by P3/P4/P5.

The question is how is the user to know that something called "pblimage" is relevant to booting from NAND on P5040? Maybe a README.corenet_ds (or board/freescale/corenet_ds/README), that points to README.pblimage?

You don't even update strings like "P3041/P5020" in README.pblimage to include P5040. :-)
Why is P4080 excluded, BTW? I realize that P4080DS doesn't have NAND, but the file talks about chips, not boards.

Quoted 8 lines
Quoted 1 line

What is the long-term plan for fixing the problem of the

environment not

Quoted 2 lines

being available until after relocation? With SPL we could use CONFIG_NAND_ENV_DST (which has some issues, but they're fixable).

[S.H] this will need export some NAND read APIs (like load ENV stuff from CONFIG_NAND_ENV_DST),
then they can be reused before relocation even not using SPL. Please suggest.

The full NAND subsystem will not work before relocation. Your options are to either use SPL (note that you'll have no 4K limitation in this case, so you can do SPD DDR init), or make the SPL code available in non-SPL context somehow. I think the former would be easier.

Quoted 4 lines
Quoted 1 line

It would also be nice to include instructions for configuring the

board

Quoted 1 line

to boot from NAND in a README

[S.H] This exist in the above readme.

I only see hardware switch settings. Is there no way to soft-boot into NAND (similar to using "pixis/qixis altbank" to boot into an alternate NOR bank)? Our e500v2-based boards have been able to do this...

Quoted 10 lines

(Freescale's user manuals often do not make

Quoted 1 line

this clear, especially for soft configuration). Also please

provide a

Quoted 1 line

built-in command (or script in the default environment) to

soft-boot into

Quoted 1 line

NAND (or if it already exists, please document it).

[S.H] I was told that when doing ramboot, I should not assume the board has a NOR flash,
for ex. on customer's board, they may only have NAND. So I did not do this.

I'm not asking you to assume that, just to provide some helpful instructions for people who happen to have both (in addition to the hardware switch instructions that are already there). Actually, these instructions (both soft boot and hardware switches) are board-specific rather than SoC-specific and should go in a board README instead. And once you know we're talking about a corenet_ds board, you know we have NOR.

-Scott

#10 lore raw

PATCH 1/3 powerpc/p5040: enable NAND boot support

Quoted 11 lines
Quoted 7 lines
Quoted 1 line

This needs more explanation. What sort of image am I supposed to

get

Quoted 1 line

when I build "P5040DS_NAND"? Where is the PBI?

[S.H] there is already a readme for p3041/p5020/p4080 (doc/README.pblimage), With the PBL tool, we will get a ramboot image "u-boot.pbl".
and also the PBI, it is shared by P3/P4/P5.

The question is how is the user to know that something called "pblimage" is relevant to booting from NAND on P5040? Maybe a README.corenet_ds (or board/freescale/corenet_ds/README), that points to README.pblimage?

[S.H] Yes. A README for corenet_ds board will be more useful, then a ramboot section points to README.pblimage.

Quoted 3 lines

You don't even update strings like "P3041/P5020" in README.pblimage to include P5040. :-)

[S.H] Thanks for pointing it out. I will update the README.pblimage in next version.

Why is P4080 excluded, BTW? I realize that P4080DS

Quoted 1 line

doesn't have NAND, but the file talks about chips, not boards.

[S.H] there is no P4080DS_NAND entry in boards.cfg...

Quoted 10 lines
Quoted 9 lines
Quoted 2 lines

What is the long-term plan for fixing the problem of the

environment not

Quoted 2 lines

being available until after relocation? With SPL we could use CONFIG_NAND_ENV_DST (which has some issues, but they're fixable).

[S.H] this will need export some NAND read APIs (like load ENV stuff from CONFIG_NAND_ENV_DST), then they can be reused before relocation even not using SPL.
Please suggest.

Quoted 4 lines

The full NAND subsystem will not work before relocation. Your options are to either use SPL (note that you'll have no 4K limitation in this case, so you can do SPD DDR init), or make the SPL code available in non- SPL context somehow. I think the former would be easier.

[S.H] Thanks. I will assess the effort base on my case.

Quoted 9 lines
Quoted 5 lines
Quoted 2 lines

It would also be nice to include instructions for configuring the

board

Quoted 1 line

to boot from NAND in a README

[S.H] This exist in the above readme.

I only see hardware switch settings. Is there no way to soft-boot into NAND (similar to using "pixis/qixis altbank" to boot into an alternate NOR bank)? Our e500v2-based boards have been able to do this...

[S.H] we do have these soft-boot instructions for corenet_ds board. And yes, they should go in README of corenet_ds board, but seems we don't have such a README.

Best Regards,
Shaohui Xie

#11 lore raw

PATCH 1/3 powerpc/p5040: enable NAND boot support

Quoted 9 lines · On 01/17/2013 11:17:15 PM, Xie Shaohui-B21989 wrote:
Quoted 1 line

You don't even update strings like "P3041/P5020" in README.pblimage

to

Quoted 1 line

include P5040. :-)

[S.H] Thanks for pointing it out. I will update the README.pblimage in next version.

Why is P4080 excluded, BTW? I realize that P4080DS

Quoted 1 line

doesn't have NAND, but the file talks about chips, not boards.

[S.H] there is no P4080DS_NAND entry in boards.cfg...

Yes, I said I realize that P4080DS (note the DS) doesn't have NAND. But P4080 the SoC should support NAND boot on custom boards that have NAND, and README.pblimage doesn't appear to be talking about specific boards (except for the hardware switch section).

-Scott