summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2022-12-03 19:35:53 +0200
committerKyösti Mälkki <kyosti.malkki@gmail.com>2022-12-07 04:50:40 +0000
commita0720431b4c0685634fd1ccc45938e9b3cc40544 (patch)
tree36598141dce48d44a948fed2ded8d75534b06337 /src
parent7a9bd2b2e4a2c0be9f9ecc04e47684639973baef (diff)
downloadcoreboot-a0720431b4c0685634fd1ccc45938e9b3cc40544.tar.gz
coreboot-a0720431b4c0685634fd1ccc45938e9b3cc40544.tar.bz2
coreboot-a0720431b4c0685634fd1ccc45938e9b3cc40544.zip
sb,soc/intel: Fix SMI handler IO trap data mask
Shift is done in multiples of 8 (1 << 3) bits. It was fixed already for i82801ix/jx. Change-Id: I5e1c2b3bf4ba68f34eb43e59fe783d5cd6e0a39a Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/70361 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src')
-rw-r--r--src/soc/intel/broadwell/pch/smihandler.c2
-rw-r--r--src/soc/intel/common/block/smm/smitraphandler.c2
-rw-r--r--src/southbridge/intel/bd82x6x/smihandler.c2
-rw-r--r--src/southbridge/intel/i82801gx/smihandler.c2
-rw-r--r--src/southbridge/intel/ibexpeak/smihandler.c2
-rw-r--r--src/southbridge/intel/lynxpoint/smihandler.c2
6 files changed, 6 insertions, 6 deletions
diff --git a/src/soc/intel/broadwell/pch/smihandler.c b/src/soc/intel/broadwell/pch/smihandler.c
index a8e206783d10..b9d5e5048201 100644
--- a/src/soc/intel/broadwell/pch/smihandler.c
+++ b/src/soc/intel/broadwell/pch/smihandler.c
@@ -429,7 +429,7 @@ static void southbridge_smi_monitor(void)
trap_cycle = RCBA32(0x1e10);
for (i = 16; i < 20; i++) {
if (trap_cycle & (1 << i))
- mask |= (0xff << ((i - 16) << 2));
+ mask |= (0xff << ((i - 16) << 3));
}
/* IOTRAP(3) SMI function call */
diff --git a/src/soc/intel/common/block/smm/smitraphandler.c b/src/soc/intel/common/block/smm/smitraphandler.c
index 99825f10ba09..42bab8388b68 100644
--- a/src/soc/intel/common/block/smm/smitraphandler.c
+++ b/src/soc/intel/common/block/smm/smitraphandler.c
@@ -70,7 +70,7 @@ void smihandler_southbridge_monitor(
trap_cycle = pcr_read32(PID_PSTH, PCR_PSTH_TRPC);
for (i = 16; i < 20; i++) {
if (trap_cycle & (1 << i))
- mask |= (0xff << ((i - 16) << 2));
+ mask |= (0xff << ((i - 16) << 3));
}
/* IOTRAP(3) SMI function call */
diff --git a/src/southbridge/intel/bd82x6x/smihandler.c b/src/southbridge/intel/bd82x6x/smihandler.c
index bf03b1a51df1..351751ec4b1c 100644
--- a/src/southbridge/intel/bd82x6x/smihandler.c
+++ b/src/southbridge/intel/bd82x6x/smihandler.c
@@ -100,7 +100,7 @@ void southbridge_smi_monitor(void)
trap_cycle = RCBA32(0x1e10);
for (i=16; i<20; i++) {
if (trap_cycle & (1 << i))
- mask |= (0xff << ((i - 16) << 2));
+ mask |= (0xff << ((i - 16) << 3));
}
/* IOTRAP(3) SMI function call */
diff --git a/src/southbridge/intel/i82801gx/smihandler.c b/src/southbridge/intel/i82801gx/smihandler.c
index 67a8cf8cf40e..9fbca38e9da4 100644
--- a/src/southbridge/intel/i82801gx/smihandler.c
+++ b/src/southbridge/intel/i82801gx/smihandler.c
@@ -49,7 +49,7 @@ void southbridge_smi_monitor(void)
trap_cycle = RCBA32(0x1e10);
for (i = 16; i < 20; i++) {
if (trap_cycle & (1 << i))
- mask |= (0xff << ((i - 16) << 2));
+ mask |= (0xff << ((i - 16) << 3));
}
/* IOTRAP(3) SMI function call */
diff --git a/src/southbridge/intel/ibexpeak/smihandler.c b/src/southbridge/intel/ibexpeak/smihandler.c
index 5fb307705359..81dde3249956 100644
--- a/src/southbridge/intel/ibexpeak/smihandler.c
+++ b/src/southbridge/intel/ibexpeak/smihandler.c
@@ -85,7 +85,7 @@ void southbridge_smi_monitor(void)
trap_cycle = RCBA32(0x1e10);
for (i = 16; i < 20; i++) {
if (trap_cycle & (1 << i))
- mask |= (0xff << ((i - 16) << 2));
+ mask |= (0xff << ((i - 16) << 3));
}
/* IOTRAP(3) SMI function call */
diff --git a/src/southbridge/intel/lynxpoint/smihandler.c b/src/southbridge/intel/lynxpoint/smihandler.c
index 769cacb792fa..52c9a000d35d 100644
--- a/src/southbridge/intel/lynxpoint/smihandler.c
+++ b/src/southbridge/intel/lynxpoint/smihandler.c
@@ -393,7 +393,7 @@ static void southbridge_smi_monitor(void)
trap_cycle = RCBA32(0x1e10);
for (i = 16; i < 20; i++) {
if (trap_cycle & (1 << i))
- mask |= (0xff << ((i - 16) << 2));
+ mask |= (0xff << ((i - 16) << 3));
}
/* IOTRAP(3) SMI function call */