[U-Boot] [PATCH] overo: add SPL support
Andreas Müller
schnitzeltony at gmx.de
Tue Dec 20 12:53:46 CET 2011
On Tuesday, December 20, 2011 12:41:08 PM you wrote:
> Dear Tom Rini,
>
> In message <CA+M6bXn_ZBqA8rosE4L4O+4Eu+grAhWgCZ=u=2Nomb6WcyL6-
w at mail.gmail.com> you wrote:
> > > I guess you really, really must use i2c before relocation? =A0If
> > > possible, this should be avoided in the first place.
> >
> > Yes, board rev detection to know how to configure SDRAM.
>
> I don't consider this a valid reason (reading the SPD EEPROM would be
> such a reason). In almost all other cases it should be suffucient to
> configure the maximum number of memory banks and the maximum size of
> the memory banks and then use get_ram_size() to determine the actual
> amount of memory and to correctly initialize the memory controller.
>
> Note that I don't insist on any changes to existing code here. This
> is just a recommendation which you may (or may not) consider for any
> future ports / implementations.
>
>
Dear Wolfgang Denk,
I agree to your concerns but - as I understood Steve Sakoman - here the
situation is slightly different:
At elder overo boards TWL4030 RTC irq is connected to gpio112. Unfortunately
this pin is also used for binary revision detection. Therefore we need to send
'shut-up' to TWL4030 via i2c to avoid reading wrong revision. In SPL this must
be done *before* SDRAM (timing) is set up, because the type of SDRAM is
revision dependent.
Hope this helps
Andreas
More information about the U-Boot
mailing list