Comment 4 for bug 837235

Revision history for this message
Ricardo Salveti (rsalveti) wrote :

Comments from Aneesh V:
"""
ES2.0 had some hw issues that makes it the most difficult w.r.to SDRAM
init.

Also I am thinking of making CONFIG_SYS_EMIF_PRECALCULATED_TIMING_REGS
the default config if the automatic detection is not quite stable.

> Hm, maybe I'm missing something here, but I also tried with all yours
> SPL patches that went upstream and had the same result.

I will test again. But I will not be surprised if the issue happens on
one device but not on another. The hw issues were in the analog part
and may depend on the particular device too.

Please also note that the automatic detection increases SPL size
considerably. This is not a problem now. But in future, if more
features have to be added to SPL, we may run out of SRAM space.
"""

So it seems making CONFIG_SYS_EMIF_PRECALCULATED_TIMING_REGS a default option is indeed a valid fix. Guess we could wait at least one week before Beta 2, and if Aneesh is not able to fix it upstream, we should just enable this option.