[coreboot-gerrit] Patch merged into coreboot/master: 05f3b11 AMD Inagua: PlatformGnbPcie.c: Allocate exact needed size for buffer

gerrit at coreboot.org gerrit at coreboot.org
Thu May 16 06:06:51 CEST 2013


the following patch was just integrated into master:
commit 05f3b117dd44776ed17bc57318f260766039b7e8
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Tue May 14 09:28:26 2013 +0200

    AMD Inagua: PlatformGnbPcie.c: Allocate exact needed size for buffer
    
    The following commit
    
        commit d0790694b0a66353e5531715648ddaa1a6d577cb
        Author: Kerry Sheh <shekairui at gmail.com>
        Date:   Thu Jan 19 13:18:37 2012 +0800
    
            Inagua: Inagua GNB ddi lanes and pcie lanes config update
    
            Reviewed-on: http://review.coreboot.org/544
    
    assigns lanes 4 and 5 to PCI device number 4, but does not
    adapt the rest of the code.
    
    After the commit above, the array `PortList []` only has four
    elements, but the buffer size `AllocHeapParams.RequestedBufferSize`
    is set to a size as it still has five elements.
    
    Correct that by setting the size for four array elements.
    
    [1] http://review.coreboot.org/#/c/3239/3/src/mainboard/amd/inagua/PlatformGnbPcie.c
    
    Change-Id: I3ff07f308ffd417d2bf73117eda9da2a1a05f199
    Reported-by: Bruce Griffith <Bruce.Griffith at se-eng.com>
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-on: http://review.coreboot.org/3246
    Tested-by: build bot (Jenkins)
    Reviewed-by: Bruce Griffith <Bruce.Griffith at se-eng.com>


See http://review.coreboot.org/3246 for details.

-gerrit



More information about the coreboot-gerrit mailing list