From 17b61437bbfebad98beb18c0a45abd069b747fd0 Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Wed, 11 Jan 2017 10:10:17 -0700 Subject: [PATCH] UPSTREAM: sb/intel/ibexpeak: Update debug code to match other chips Other chips dump tco_status here if it wasn't handled, which makes sense. tco_sts can't be zero here, because the call would have already returned if it were. Also, dump_tco_status wouldn't print anything if tco_sts were zero. This will still only print the debug information if DEBUG_SMI is enabled in Kconfig, so in general, this change won't have much of an effect on anything. BUG=none BRANCH=none TEST=none Change-Id: Icfe18802dfe054409e3e0ca111a3a0c272ab2a2d Signed-off-by: Patrick Georgi Original-Commit-Id: 3e3b858888d735b45c2868fc50f6de53ca0f868a Original-Found-by: Coverity Scan #1229598 Original-Change-Id: Id2c69a16817ba18dfa051f514138fbc04a2f7bee Original-Signed-off-by: Martin Roth Original-Reviewed-on: https://review.coreboot.org/18101 Original-Tested-by: build bot (Jenkins) Original-Reviewed-by: Paul Menzel Original-Reviewed-by: Nico Huber Reviewed-on: https://chromium-review.googlesource.com/428256 Commit-Ready: Aaron Durbin --- src/southbridge/intel/ibexpeak/smihandler.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/southbridge/intel/ibexpeak/smihandler.c b/src/southbridge/intel/ibexpeak/smihandler.c index ec91bdf3fc..a0b963ea3d 100644 --- a/src/southbridge/intel/ibexpeak/smihandler.c +++ b/src/southbridge/intel/ibexpeak/smihandler.c @@ -709,7 +709,7 @@ static void southbridge_smi_tco(void) } else if (tco_sts & (1 << 3)) { /* TIMEOUT */ /* Handle TCO timeout */ printk(BIOS_DEBUG, "TCO Timeout.\n"); - } else if (!tco_sts) { + } else { dump_tco_status(tco_sts); } }