[coreboot-gerrit] Patch merged into coreboot/master: PCIe : Adding some error/not-null condition checking

gerrit at coreboot.org gerrit at coreboot.org
Wed Jun 17 11:55:31 CEST 2015


the following patch was just integrated into master:
commit 0cd0d28f0a4a246c0901a5e1f4685205bb683ebd
Author: Pratik Prajapati <pratikkumar.v.prajapati at intel.com>
Date:   Tue Jun 9 12:06:20 2015 -0700

    PCIe : Adding some error/not-null condition checking
    
    This patch checks for following conditions
    (1) while enabling LTR, if PCI_CAP_ID_PCIE is don't found
        then don't enable LTR.
    (2)
        2.1) set_L1_ss_latency is member if ops_pci, which could be NULL.
    	 so confirm ops_pci is not NULL before calling its member function.
        2.2) if PCI_CAP_ID_PCIE is not found, then don't try to set latency.
    
    BUG=none
    BRANCH=none
    TEST=build and boot coreboot with L1 substate enabled on sklrvp3.
    
    Change-Id: I31965266f81f2a12ee719f69ed9a20b096c8b315
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 3592a7c974186f2f1113cb002db4632c8f1ab181
    Original-Change-Id: I95041490f9fafd2d6f57a8279614ccb7994a1447
    Original-Signed-off-by: Pratik Prajapati <pratikkumar.v.prajapati at intel.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/276423
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Original-Commit-Queue: Naveenkrishna Ch <naveenkrishna.ch at intel.com>
    Original-Tested-by: Naveenkrishna Ch <naveenkrishna.ch at intel.com>
    Reviewed-on: http://review.coreboot.org/10559
    Tested-by: build bot (Jenkins)
    Reviewed-by: Kyösti Mälkki <kyosti.malkki at gmail.com>


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

-gerrit



More information about the coreboot-gerrit mailing list