[coreboot] Patch merged into coreboot/master: 37ef52d libpayload: Correct a constant used for scanning for USB controllers.

gerrit at coreboot.org gerrit at coreboot.org
Tue Feb 26 07:39:07 CET 2013


the following patch was just integrated into master:
commit 37ef52d44b3d527d6a4fb84eaffa4260b7521d4a
Author: Gabe Black <gabeblack at google.com>
Date:   Wed Nov 21 01:52:27 2012 -0800

    libpayload: Correct a constant used for scanning for USB controllers.
    
    When checking to see if a PCI device exists at a particular bus/dev/func,
    libpayload was checking the vendor and device id fields together against a 16
    bit 0xffff. The two fields together are 32 bits, however, so the check was
    never true, and all dev/func combinations on a particular bus would be
    checked. That was slightly wasteful, but had relatively small impact.
    
    Change-Id: Iad537295c33083243940b18e7a99af92857e1ef2
    Signed-off-by: Gabe Black <gabeblack at google.com>
    Reviewed-on: http://review.coreboot.org/2521
    Tested-by: build bot (Jenkins)
    Reviewed-by: Dave Frodin <dave.frodin at se-eng.com>
    Reviewed-by: Anton Kochkov <anton.kochkov at gmail.com>

Build-Tested: build bot (Jenkins) at Tue Feb 26 02:35:54 2013, giving +1
Reviewed-By: Anton Kochkov <anton.kochkov at gmail.com> at Tue Feb 26 07:39:06 2013, giving +2
See http://review.coreboot.org/2521 for details.

-gerrit



More information about the coreboot mailing list