Re: [U-Boot] [PATCHv2 02/20] arm: socfpga: spl: Add SRAM section

2015-03-09 Thread Dinh Nguyen


On 3/5/15 2:59 PM, Marek Vasut wrote:
 On Wednesday, March 04, 2015 at 10:34:30 PM, Dinh Nguyen wrote:
 On 03/04/2015 01:39 PM, Marek Vasut wrote:
 On Wednesday, March 04, 2015 at 07:52:04 PM, Dinh Nguyen wrote:
 On 03/04/2015 06:39 AM, Marek Vasut wrote:
 On Monday, March 02, 2015 at 05:27:50 PM,
 dingu...@opensource.altera.com

 wrote:
 From: Dinh Nguyen dingu...@opensource.altera.com

 Add a section of SRAM to the SPL linker file.

 Signed-off-by: Dinh Nguyen dingu...@opensource.altera.com

 Hi!

 Can we not use the generic SPL stuff ( ./arch/arm/cpu/u-boot-spl.lds )
 here instead please ?

 I looked at this, but I didn't see way to overlay in an SRAM section. Do
 you have an example of how to do this?

 Hi!

 Is it really needed at all in the first place ? Why ?

 Isn't this need to specify the SPL to be located in the sram section? I
 see other platforms doing the same thing, ie. zynq, sunxi.
 
 CONFIG_SPL_TEXT_BASE is what specifies the address to which the SPL is linked.
 This is the address where the BootROM should place the SPL and jump to it's 
 first instruction.
 
 I don't think you explicitly need to hack linker scripts to add this section.

Ah yes, I think I can make use of it now.

 
 btw. Tom Rini has a different email address (@konsulko) now.

Right...saw it right after I sent out this series.

 
 btw2. is there any way I can help you with hacking on the SPL please ?
 

Let me post a v3 within this week, and I would appreciate any help. Does
posting the series to git repo help you to help me?

Dinh
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCHv2 02/20] arm: socfpga: spl: Add SRAM section

2015-03-05 Thread Marek Vasut
On Wednesday, March 04, 2015 at 10:34:30 PM, Dinh Nguyen wrote:
 On 03/04/2015 01:39 PM, Marek Vasut wrote:
  On Wednesday, March 04, 2015 at 07:52:04 PM, Dinh Nguyen wrote:
  On 03/04/2015 06:39 AM, Marek Vasut wrote:
  On Monday, March 02, 2015 at 05:27:50 PM,
  dingu...@opensource.altera.com
  
  wrote:
  From: Dinh Nguyen dingu...@opensource.altera.com
  
  Add a section of SRAM to the SPL linker file.
  
  Signed-off-by: Dinh Nguyen dingu...@opensource.altera.com
  
  Hi!
  
  Can we not use the generic SPL stuff ( ./arch/arm/cpu/u-boot-spl.lds )
  here instead please ?
  
  I looked at this, but I didn't see way to overlay in an SRAM section. Do
  you have an example of how to do this?
  
  Hi!
  
  Is it really needed at all in the first place ? Why ?
 
 Isn't this need to specify the SPL to be located in the sram section? I
 see other platforms doing the same thing, ie. zynq, sunxi.

CONFIG_SPL_TEXT_BASE is what specifies the address to which the SPL is linked.
This is the address where the BootROM should place the SPL and jump to it's 
first instruction.

I don't think you explicitly need to hack linker scripts to add this section.

btw. Tom Rini has a different email address (@konsulko) now.

btw2. is there any way I can help you with hacking on the SPL please ?

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCHv2 02/20] arm: socfpga: spl: Add SRAM section

2015-03-04 Thread Marek Vasut
On Wednesday, March 04, 2015 at 07:52:04 PM, Dinh Nguyen wrote:
 On 03/04/2015 06:39 AM, Marek Vasut wrote:
  On Monday, March 02, 2015 at 05:27:50 PM, dingu...@opensource.altera.com 
wrote:
  From: Dinh Nguyen dingu...@opensource.altera.com
  
  Add a section of SRAM to the SPL linker file.
  
  Signed-off-by: Dinh Nguyen dingu...@opensource.altera.com
  
  Hi!
  
  Can we not use the generic SPL stuff ( ./arch/arm/cpu/u-boot-spl.lds )
  here instead please ?
 
 I looked at this, but I didn't see way to overlay in an SRAM section. Do
 you have an example of how to do this?

Hi!

Is it really needed at all in the first place ? Why ?

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCHv2 02/20] arm: socfpga: spl: Add SRAM section

2015-03-04 Thread Dinh Nguyen
On 03/04/2015 06:39 AM, Marek Vasut wrote:
 On Monday, March 02, 2015 at 05:27:50 PM, dingu...@opensource.altera.com 
 wrote:
 From: Dinh Nguyen dingu...@opensource.altera.com

 Add a section of SRAM to the SPL linker file.

 Signed-off-by: Dinh Nguyen dingu...@opensource.altera.com
 
 Hi!
 
 Can we not use the generic SPL stuff ( ./arch/arm/cpu/u-boot-spl.lds ) here
 instead please ?
 

I looked at this, but I didn't see way to overlay in an SRAM section. Do
you have an example of how to do this?

Thanks,
Dinh

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCHv2 02/20] arm: socfpga: spl: Add SRAM section

2015-03-04 Thread Dinh Nguyen
On 03/04/2015 01:39 PM, Marek Vasut wrote:
 On Wednesday, March 04, 2015 at 07:52:04 PM, Dinh Nguyen wrote:
 On 03/04/2015 06:39 AM, Marek Vasut wrote:
 On Monday, March 02, 2015 at 05:27:50 PM, dingu...@opensource.altera.com 
 wrote:
 From: Dinh Nguyen dingu...@opensource.altera.com

 Add a section of SRAM to the SPL linker file.

 Signed-off-by: Dinh Nguyen dingu...@opensource.altera.com

 Hi!

 Can we not use the generic SPL stuff ( ./arch/arm/cpu/u-boot-spl.lds )
 here instead please ?

 I looked at this, but I didn't see way to overlay in an SRAM section. Do
 you have an example of how to do this?
 
 Hi!
 
 Is it really needed at all in the first place ? Why ?
 

Isn't this need to specify the SPL to be located in the sram section? I
see other platforms doing the same thing, ie. zynq, sunxi.

Dinh

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCHv2 02/20] arm: socfpga: spl: Add SRAM section

2015-03-04 Thread Marek Vasut
On Monday, March 02, 2015 at 05:27:50 PM, dingu...@opensource.altera.com wrote:
 From: Dinh Nguyen dingu...@opensource.altera.com
 
 Add a section of SRAM to the SPL linker file.
 
 Signed-off-by: Dinh Nguyen dingu...@opensource.altera.com

Hi!

Can we not use the generic SPL stuff ( ./arch/arm/cpu/u-boot-spl.lds ) here
instead please ?

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot