[coreboot] [PATCH] Enable maximum decoded range for VT8237*
Stefan Reinauer
stepan at coresystems.de
Tue Apr 20 23:59:40 CEST 2010
On 4/20/10 11:47 PM, Rudolf Marek wrote:
Does it work better like this?
> /* Power management controller */
> dev = pci_locate_device(PCI_ID(PCI_VENDOR_ID_VIA,
> PCI_DEVICE_ID_VIA_VT8237R_LPC), 0);
>
> if (dev == PCI_DEV_INVALID) {
> /* Power management controller */
> dev = pci_locate_device(PCI_ID(PCI_VENDOR_ID_VIA,
> PCI_DEVICE_ID_VIA_VT8237S_LPC), 0);
} // close scope early
>
> if (dev == PCI_DEV_INVALID)
> return;
> pci_write_config8(dev, 0x41, 0x7f);
If not, we should make a test case and send it to eric.
....
> +static void bootblock_southbridge_init(void) {
>
functions should have their opening bracket on the next line
> +
> + /* ROM decode last 8MB FF800000 - FFFFFFFF on VT8237S/VT8237A */
> + /* ROM decode last 4MB FFC00000 - FFFFFFFF on VT8237R */
> +
> + /* get SB on 0:11.0, because pci_locate_device construct similar in
> + enable_rom_decode() did not worked for some reason */
> + pci_write_config8(PCI_DEV(0,0x11,0), 0x41, 0x7f);
> +}
>
Is the device always on 0, 0x11, 0? If not, we should add a more
explicit comment saying TODO and explaining why the broken code was
checked in.
More information about the coreboot
mailing list