[SeaBIOS] [PATCH v3 1/6] pci: add two-pass pci initialization code

Gerd Hoffmann kraxel at redhat.com
Fri Jul 8 10:00:48 CEST 2011


   Hi,

> 0x1 would make me more comfortable.
> I am guessing index + shift is almost always<  31,
> but still.

fixed.

> This actually gives a bigger value than
> necessary if the input is a power of 2 already.

fixed.

>> +        struct pci_bus *s = busses + dev->secondary_bus;
>
>
> would be nice to move to top of the scope.

that is actually seabios code style ;)

>> +    if (pci_bios_init_root_regions(start, end) != 0) {
>> +        dprintf(1, "PCI: out of address space\n");
>> +        /* Hmm, what do now? */
>
> stall?

patch #2 makes that a panic.

cheers,
   Gerd



More information about the SeaBIOS mailing list