[LinuxBIOS] [PATCH] 440bx raminit changes
Peter Stuge
stuge-linuxbios at cdy.org
Tue May 22 16:08:19 CEST 2007
On Tue, May 22, 2007 at 02:23:30PM +0100, Ceri wrote:
> Comments pls?
Have you had a chance to test it?
> + NBXCFG, 0x00000000, 0xff00a00c,PCI_REGISTER_32BIT,
[..]
> + FDHC, 0x00, 0x00, PCI_REGISTER_8BIT,
> + for (i = 0; i < max; i += 4) {
> + switch(register_values[i+3]){
> + case PCI_REGISTER_8BIT: reg = pci_read_config8(ctrl->d0, register_values[i]); break;
> + case PCI_REGISTER_16BIT: reg = pci_read_config16(ctrl->d0, register_values[i]);break;
> + case PCI_REGISTER_32BIT: reg = pci_read_config32(ctrl->d0, register_values[i]);break;
> + default: print_debug("Invalid PCI register size in table\n");
> + }
> +
Please use more lines and the kernel switch style.
switch () {
case ..REG_8:
reg = ...
break;
case ..REG_16:
..
break;
}
Maybe even use different temporary storage for different lengths.
//Peter
More information about the coreboot
mailing list