[LinuxBIOS] [PATCH] Add Intel i810, i82801aa (ICH) and Asus MEW-VM support
Peter Stuge
stuge-linuxbios at cdy.org
Sat May 26 16:21:03 CEST 2007
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 long)(translate_i810_to_mb[drp_value]);
> > + /* Now do the same for DIMM 1 */
> > + drp_value = pci_read_config8(mc_dev, DRP) >> 4;
>
> Just a minor comment:
>
> drp_value >>= 4
>
> should work, too? No need to read the same value twice from the
> register...
First &0xf, then >> 4 so not re-reading will always give 0.
Perhaps move the & to where it's used.
//Peter
More information about the coreboot
mailing list