[SeaBIOS] [PATCH 0/7] vgabios improvements
H. Peter Anvin
hpa at zytor.com
Thu Apr 17 01:31:38 CEST 2014
On 04/16/2014 03:59 PM, Peter Stuge wrote:
> 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. :\
>
Uh... seriously, robustness is a good thing, and it is not entirely
clear that this responsibility necessarily belongs in Coreboot.
But especially if you are touching the command register anyway, the cost
is literally zero to just do it.
-hpa
More information about the SeaBIOS
mailing list