[U-Boot] [PATCH 5/7] OMAP5: EMIF: Add support for DDR3 device

[R, Sricharan]

>> >> - ? ? if (!in_sdram)
>> >> - ? ? ? ? ? ? lpddr2_init(base, regs);
>> >> + ? ? if (!in_sdram) {
>> >> + ? ? ? ? ? ? if (omap_revision() != OMAP5432_ES1_0)
>> >> + ? ? ? ? ? ? ? ? ? ? lpddr2_init(base, regs);
>> >> + ? ? ? ? ? ? else
>> >> + ? ? ? ? ? ? ? ? ? ? ddr3_init(base, regs);
>> >> + ? ? }
>> >
>> > In omap4+ land do we have any other way of telling which family we're
>> > on? ?I ask since I'm preparing to add DDR3 support to am33xx and I'd
>> > like to switch it over to the common emif framework as well.
>> >
>>   I think the SDRAM_TYPE[31:29] EMIF_SDRAM_CONFIG
>>  register should tell the connected device at reset.
>>  I will cross confirm this on the board and tell.
>
> OK.  I can't find the omap5430 TRM quickly but on the am335x one,
> SDRAM_TYPE is 3 for DDR3, 2 for DDR2 and 1 for LPDDR1 and 0 for DDR1.

 ah, this is what is there on OMAP5.
   3 for DDR3
   4 for LPDDR2-S4,
   5 for LPDDR2-S2

Atleast DDR3 encoding is same. So we can differentiate bw DDR3 and 2
in same way.
Is the reset value set correctly on am335x according to the device connected?

Thanks,
 Sricharan