[PATCH] v2: i810: Add support for multiple DIMMs

[Uwe Hermann]

See patch.

This is the result of many hours of debugging and testing; parts of the
patch are based on a previous version by Elia Yehuda 
(hence both Signed-off-by's).

Remaining TODO list for later:

- Correctly determine/set CAS latency, use it in do_ram_command().

- Commit the BUFF_SC patch or an improved version of it; I'll gather
   some more sample values so we can findout whether or not those numbers
   are the same for all boards.

- Fix onboard VGA.

Anything I forgot?

Uwe.

[Peter Stuge]

Uwe Hermann wrote:
> i810: Add support for multiple DIMMs, both single-sided and double-sided,
> as well as most (all?) combinations thereof.
>
> Drop some unused code, the unused row_offset variable, and obsolete comments.
> Also, fix a typo (thanks to Stefan Reinauer for noticing).
>
> This is tested on the MSI MS-6178 with a number of different DIMM
> combinations and so far all of them worked fine.
>
> Signed-off-by: Elia Yehuda 
> Signed-off-by: Uwe Hermann

Acked-by: Peter Stuge

[Uwe Hermann]

On Fri, Nov 21, 2008 at 05:04:26PM +0100, Peter Stuge wrote:
> Uwe Hermann wrote:
> > i810: Add support for multiple DIMMs, both single-sided and double-sided,
> > as well as most (all?) combinations thereof.
> >
> > Drop some unused code, the unused row_offset variable, and obsolete comments.
> > Also, fix a typo (thanks to Stefan Reinauer for noticing).
> >
> > This is tested on the MSI MS-6178 with a number of different DIMM
> > combinations and so far all of them worked fine.
> >
> > Signed-off-by: Elia Yehuda 
> > Signed-off-by: Uwe Hermann
>
> Acked-by: Peter Stuge

Thanks, 3765.

Uwe.

[Elia Yehuda]

Really nice (and CLEAN...) patch :)

i do have two questions -

1. in sdram_set_spd_registers(),  what is the correct order of the 3
following commands:

spd_set_dram_size();
set_dram_buffer_strength();
set_dram_timing();

since ive noticed its not the same as in other related intel northbridges
(not that it counts for something...).
i would assume timing & strength should be called before setting the size.
no?

2. also, in do_ram_command(), i have no idea how you got those addr_offset
from... can you please elaborate or link me to the proper page on the
datasheet (or similar)?

thanks again,

Elia.

On Fri, Nov 21, 2008 at 7:15 PM, Uwe Hermann  wrote:
> On Fri, Nov 21, 2008 at 05:04:26PM +0100, Peter Stuge wrote:
> > Uwe Hermann wrote:
> > > i810: Add support for multiple DIMMs, both single-sided and
>
> double-sided,
> > > as well as most (all?) combinations thereof.
> > >
> > > Drop some unused code, the unused row_offset variable, and obsolete
>
> comments.
> > > Also, fix a typo (thanks to Stefan Reinauer for noticing).
> > >
> > > This is tested on the MSI MS-6178 with a number of different DIMM
> > > combinations and so far all of them worked fine.
> > >
> > > Signed-off-by: Elia Yehuda 
> > > Signed-off-by: Uwe Hermann
> >
> > Acked-by: Peter Stuge
>
> Thanks, 3765.
> Uwe.
>
> http://www.hermann-uwe.de  | http://www.holsham-traders.de
> http://www.crazy-hacks.org | http://www.unmaintained-free-software.org