[coreboot-gerrit] Patch set updated for coreboot: soc/intel/apollolake: remove errant semicolon

Aaron Durbin (adurbin@chromium.org) gerrit at coreboot.org
Tue May 10 16:00:36 CEST 2016


Aaron Durbin (adurbin at chromium.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14755

-gerrit

commit 172c843058f44b6fd90bc1ef470a025d1d5e8077
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Tue May 10 08:43:56 2016 -0500

    soc/intel/apollolake: remove errant semicolon
    
    Remove a semicolon which shouldn't be there.
    
    Change-Id: I38f785fa13ea9fee91813f165a085ff54e1b75fb
    Found-by: Coverity
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 src/soc/intel/apollolake/lpc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/soc/intel/apollolake/lpc.c b/src/soc/intel/apollolake/lpc.c
index ce9f001..48dfb1f 100644
--- a/src/soc/intel/apollolake/lpc.c
+++ b/src/soc/intel/apollolake/lpc.c
@@ -58,7 +58,7 @@ static void lpc_init(struct device *dev)
 	scnt &= ~(SCNT_EN | SCNT_MODE);
 	if (cfg->serirq_mode == SERIRQ_QUIET)
 		scnt |= SCNT_EN;
-	else if (cfg->serirq_mode == SERIRQ_CONTINUOUS);
+	else if (cfg->serirq_mode == SERIRQ_CONTINUOUS)
 		scnt |= SCNT_EN | SCNT_MODE;
 	pci_write_config8(dev, REG_SERIRQ_CTL, scnt);
 }



More information about the coreboot-gerrit mailing list