[PATCH 2/3] arm: mach-omap2: sdram-nokia: add 200 MHz memory timings info

[Aaro Koskinen]

Hi,

On Fri, 9 Dec 2011, Tony Lindgren wrote:
> * Aaro Koskinen <aaro.koskinen at nokia.com> [111209 06:33]:
>> From: Igor Dmitriev <ext-dmitriev.igor at nokia.com>
>>
>> Add memory timing info regarding the 200 MHz memory in sdram-nokia.
>>
>> Signed-off-by: Igor Dmitriev <ext-dmitriev.igor at nokia.com>
>> Signed-off-by: Eduardo Valentin <eduardo.valentin at nokia.com>
>> Signed-off-by: Aaro Koskinen <aaro.koskinen at nokia.com>
>> ---
>>  arch/arm/mach-omap2/sdram-nokia.c |   25 ++++++++++++++++++++++++-
>>  1 files changed, 24 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/sdram-nokia.c b/arch/arm/mach-omap2/sdram-nokia.c
>> index 14caa22..b394a0e 100644
>> --- a/arch/arm/mach-omap2/sdram-nokia.c
>> +++ b/arch/arm/mach-omap2/sdram-nokia.c
>> @@ -1,7 +1,7 @@
>>  /*
>>   * SDRC register values for Nokia boards
>>   *
>> - * Copyright (C) 2008, 2010 Nokia Corporation
>> + * Copyright (C) 2008, 2010-2011 Nokia Corporation
>>   *
>>   * Lauri Leukkunen <lauri.leukkunen at nokia.com>
>>   *
>> @@ -107,14 +107,37 @@ static const struct sdram_timings nokia_195dot2mhz_timings[] = {
>>  	},
>>  };
>>
>> +static const struct sdram_timings nokia_200mhz_timings[] = {
>> +	{
>> +		.casl = 3,
>> +		.tDAL = 30000,
>> +		.tDPL = 15000,
>> +		.tRRD = 10000,
>> +		.tRCD = 20000,
>> +		.tRP = 15000,
>> +		.tRAS = 40000,
>> +		.tRC = 55000,
>> +		.tRFC = 140000,
>> +		.tXSR = 200000,
>> +
>> +		.tREF = 7800,
>> +
>> +		.tXP = 2,
>> +		.tCKE = 4,
>> +		.tWTR = 2
>> +	},
>> +};
>> +
>>  static const struct {
>>  	long rate;
>>  	struct sdram_timings const *data;
>>  } nokia_timings[] = {
>>  	{ 83000000, nokia_166mhz_timings },
>>  	{ 97600000, nokia_97dot6mhz_timings },
>> +	{ 100000000, nokia_200mhz_timings },
>
> Is that a typo above? Should it say nokia_100mhz_timings?

No, it's not a typo, same data is used. (Like 83/166 already in the table)

>>  	{ 166000000, nokia_166mhz_timings },
>>  	{ 195200000, nokia_195dot2mhz_timings },
>> +	{ 200000000, nokia_200mhz_timings },
>>  };
>>  static struct omap_sdrc_params nokia_sdrc_params[ARRAY_SIZE(nokia_timings) + 1];
>
> Regards,
>
> Tony

A.