Re: [U-Boot] SPL Linux Boot

2012-03-19 Thread Tom Rini
On Fri, Mar 16, 2012 at 02:24:42PM -0700, Tom Rini wrote:
 On Fri, Mar 16, 2012 at 09:38:10PM +0100, Wolfgang Denk wrote:
  Dear Tom Rini,
  
  In message 20120316193030.GC31661@bill-the-cat you wrote:
  
   OK, that said, I'm OK with everything so far and you had the last round
   of please fix/change this comments.  Are you happy with it all now?
  
  We can iterate forever on improvements, or we can get it in now.
  Let's do the latter :-)
 
 OK.  But now we add to the ELDK 4.2 saga.  This series adds not just a
 warning (well, 5 for CONFIG_OMAP I guess, 1 for everything else) for
 everything ARM but also breaks everything else over undefined reference
 to 'raise'.  ELDK 5.1 is fine.  Now what?

Welp, now I don't get it.  I saw this Friday, I don't see it today.  I
blame gremlins.  MAKEALL -a arm is going along rather nicely, just the
one warning Stefano also sees.  I'll go fix that, post what I changed
and get out an updated pull request for Albert.

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


Re: [U-Boot] SPL Linux Boot

2012-03-17 Thread stefano babic
Am 17/03/2012 05:34, schrieb Thomas Weber:
 Hello,
 On 03/15/2012 03:01 PM, Stefano Babic wrote:
 Hi,


Hi Thomas,

 http://lists.denx.de/mailman/listinfo/u-boot
 It applies on TI-u-boot, but TI-u-boot is still broken for Devkit8000
 (SOFT_ECC).

Yes, I have the same problem, too. The commit breaking TI boards is
reverted in u-boot mainline.

 After merging mainline-u-boot into TI-u-boot it works fine:
 
 Tested-by: Thomas Weber we...@corscience.de on Devkit8000
 

Thanks for testing,

Stefano


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2012-03-17 Thread Wolfgang Denk
Dear Tom Rini,

In message 20120316212442.GE31661@bill-the-cat you wrote:

 OK.  But now we add to the ELDK 4.2 saga.  This series adds not just a
 warning (well, 5 for CONFIG_OMAP I guess, 1 for everything else) for
 everything ARM but also breaks everything else over undefined reference
 to 'raise'.  ELDK 5.1 is fine.  Now what?

This raise issue is probably due to a misconfiguration.

See arch/arm/config.mk, search for For EABI, make sure to provide
raise().

Check why arch/arm/lib/eabi_compat.c does not get compiled / linked.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
How can you tell when sour cream goes bad?
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2012-03-17 Thread Tom Rini
On Sat, Mar 17, 2012 at 8:16 AM, Wolfgang Denk w...@denx.de wrote:
 Dear Tom Rini,

 In message 20120316212442.GE31661@bill-the-cat you wrote:

 OK.  But now we add to the ELDK 4.2 saga.  This series adds not just a
 warning (well, 5 for CONFIG_OMAP I guess, 1 for everything else) for
 everything ARM but also breaks everything else over undefined reference
 to 'raise'.  ELDK 5.1 is fine.  Now what?

 This raise issue is probably due to a misconfiguration.

 See arch/arm/config.mk, search for For EABI, make sure to provide
 raise().

 Check why arch/arm/lib/eabi_compat.c does not get compiled / linked.

Even if ELDK5.1 is totally fine?  And other modern toolchains as well..

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


Re: [U-Boot] SPL Linux Boot

2012-03-17 Thread Stefano Babic
On 17/03/2012 16:24, Tom Rini wrote:
 On Sat, Mar 17, 2012 at 8:16 AM, Wolfgang Denk w...@denx.de wrote:
 Dear Tom Rini,


Hi Tom,

 In message 20120316212442.GE31661@bill-the-cat you wrote:

 OK.  But now we add to the ELDK 4.2 saga.  This series adds not just a
 warning (well, 5 for CONFIG_OMAP I guess, 1 for everything else) for
 everything ARM but also breaks everything else over undefined reference
 to 'raise'.  ELDK 5.1 is fine.  Now what?

 This raise issue is probably due to a misconfiguration.

 See arch/arm/config.mk, search for For EABI, make sure to provide
 raise().

 Check why arch/arm/lib/eabi_compat.c does not get compiled / linked.
 
 Even if ELDK5.1 is totally fine?  And other modern toolchains as well..
 

Compiling with ELDK-4.2, I see only a couple of warnings more. But
wihout any problem with raise. Really I had to revert Revert
config.mk: Check for -fstack-usage support, but this is another story.

Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2012-03-17 Thread Wolfgang Denk
Dear Tom Rini,

In message ca+m6bxk90tfwearc-dtfo3f8rg5jbmi7iz+kxmxgcxymhzf...@mail.gmail.com 
you wrote:

  See arch/arm/config.mk, search for For EABI, make sure to provide
  raise().
 
  Check why arch/arm/lib/eabi_compat.c does not get compiled / linked.
 
 Even if ELDK5.1 is totally fine?  And other modern toolchains as well..

Yes, this problem is supposed to be handled by the code I mentioned.

There must be an error somewhere, if it doesn;t work.

Does arch/arm/lib/eabi_compat.c get compiled and linked?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Randal said it would be tough to do in sed. He didn't say  he  didn't
understand  sed.  Randal  understands sed quite well. Which is why he
uses Perl. :-) - Larry Wall in 7...@jpl-devvax.jpl.nasa.gov
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2012-03-16 Thread Tom Rini
On Thu, Mar 15, 2012 at 03:01:33PM +0100, Stefano Babic wrote:

 Hi,
 
 this should be the final version for the patchset introducing
 direct Linux booting from SPL.
 
 Only Patch 1/14: Add cmd_spl command was modified.
 Patches 2/13 up to 12/13 are untouched from V13.
 Patch 13/13 fix a small issue by compiling the cam_enc_4xx
 board (Tom Rini)

My first question is, are we hoping to get this in for v2012.03 or push
it off one more time, but really get it in (and put it in a /next branch
now'ish) ?

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


Re: [U-Boot] SPL Linux Boot

2012-03-16 Thread Wolfgang Denk
Dear Tom Rini,

In message 20120316172653.GD4234@bill-the-cat you wrote:
 
  Only Patch 1/14: Add cmd_spl command was modified.
  Patches 2/13 up to 12/13 are untouched from V13.
  Patch 13/13 fix a small issue by compiling the cam_enc_4xx
  board (Tom Rini)
 
 My first question is, are we hoping to get this in for v2012.03 or push
 it off one more time, but really get it in (and put it in a /next branch
 now'ish) ?

I want to get this into the upcoming release.  I rather delay the
release then keeping thie queued even longer.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Often it is fatal to live too long.  - Racine
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2012-03-16 Thread Tom Rini
On Fri, Mar 16, 2012 at 08:24:55PM +0100, Wolfgang Denk wrote:
 Dear Tom Rini,
 
 In message 20120316172653.GD4234@bill-the-cat you wrote:
  
   Only Patch 1/14: Add cmd_spl command was modified.
   Patches 2/13 up to 12/13 are untouched from V13.
   Patch 13/13 fix a small issue by compiling the cam_enc_4xx
   board (Tom Rini)
  
  My first question is, are we hoping to get this in for v2012.03 or push
  it off one more time, but really get it in (and put it in a /next branch
  now'ish) ?
 
 I want to get this into the upcoming release.  I rather delay the
 release then keeping thie queued even longer.

OK, that said, I'm OK with everything so far and you had the last round
of please fix/change this comments.  Are you happy with it all now?

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


Re: [U-Boot] SPL Linux Boot

2012-03-16 Thread Wolfgang Denk
Dear Tom Rini,

In message 20120316193030.GC31661@bill-the-cat you wrote:

 OK, that said, I'm OK with everything so far and you had the last round
 of please fix/change this comments.  Are you happy with it all now?

We can iterate forever on improvements, or we can get it in now.
Let's do the latter :-)

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
I believe you find life such a problem because you  think  there  are
the  good  people  and the bad people. You're wrong, of course. There
are, always and only, the bad people, but some of them are  on  oppo-
site sides.  - Terry Pratchett, _Guards! Guards!_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2012-03-16 Thread Tom Rini
On Fri, Mar 16, 2012 at 09:38:10PM +0100, Wolfgang Denk wrote:
 Dear Tom Rini,
 
 In message 20120316193030.GC31661@bill-the-cat you wrote:
 
  OK, that said, I'm OK with everything so far and you had the last round
  of please fix/change this comments.  Are you happy with it all now?
 
 We can iterate forever on improvements, or we can get it in now.
 Let's do the latter :-)

OK.  But now we add to the ELDK 4.2 saga.  This series adds not just a
warning (well, 5 for CONFIG_OMAP I guess, 1 for everything else) for
everything ARM but also breaks everything else over undefined reference
to 'raise'.  ELDK 5.1 is fine.  Now what?

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


Re: [U-Boot] SPL Linux Boot

2012-03-16 Thread Thomas Weber
Hello,
On 03/15/2012 03:01 PM, Stefano Babic wrote:
 Hi,

 this should be the final version for the patchset introducing
 direct Linux booting from SPL.

 Only Patch 1/14: Add cmd_spl command was modified.
 Patches 2/13 up to 12/13 are untouched from V13.
 Patch 13/13 fix a small issue by compiling the cam_enc_4xx
 board (Tom Rini)

 Changelog
 --

 [PATCH V14 01/13] Add cmd_spl command
 [PATCH V14 02/13] devkit8000: add config for spl command
 [PATCH V14 03/13] omap-common: Add NAND SPL linux booting
 [PATCH V14 04/13] devkit8000/spl: init GPMC for dm9000 in SPL
 [PATCH V14 05/13] omap-common/spl: Add linux boot to SPL
 [PATCH V14 06/13] omap/spl: change output of spl_parse_image_header
 [PATCH V14 07/13] devkit8000: Implement and activate direct OS boot
 [PATCH V14 08/13] Add cache functions to SPL for armv7
 [PATCH V14 09/13] OMAP3: SPL: do not call I2C init if no I2C is set.
 [PATCH V14 10/13] SPL: call cleanup_before_linux() before booting
 [PATCH V14 11/13] OMAP3: twister: add support to boot Linux from SPL
 [PATCH V14 12/13] arm: Add Prep subcommand support to bootm
 [PATCH V14 13/13] cam_enc_4xx: Rename 'images' to 'imgs'
 ___
 U-Boot mailing list
 U-Boot@lists.denx.de
 http://lists.denx.de/mailman/listinfo/u-boot
It applies on TI-u-boot, but TI-u-boot is still broken for Devkit8000
(SOFT_ECC).
After merging mainline-u-boot into TI-u-boot it works fine:

Tested-by: Thomas Weber we...@corscience.de on Devkit8000

Thanks,

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


[U-Boot] SPL Linux Boot

2012-03-15 Thread Stefano Babic
Hi,

this should be the final version for the patchset introducing
direct Linux booting from SPL.

Only Patch 1/14: Add cmd_spl command was modified.
Patches 2/13 up to 12/13 are untouched from V13.
Patch 13/13 fix a small issue by compiling the cam_enc_4xx
board (Tom Rini)

Changelog
--

[PATCH V14 01/13] Add cmd_spl command
[PATCH V14 02/13] devkit8000: add config for spl command
[PATCH V14 03/13] omap-common: Add NAND SPL linux booting
[PATCH V14 04/13] devkit8000/spl: init GPMC for dm9000 in SPL
[PATCH V14 05/13] omap-common/spl: Add linux boot to SPL
[PATCH V14 06/13] omap/spl: change output of spl_parse_image_header
[PATCH V14 07/13] devkit8000: Implement and activate direct OS boot
[PATCH V14 08/13] Add cache functions to SPL for armv7
[PATCH V14 09/13] OMAP3: SPL: do not call I2C init if no I2C is set.
[PATCH V14 10/13] SPL: call cleanup_before_linux() before booting
[PATCH V14 11/13] OMAP3: twister: add support to boot Linux from SPL
[PATCH V14 12/13] arm: Add Prep subcommand support to bootm
[PATCH V14 13/13] cam_enc_4xx: Rename 'images' to 'imgs'
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] SPL Linux Boot V13

2012-02-04 Thread Stefano Babic
Rebased patchset to add Linux booting from SPL. As discussed
in the ML, this patchset does not make SPL common to all SOCs
to avoid breaking some architecture - this job will be done later
with a different patchset.

For this reason, a couple of patches moving files in the common/ 
directory are dropped - the other patches are not changed 
from V12.
(exception due only to rebase on current u-boot-ti)

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


Re: [U-Boot] SPL Linux Boot

2012-02-01 Thread Stefano Babic
On 31/01/2012 15:26, Tom Rini wrote:

 A way can be to add an arch weak function to add arch specific
 initialization inside  board_init_*() functions, if any. Have you seen
 other issues other what I report now ?
 
 That sounds about right.  In short, ./MAKEALL -s arm still needs to
 build before we can take this series in, so perhaps we want to
 separate back out the make SPL stuff more generic from the let SPL
 for omap* boot Linux series.

I agree - your proposal makes things easier.  I split the patches and I
will resend the patchset to add only booting Linux from SPL - and I will
rework the patches to make SPL generic to other SOCs later, when the
first patchset will be merged.

Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2012-01-31 Thread Stefano Babic
On 30/01/2012 19:05, Tom Rini wrote:

Hi Tom,

 OK, doing a MAKEALL for omap3/4/5/am33xx/davinci shows we've got a few
 missing include problems, I'll go fix that up.
 
 OK, was it when you took over the series that you moved stuff to
 common/spl.c ?  That idea is breaking, badly, non-omap platforms.

I am trying to summarize the different problems we have.
Comparing to u-boot-ti, the patchset breaks the following boards:

davinci:
da850_am18xxevm da850evm hawkboard_nand

omap4/5:
fixed including spl.h into arch/arm/cpu/armv7/omap-common/boot-common.c

mx28:
m28evk mx28evk

I have not seen other broken boards - let me know if you find something
else.

On omap4/5 is only a thing of include files, as you reported.

A main issue is with davinci, because we have currently two different
spl.c (arch/arm/cpu/arm926ejs/davinci/spl.c on davinci). The code can be
factorized, if davinci will use preloader_console_init() as omap.

The same problem rises with mx28: there is a specific file
arch/arm/cpu/arm926ejs/mx28/spl_boot.c, that redefines board_init_f()
and board_init_r() - as we try to have a common spl code, this muist be
fixed, too.

A way can be to add an arch weak function to add arch specific
initialization inside  board_init_*() functions, if any. Have you seen
other issues other what I report now ?

Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2012-01-31 Thread Tom Rini
On Tue, Jan 31, 2012 at 4:25 AM, Stefano Babic sba...@denx.de wrote:
 On 30/01/2012 19:05, Tom Rini wrote:

 Hi Tom,

 OK, doing a MAKEALL for omap3/4/5/am33xx/davinci shows we've got a few
 missing include problems, I'll go fix that up.

 OK, was it when you took over the series that you moved stuff to
 common/spl.c ?  That idea is breaking, badly, non-omap platforms.

 I am trying to summarize the different problems we have.
 Comparing to u-boot-ti, the patchset breaks the following boards:

 davinci:
        da850_am18xxevm da850evm hawkboard_nand

 omap4/5:
        fixed including spl.h into arch/arm/cpu/armv7/omap-common/boot-common.c

 mx28:
        m28evk mx28evk

 I have not seen other broken boards - let me know if you find something
 else.

 On omap4/5 is only a thing of include files, as you reported.

 A main issue is with davinci, because we have currently two different
 spl.c (arch/arm/cpu/arm926ejs/davinci/spl.c on davinci). The code can be
 factorized, if davinci will use preloader_console_init() as omap.

 The same problem rises with mx28: there is a specific file
 arch/arm/cpu/arm926ejs/mx28/spl_boot.c, that redefines board_init_f()
 and board_init_r() - as we try to have a common spl code, this muist be
 fixed, too.

 A way can be to add an arch weak function to add arch specific
 initialization inside  board_init_*() functions, if any. Have you seen
 other issues other what I report now ?

That sounds about right.  In short, ./MAKEALL -s arm still needs to
build before we can take this series in, so perhaps we want to
separate back out the make SPL stuff more generic from the let SPL
for omap* boot Linux series.

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


Re: [U-Boot] SPL Linux Boot

2012-01-25 Thread Stefano Babic
Hi Tom,

last version of patchset has not received comments since a long time -
Simon sends to me a confirm that tests on the devkit8000 are successful.

Any chance to get the patchset merged soon or do you have any open points ?

Thanks,
Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2012-01-25 Thread Tom Rini
On Wed, Jan 25, 2012 at 4:50 AM, Stefano Babic sba...@denx.de wrote:
 Hi Tom,

 last version of patchset has not received comments since a long time -
 Simon sends to me a confirm that tests on the devkit8000 are successful.

 Any chance to get the patchset merged soon or do you have any open points ?

I've sent Albert a pull request now for the few things I've got queued
up other than this.  Once he takes that I'll take the Linux SPL series
in, give it a few whacks and move it along to Albert.

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


[U-Boot] SPL Linux Boot - updated

2012-01-04 Thread Stefano Babic
Hi,

here an updated version of the patchset to boot Linux directly from SPL.
It fixes the (minor) comments received in version 11.

List of changes:

- drop commented lines into arch/arm/cpu/armv7/Makefile after checking build
  for OMAP4/5
- use CONFIG_SPL_I2C_SUPPORT to protect I2C code
- fix TAB / spaces issues (codestyle)
- add missing include to fix OMAP4/5 build
- use a different GPIO on the twister board to select image to boot

[PATCH V12 01/14] Add cmd_spl command
[PATCH V12 02/14] devki8000: add config for spl command
[PATCH V12 03/14] omap-common: Add NAND SPL linux booting
[PATCH V12 04/14] devkit8000/spl: init GPMC for dm9000 in SPL
[PATCH V12 05/14] omap-common/spl: Add linux boot to SPL
[PATCH V12 06/14] omap/spl: change output of spl_parse_image_header
[PATCH V12 07/14] devkit8000: Implement and activate direct OS boot
[PATCH V12 08/14] Add cache functions to SPL for armv7
[PATCH V12 09/14] OMAP3: SPL: do not call I2C init if no I2C is set.
[PATCH V12 10/14] OMAP3: move SPL files to be used by other
[PATCH V12 11/14] TI: SPL: make SPL available for other SOCs as TI
[PATCH V12 12/14] SPL: call cleanup_before_linux() before booting
[PATCH V12 13/14] Fix build OMAP4/5
[PATCH V12 14/14] OMAP3: twister: add support to boot Linux from SPL
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2011-12-19 Thread Stefano Babic
On 16/12/2011 16:45, Tom Rini wrote:
 On Fri, Dec 16, 2011 at 8:37 AM, Stefano Babic sba...@denx.de wrote:
 Hi,


Hi Tom,

 I'm going to mark this series as RFC in patchwork as I see you did
 +#ifneq... in some Makefiles and I know you know better :)  That said,
 can you confirm MAKEALL -s omap3 -s omap4 -s omap5 works?

there is a missing include in this patchset (fixed in next version), but
I can compile all.

  omap4/5 has
 smaller size requirements than omap3(ish) stuff and I don't have 100%
 faith in linker magic to make sure the cache stuff added to SPL in
 arch/arm/lib/ isn't going to bloat up the omap4/5 stuff slightly.


Checking MLO without and with this patchset for an OMAP4 board (panda),
I get:

without patchset (u-boot-ti): 33012 bytes
with the patchset (SPL_OS not enabled): 33424

I cannot test it, but is seems still in order, right ?

Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2011-12-19 Thread Tom Rini
On Mon, Dec 19, 2011 at 1:43 AM, Stefano Babic sba...@denx.de wrote:
 On 16/12/2011 16:45, Tom Rini wrote:
 On Fri, Dec 16, 2011 at 8:37 AM, Stefano Babic sba...@denx.de wrote:
 Hi,


 Hi Tom,

 I'm going to mark this series as RFC in patchwork as I see you did
 +#ifneq... in some Makefiles and I know you know better :)  That said,
 can you confirm MAKEALL -s omap3 -s omap4 -s omap5 works?

 there is a missing include in this patchset (fixed in next version), but
 I can compile all.

  omap4/5 has
 smaller size requirements than omap3(ish) stuff and I don't have 100%
 faith in linker magic to make sure the cache stuff added to SPL in
 arch/arm/lib/ isn't going to bloat up the omap4/5 stuff slightly.


 Checking MLO without and with this patchset for an OMAP4 board (panda),
 I get:

 without patchset (u-boot-ti): 33012 bytes
 with the patchset (SPL_OS not enabled): 33424

 I cannot test it, but is seems still in order, right ?

Yes, when it's too large it fails to link.  Thanks!

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


[U-Boot] SPL Linux Boot

2011-12-16 Thread Stefano Babic
Hi,

this is a new submission of the Simon's work to add a way to
boot Linux directly from the SPL. From the last patchset series,
the main changes introduced by me are the following:

- compile the cache functions in SPL to make cleanup_before_linux()
  available.
- do not call IC function on boards where I2C is not available
- fix a bug from last version (fallback to U-Boot)
- move all related SPL stuff to common/ and fix omap related
  functions.

Tested on the twister board (patches for this board in u-boot-ti, next)

Note: to test it is also required Simon's patch:

arm: Add Prep subcommand support to bootm

http://patchwork.ozlabs.org/patch/129753/


[PATCH V11  01/13] Add cmd_spl command
[PATCH V11  02/13] devki8000: add config for spl command
[PATCH V11  03/13] omap-common: Add NAND SPL linux booting
[PATCH V11  04/13] devkit8000/spl: init GPMC for dm9000 in SPL
[PATCH V11  05/13] omap-common/spl: Add linux boot to SPL
[PATCH V11  06/13] omap/spl: change output of spl_parse_image_header
[PATCH V11  07/13] devkit8000: Implement and activate direct OS boot
[PATCH V11  08/13] Add cache functions to SPL for armv7
[PATCH V11  09/13] OMAP3: SPL: do not call I2C init if no I2C is
[PATCH V11  10/13] OMAP3: move SPL files to be used by other
[PATCH V11  11/13] TI: SPL: make SPL available for other SOCs as TI
[PATCH V11  12/13] SPL: call cleanup_before_linux() before booting
[PATCH V11  13/13] OMAP3: twister: add support to boot Linux from
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] SPL Linux Boot

2011-12-16 Thread Tom Rini
On Fri, Dec 16, 2011 at 8:37 AM, Stefano Babic sba...@denx.de wrote:
 Hi,

 this is a new submission of the Simon's work to add a way to
 boot Linux directly from the SPL. From the last patchset series,
 the main changes introduced by me are the following:

 - compile the cache functions in SPL to make cleanup_before_linux()
  available.
 - do not call IC function on boards where I2C is not available
 - fix a bug from last version (fallback to U-Boot)
 - move all related SPL stuff to common/ and fix omap related
  functions.

 Tested on the twister board (patches for this board in u-boot-ti, next)

 Note: to test it is also required Simon's patch:

        arm: Add Prep subcommand support to bootm

 http://patchwork.ozlabs.org/patch/129753/

I'm going to mark this series as RFC in patchwork as I see you did
+#ifneq... in some Makefiles and I know you know better :)  That said,
can you confirm MAKEALL -s omap3 -s omap4 -s omap5 works?  omap4/5 has
smaller size requirements than omap3(ish) stuff and I don't have 100%
faith in linker magic to make sure the cache stuff added to SPL in
arch/arm/lib/ isn't going to bloat up the omap4/5 stuff slightly.

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


Re: [U-Boot] SPL Linux Boot

2011-12-16 Thread Stefano Babic
On 16/12/2011 16:45, Tom Rini wrote:
 On Fri, Dec 16, 2011 at 8:37 AM, Stefano Babic sba...@denx.de wrote:
 Hi,

 this is a new submission of the Simon's work to add a way to
 boot Linux directly from the SPL. From the last patchset series,
 the main changes introduced by me are the following:

 - compile the cache functions in SPL to make cleanup_before_linux()
  available.
 - do not call IC function on boards where I2C is not available
 - fix a bug from last version (fallback to U-Boot)
 - move all related SPL stuff to common/ and fix omap related
  functions.

 Tested on the twister board (patches for this board in u-boot-ti, next)

 Note: to test it is also required Simon's patch:

arm: Add Prep subcommand support to bootm

 http://patchwork.ozlabs.org/patch/129753/
 
 I'm going to mark this series as RFC in patchwork as I see you did
 +#ifneq... in some Makefiles and I know you know better :)  That said,
 can you confirm MAKEALL -s omap3 -s omap4 -s omap5 works?

I'll do, I have not yet tested and I built OMAP3. I have seen a couple
of other things due to the moved stuff that I should fix.

  omap4/5 has
 smaller size requirements than omap3(ish) stuff and I don't have 100%
 faith in linker magic to make sure the cache stuff added to SPL in
 arch/arm/lib/ isn't going to bloat up the omap4/5 stuff slightly.

Ok, understood, I'll check it - if the added code is too much, I will
introduce a new CONFIG_SPL_SUPPORT_CACHE to make build optional.

Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot