[SeaBIOS] [PATCH 0/7] vgabios improvements

Peter Stuge peter at stuge.se
Thu Apr 17 00:59:28 CEST 2014

Gerd Hoffmann wrote:
> > -            pci_config_maskw(bdf, PCI_COMMAND, 0, PCI_COMMAND_MASTER);
> > +            pci_config_maskw(bdf, PCI_COMMAND, 0, PCI_COMMAND_MASTER | PCI_COMMAND_IO | PCI_COMMAND_MEMORY);
> I think that should go in.  If we need it, we better make sure it
> is enabled instead of expecting someone else to do it.

I think this reasoning is completely broken; what you are saying is
that SeaBIOS should in principle do the complete hardware init.

It is IMO broken enough that SeaBIOS does so for QEMU - please don't
push that into SeaBIOS built for coreboot, but fix the actual
problem where it exists (in coreboot), instead of making a workaround
in SeaBIOS under the guise of "we need it so we must do it ourselves"

It sounds like NIH syndrome. :\


More information about the SeaBIOS mailing list