[coreboot] [commit] r5426 - in trunk/src: cpu/amd/mtrr cpu/x86/mtrr lib mainboard/amd/dbm690t mainboard/amd/mahogany mainboard/amd/mahogany_fam10 mainboard/amd/pistachio mainboard/amd/serengeti_cheetah_fam10 ...

[Myles Watson]

On Wed, Apr 14, 2010 at 6:50 AM, Stefan Reinauer <stepan at coresystems.de>wrote:

>  On 4/14/10 2:36 PM, Myles Watson wrote:
>
>  Modified: trunk/src/lib/debug.c
>>
>> ==============================================================================
>> --- trunk/src/lib/debug.c       Wed Apr 14 12:12:23 2010        (r5425)
>> +++ trunk/src/lib/debug.c       Wed Apr 14 13:40:34 2010        (r5426)
>> @@ -33,7 +33,7 @@
>>  {
>>        device_t dev;
>>        for (dev = PCI_DEV(0, 0, 0);
>> -            dev <= PCI_DEV(0, 0x1f, 0x7); dev += PCI_DEV(0, 0, 1)) {
>> +            dev <= PCI_DEV(0xff, 0x1f, 0x7); dev += PCI_DEV(0, 0, 1)) {
>>
>
>   It looks like this slipped in.  I thought it hangs some boards to probe
> PCI busses that aren't there.
>
> Hm,... do you happen to know which ones?
>
>
>  http://www.coreboot.org/pipermail/coreboot/2009-October/053330.html

I think this is what I was remembering, so it may not be relevant.

There are about 15 places where this function is defined.

Thanks,
Myles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100414/6c995cba/attachment.html>