On Thu, Oct 07, 2010 at 09:20:58AM -0700, Stefan Reinauer wrote:
> Acked-by: Stefan Reinauer <[email protected]>

Thanks, r5918.


> > +   dev = pci_locate_device(PCI_ID(PCI_VENDOR_ID_INTEL,
> > +                                  PCI_DEVICE_ID_INTEL_82371AB_ISA), 0);
> > +
> Does it make sense to use pci_locate_device_on_bus() here instead? Well,
> maybe not because we start on bus 0 and the device is always there.
> Maybe it's kind of too philosophical to distinguish.

Hm, dunno. Maybe it saves some microseconds during boot or makes the
image slightly smaller? In that case it might make sense, but otherwise
it probably doesn't matter much.


Uwe.
-- 
http://hermann-uwe.de     | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org

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

Reply via email to