summaryrefslogtreecommitdiff
path: root/src/soc/intel/common/block
diff options
context:
space:
mode:
authorTim Wawrzynczak <twawrzynczak@chromium.org>2019-08-08 11:21:51 -0600
committerPatrick Georgi <pgeorgi@google.com>2019-08-21 09:34:17 +0000
commit44348185d97036012353c8d53211c68c2444b36d (patch)
tree4186a74b1312bcc7429cd45b89205d7121448b30 /src/soc/intel/common/block
parent220ac049ba3898911a7af990df82b56768a47aaf (diff)
downloadcoreboot-44348185d97036012353c8d53211c68c2444b36d.tar.xz
soc/intel/common/smm: Add missing printk statement
SMI trap handler was missing a printk statement, which caused Coverity to flag "data &= mask;" as a redundant operation. Change-Id: I71da74e5e08e7d7e6d61c1925db19324efd73f0a Found-by: Coverity CID 1381621 Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/34797 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin Roth <martinroth@google.com> Reviewed-by: Subrata Banik <subrata.banik@intel.com>
Diffstat (limited to 'src/soc/intel/common/block')
-rw-r--r--src/soc/intel/common/block/smm/smitraphandler.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/soc/intel/common/block/smm/smitraphandler.c b/src/soc/intel/common/block/smm/smitraphandler.c
index bfa984651e..974c4897a8 100644
--- a/src/soc/intel/common/block/smm/smitraphandler.c
+++ b/src/soc/intel/common/block/smm/smitraphandler.c
@@ -108,6 +108,7 @@ void smihandler_southbridge_monitor(
/* Trapped write data */
data = pcr_read32(PID_PSTH, PCR_PSTH_TRPD);
data &= mask;
+ printk(BIOS_DEBUG, " iotrap read data = 0x%08x\n", data);
}
}