Re: [LinuxBIOS] [PATCH] Add Intel i810, i82801aa (ICH) and Asus MEW-VM support

2007-05-27 Thread Corey Osgood
Just a couple quick responses, I'll get to work on changes as soon as I can coax fedora into working correctly again. minicom.cap attached, I forgot it before. On Sat, 2007-05-26 at 15:29 +0200, Uwe Hermann wrote: Also, the i82801aa is almost completely copy and pasted from the i82801ca, with

Re: [LinuxBIOS] [PATCH] Add Intel i810, i82801aa (ICH) and Asus MEW-VM support

2007-05-27 Thread joe
Poll: do we want i810 or rather i82810? For the southbridges we use the full names (e.g. i82801aa), why don't we do the same with the northbridges? (yes, that would also change i440bx to i82443bx) Hello, Yes I also agree, as far as Intel chips, it would aviod confusion to just use their

Re: [LinuxBIOS] [PATCH] Add Intel i810, i82801aa (ICH) and Asus MEW-VM support

2007-05-27 Thread Uwe Hermann
On Sun, May 27, 2007 at 04:48:51AM -0400, Corey Osgood wrote: #define I82801AA 1 Agreed, as long as no further fixup is necessary. Perhaps I'll just hold onto this patch until I can fully test out the southbridge function, or else we could commit as-is and I'll do some cleanup later. I

Re: [LinuxBIOS] [PATCH] Add Intel i810, i82801aa (ICH) and Asus MEW-VM support

2007-05-26 Thread Peter Stuge
Thanks for reviewing the patch! On Sat, May 26, 2007 at 03:29:31PM +0200, Uwe Hermann wrote: + /*First get the value for DIMM 0 */ + drp_value = pci_read_config8(mc_dev, DRP) 0xf; + /* Translate it to MB and add to tomk */ + tomk = (unsigned

Re: [LinuxBIOS] [PATCH] Add Intel i810, i82801aa (ICH) and Asus MEW-VM support

2007-05-26 Thread Uwe Hermann
On Sat, May 26, 2007 at 04:21:03PM +0200, Peter Stuge wrote: + drp_value = pci_read_config8(mc_dev, DRP) 0xf; + /* Translate it to MB and add to tomk */ + tomk = (unsigned long)(translate_i810_to_mb[drp_value]); + /* Now do the same for DIMM 1 */