[coreboot] Patch merged into coreboot/master: f35ce49 ASRock E350M1: Remove non-existing PCI devices 12.1 and 13.1

gerrit at coreboot.org gerrit at coreboot.org
Sat Mar 2 09:48:18 CET 2013


the following patch was just integrated into master:
commit f35ce497d1bbf646e3397ba34dc350b43ac81a44
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Wed Feb 27 11:17:59 2013 +0100

    ASRock E350M1: Remove non-existing PCI devices 12.1 and 13.1
    
    Looking at the coreboot log
    
        […]
        PCI: 00:12.0 [1002/4397] enabled
        sb800_enable() PCI: Static device PCI: 00:12.1 not found, disabling it.
        sb800_enable() PCI: 00:12.2 [1002/4396] ops
        PCI: 00:12.2 [1002/4396] enabled
        sb800_enable() PCI: 00:13.0 [1002/4397] ops
        PCI: 00:13.0 [1002/4397] enabled
        sb800_enable() PCI: Static device PCI: 00:13.1 not found, disabling it.
        sb800_enable() PCI: 00:13.2 [1002/4396] ops
        PCI: 00:13.2 [1002/4396] enabled
        […]
    
    and the `lspci -tnvv` output running the proprietary vendor BIOS
    attached to the Wiki page of the ASRock E350M1 [1][2]
    
            -[0000:00]-+-00.0  1022:1510
                       +-01.0  1002:9802
                       +-01.1  1002:1314
                       +-04.0-[01]--
                       +-11.0  1002:4391
                       +-12.0  1002:4397
                       +-12.2  1002:4396
                       +-13.0  1002:4397
                       +-13.2  1002:4396
            […]
    
    both PCI devices do not exist, so remove them from `devicetree.cb`.
    
    Commit 48918f7 [3]
    
        Persimmon, Inagua: PCI devs 12.1, 13.1 (USB) don't exist, but 14.6 (GEC) does
    
    did the same for AMD Inagua and AMD Persimmon.
    
    [1] http://www.coreboot.org/ASRock_E350M1
    [2] http://www.coreboot.org/File:ASRock_E350M1_info_dump.tar.bz2
    [3] http://review.coreboot.org/2463
    
    Change-Id: Ief6de1bda093d1f29d5925985e5c3839cdded537
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-on: http://review.coreboot.org/2536
    Tested-by: build bot (Jenkins)
    Reviewed-by: Martin Roth <martin.roth at se-eng.com>

Build-Tested: build bot (Jenkins) at Wed Feb 27 11:59:36 2013, giving +1
Reviewed-By: Martin Roth <martin.roth at se-eng.com> at Sat Mar  2 02:13:44 2013, giving +2
See http://review.coreboot.org/2536 for details.

-gerrit



More information about the coreboot mailing list