On Sat, Oct 18, 2008 at 2:48 PM, Rudolf Marek <[EMAIL PROTECTED]> wrote:

> Hi all,
>
> I think one _must_ disable the decoders first by disabling space in 
> PCI_COMMAND
> register. Check u-boot how it does:
> http://git.denx.de/cgi-bin/gitweb.cgi?p=u-boot.git;a=blob;f=drivers/pci/pci_auto.c;hb=HEAD
>
> It first disable the cmdstat = (cmdstat & ~(PCI_COMMAND_IO |
> PCI_COMMAND_MEMORY)) | PCI_COMMAND_MASTER;
>

What you are saying is true. The issue in this case is that the VGA
BIOS is doing the wrong thing, and coreboot has to accommodate the VGA
BIOS incorrect behaviour.

It is not something we can fix. coreboot has to tolerate the improper
behaviour of PCI expansion ROMs and VGA BIOS.

ron

--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to