public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: "Andreas Müller" <schnitzeltony@gmx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] overo: add SPL support
Date: Wed, 21 Dec 2011 02:00:21 +0100	[thread overview]
Message-ID: <201112210200.22049.schnitzeltony@gmx.de> (raw)
In-Reply-To: <CAOSpxdYehNYQYX2-gbBxmWto7-DRrJoVXXhAiHitvQJ=9HNiYQ@mail.gmail.com>

On Tuesday, December 20, 2011 02:55:50 PM you wrote:
> On Tue, Dec 20, 2011 at 4:20 AM, Igor Grinberg <grinberg@compulab.co.il>wrote:
> > What about forging some very not optimized default DRAM settings,
> > that suit any assembled DRAM and then when you have I2C access,
> > reconfigure it - is it possible?
> 
> The board ID is used to determine some fairly fundamental things like how
> the address bits are multiplexed, bank size, number of banks, and timing.
> 
> Perhaps it might be possible to determine some non-optimal settings that
> can work with the current set of POP memories used, and also a scheme to
> modify the above on the fly while executing from said ram, but then one
> would have to revisit this every time a new vendor/type of POP was used.
> 
> That seems a lot more complex than the current method.
> 
> I suppose we could just drop support for the old boards in u-boot.  Those
> folks could continue to use the current x-load solution. 
I am afraid you are right. It seems that the current U-boot/OMAP/SPL environment 
it is not designed to have i2c in early state when SDRAM is set up. 
My experience:
* After moving the i2c variables to SDRAM, i2c_set_bus_num (and thereby 
i2c_init) work but i2c_write hangs when calling udelay because timer is not yet 
running.
* This can be worked around by calling timer_init() within s_init(). With this 
setup booting is possible but I get MMC timeout messages from u-boot although 
kernel is loaded properly.
* I don't know if this is the reason for timeout but I think successful booting 
is just by accident because the register pointer to global data is not yet set 
(i2c and timer use global data). I tried to setup global data earlier but up to 
now without success. Even if successful I think the number of friends for such a 
deep change is limited...
Not to be misunderstood: This is no criticism to anybody. The experince that a 
design does not fit for a (corner case) request I have had in several projects :)
> Or perhaps someone will come up with a more clever idea!
> 
My suggestion: 

* Soon: Clean up the patch already sent as reviewed and leave i2c TWL4030 shut-
up out. As fallback for old boards leave x-load in oe meta-gumstix  (maybe just 
create a different machine configuration).
* Later: Think about a 'more probing' approach as suggested in this thread (or a 
combination e.g if revision 1 detected, check size of SDRAM and with the result 
deciding if this is a revision 1/0).

Anyway: The informations regarding SDRAM I copied from x-load. Since this is a 
bit fishing in the dark: Are there documents from gumstix available, describing 
which memories were used (with which revision :) or are these secret IP?

Regards

Andreas

  reply	other threads:[~2011-12-21  1:00 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-15 14:34 [U-Boot] [PATCH] overo: add SPL support Andreas Müller
2011-12-15 18:32 ` Tom Rini
2011-12-15 21:12   ` Andreas Müller
2011-12-20  1:03     ` Andreas Müller
2011-12-20  1:08       ` Tom Rini
2011-12-20  1:15         ` Andreas Müller
2011-12-20  5:43         ` Wolfgang Denk
2011-12-20  5:45           ` Tom Rini
2011-12-20 11:41             ` Wolfgang Denk
2011-12-20 11:53               ` Andreas Müller
2011-12-20 12:06                 ` Wolfgang Denk
2011-12-20 12:39                   ` Andreas Müller
2011-12-20 16:36                     ` Aneesh V
2011-12-20 21:17                     ` Wolfgang Denk
2011-12-20 12:20                 ` Igor Grinberg
2011-12-20 13:55                   ` Steve Sakoman
2011-12-21  1:00                     ` Andreas Müller [this message]
  -- strict thread matches above, loose matches on Subject: below --
2011-12-14 16:27 Andreas Müller
2011-12-14 17:24 ` Steve Sakoman
2011-12-14 23:00   ` Andreas Müller

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=201112210200.22049.schnitzeltony@gmx.de \
    --to=schnitzeltony@gmx.de \
    --cc=u-boot@lists.denx.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox