[coreboot-gerrit] New patch to review for coreboot: soc/intel/quark: Fix legacy GPIO reads

Lee Leahy (leroy.p.leahy@intel.com) gerrit at coreboot.org
Mon Jul 18 19:42:41 CEST 2016


Lee Leahy (leroy.p.leahy at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15732

-gerrit

commit 78b7b5dfd7a3d341dcc82fefdac931ee5fea86cf
Author: Lee Leahy <leroy.p.leahy at intel.com>
Date:   Mon Jul 18 10:39:55 2016 -0700

    soc/intel/quark: Fix legacy GPIO reads
    
    Add missing break to LEG_GPIO_REGS case to return the correct value for
    legacy GPIO reads.  Fixes coverity issue CID 1357460.
    
    TEST=Build and run on Galileo Gen2.
    
    Change-Id: I6c52e8801a32f510ac94276fe0c097850cbfde57
    Signed-off-by: Lee Leahy <leroy.p.leahy at intel.com>
---
 src/soc/intel/quark/reg_access.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/soc/intel/quark/reg_access.c b/src/soc/intel/quark/reg_access.c
index c468807..25a52ff 100644
--- a/src/soc/intel/quark/reg_access.c
+++ b/src/soc/intel/quark/reg_access.c
@@ -251,6 +251,7 @@ static uint64_t reg_read(struct reg_script_context *ctx)
 	case LEG_GPIO_REGS:
 		ctx->display_prefix = "Legacy GPIO";
 		value = reg_legacy_gpio_read(step->reg);
+		break;
 
 	case PCIE_AFE_REGS:
 		ctx->display_prefix = "PCIe AFE";



More information about the coreboot-gerrit mailing list