Hello Lijian Zhao,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/33960
to review the following change.
Change subject: soc/intel/icelake: Fix outb order ......................................................................
soc/intel/icelake: Fix outb order
Similar to CB:33940, fix outb orders.
Change-Id: I1d35235abc7e02e6058f07809b738635861cc9e4 Signed-off-by: Lijian Zhao lijian.zhao@intel.com --- M src/soc/intel/icelake/espi.c 1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/60/33960/1
diff --git a/src/soc/intel/icelake/espi.c b/src/soc/intel/icelake/espi.c index 9ca0c7c..8ab909d 100644 --- a/src/soc/intel/icelake/espi.c +++ b/src/soc/intel/icelake/espi.c @@ -194,10 +194,10 @@
/* Setup NMI on errors, disable SERR */ reg8 = (inb(0x61)) & 0xf0; - outb(0x61, (reg8 | (1 << 2))); + outb((reg8 | (1 << 2)), 0x61);
/* Disable NMI sources */ - outb(0x70, (1 << 7)); + outb((1 << 7), 0x70); };
static void clock_gate_8254(const struct device *dev)