[U-Boot] [PATCHv2 2/3] OMAP3: SDRC: Introduce Numonyx DDR type
Enric Balletbò i Serra
eballetbo at gmail.com
Sun Oct 10 11:36:42 CEST 2010
Hi Wolfgang
2010/10/10 Wolfgang Denk <wd at denx.de>:
> Dear Enric Balletbo i Serra,
>
> In message <1286699659-2952-3-git-send-email-eballetbo at gmail.com> you wrote:
>> Introduce Numonyx DDR timings and provide CONFIG_OMAP3_NUMONYX_DDR
>> config options to allow for platform files to setup their timings.
>>
>> Signed-off-by: Enric Balletbo i Serra <eballetbo at gmail.com>
>> ---
>> arch/arm/include/asm/arch-omap3/mem.h | 42 +++++++++++++++++++++++++++++++++
>> 1 files changed, 42 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/include/asm/arch-omap3/mem.h b/arch/arm/include/asm/arch-omap3/mem.h
>> index a78cf9f..d574f89 100644
>> --- a/arch/arm/include/asm/arch-omap3/mem.h
>> +++ b/arch/arm/include/asm/arch-omap3/mem.h
>> @@ -128,6 +128,44 @@ enum {
>> (MICRON_XSR_165 << 0) | (MICRON_TXP_165 << 8) | \
>> (MICRON_TWTR_165 << 16))
>>
>> +/* NUMONYX part of IGEP v2 (165MHz optimized) 6.06ns
>> + * ACTIMA
>
> Incorrect multiline comment style. Please break into:
>
> /*
> * NUMONYX part ...
I'll fix in version 3
>
>
>> + * TDAL = Twr/Tck + Trp/tck = 15/6 + 18/6 = 2.5 + 3 = 5.5 -> 6
>> + * TDPL (Twr) = 15/6 = 2.5 -> 3
>> + * TRRD = 12/6 = 2
>> + * TRCD = 22.5/6 = 3.75 -> 4
>> + * TRP = 18/6 = 3
>> + * TRAS = 42/6 = 7
>> + * TRC = 60/6 = 10
>> + * TRFC = 140/6 = 23.3 -> 24
>> + * ACTIMB
>> + * TWTR = 2
>> + * TCKE = 2
>> + * TXSR = 200/6 = 33.3 -> 34
>> + * TXP = 1.0 + 1.1 = 2.1 -> 3
>> + */
>> +#define NUMONYX_TDAL_165 6
>> +#define NUMONYX_TDPL_165 3
>> +#define NUMONYX_TRRD_165 2
>> +#define NUMONYX_TRCD_165 4
>> +#define NUMONYX_TRP_165 3
>> +#define NUMONYX_TRAS_165 7
>> +#define NUMONYX_TRC_165 10
>> +#define NUMONYX_TRFC_165 24
>> +#define NUMONYX_V_ACTIMA_165 ((NUMONYX_TRFC_165 << 27) | \
>> + (NUMONYX_TRC_165 << 22) | (NUMONYX_TRAS_165 << 18) | \
>> + (NUMONYX_TRP_165 << 15) | (NUMONYX_TRCD_165 << 12) | \
>> + (NUMONYX_TRRD_165 << 9) | (NUMONYX_TDPL_165 << 6) | \
>> + (NUMONYX_TDAL_165))
>> +
>> +#define NUMONYX_TWTR_165 2
>> +#define NUMONYX_TCKE_165 2
>> +#define NUMONYX_TXP_165 3
>> +#define NUMONYX_XSR_165 34
>> +#define NUMONYX_V_ACTIMB_165 ((NUMONYX_TCKE_165 << 12) | \
>> + (NUMONYX_XSR_165 << 0) | (NUMONYX_TXP_165 << 8) | \
>> + (NUMONYX_TWTR_165 << 16))
>> +
>> #ifdef CONFIG_OMAP3_INFINEON_DDR
>> #define V_ACTIMA_165 INFINEON_V_ACTIMA_165
>> #define V_ACTIMB_165 INFINEON_V_ACTIMB_165
>> @@ -136,6 +174,10 @@ enum {
>> #define V_ACTIMA_165 MICRON_V_ACTIMA_165
>> #define V_ACTIMB_165 MICRON_V_ACTIMB_165
>> #endif
>> +#ifdef CONFIG_OMAP3_NUMONYX_DDR
>> +#define V_ACTIMA_165 NUMONYX_V_ACTIMA_165
>> +#define V_ACTIMB_165 NUMONYX_V_ACTIMB_165
>> +#endif
>
> You are using exactly the same settings as for the
> #ifdef CONFIG_OMAP3_MICRON_DDR above. Why do you repeat the lines
> then? Rather use a "#if defined(...) || defined(...)"
I'm not agree the timmings are not the same, here a list of differences
MICRON_TRCD_165(3) != NUMONYX_TRCD_165(4)
MICRON_TRFC_165(21) != NUMONYX_TRFC_165(24)
MICRON_TWTR_165(1) != NUMONYX_TWTR_165(2)
MICRON_TCKE_165(1) != NUMONYX_TCKE_165(2)
MICRON_XSR_165(23) != NUMONYX_XSR_165(34)
MICRON_TXP_165(5) != NUMONYX_TXP_165(3)
I'll send the version 3 with all your comments except this.
Cheers,
Enric
More information about the U-Boot
mailing list