Re: [coreboot] Fwd: SimNOW VGA int 1a

[ron minnich]

On Sat, Oct 18, 2008 at 2:48 PM, Rudolf Marek  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_...
>
> 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