[coreboot] Patch for enableing 440BX NB to use large memory modules

2010-04-26 Thread Anders Jenbo
440BX can partially use memory larger then what is supported by seeing them as being a max size supported module. Single sided 512MB (what I used to test), becomes a 128MB. A asymmetric 384MB double sided modules, with 256MB on one side and 128MB on the other. Becomes two sides with 128MB (a 256MB

Re: [coreboot] Patch for enableing 440BX NB to use large memory modules

2010-04-26 Thread Paul Menzel
Am Montag, den 26.04.2010, 03:01 +0200 schrieb Anders Jenbo: > 440BX can partially use memory larger then what is supported by seeing > them as being a max size supported module. > > Single sided 512MB (what I used to test), becomes a 128MB. > A asymmetric 384MB double sided modules, with 256MB on

Re: [coreboot] Patch for enableing 440BX NB to use large memory modules

2010-04-26 Thread Stefan Reinauer
On 4/26/10 3:01 AM, Anders Jenbo wrote: > 440BX can partially use memory larger then what is supported by seeing > them as being a max size supported module. > > Single sided 512MB (what I used to test), becomes a 128MB. > A asymmetric 384MB double sided modules, with 256MB on one side and > 128MB

Re: [coreboot] Patch for enableing 440BX NB to use large memory modules

2010-04-26 Thread Anders Jenbo
> > Single sided 512MB (what I used to test), becomes a 128MB. > > A asymmetric 384MB double sided modules, with 256MB on one side > and > > 128MB on the other. Becomes two sides with 128MB (a 256MB module). > > This is also enabled in at least some BIOSs (GA-BXC). > > > > The patch also cleans up

Re: [coreboot] Patch for enableing 440BX NB to use large memory modules

2010-04-26 Thread Stefan Reinauer
On 4/26/10 4:24 PM, Anders Jenbo wrote: > Stefan, since you acked the patch, should I still correct the typos that Paul > pointed out and split it in to a patch, one with the added memory support and > one with the cleaned white space? > That would be awesome! -- coresystems GmbH • Brahms

Re: [coreboot] Patch for enableing 440BX NB to use large memory modules

2010-04-26 Thread Peter Stuge
Anders Jenbo wrote: > should I still correct the typos that Paul pointed out and split it > in to a patch, one with the added memory support and one with the > cleaned white space? Yes please. It's not only important to separate whitespace to make review as easy as possible, but it is also very im

Re: [coreboot] Patch for enableing 440BX NB to use large memory modules

2010-04-26 Thread Anders Jenbo
man, 26 04 2010 kl. 16:53 +0200, skrev Stefan Reinauer: > On 4/26/10 4:24 PM, Anders Jenbo wrote: > > > Stefan, since you acked the patch, should I still correct the typos that > > Paul pointed out and split it in to a patch, one with the added memory > > support and one with the cleaned white s