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
next prev parent 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