Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/62060 )
Change subject: util/superiotool/fintek.c: Fix F71808A hardware monitor readouts ......................................................................
util/superiotool/fintek.c: Fix F71808A hardware monitor readouts
Fix readouts from the hardware monitor on Fintek F71808A Super I/O. The HWM port is +5 to the base address stored in LDN 0x4 at index 0x60/0x61.
Referred to util/superiotool/winbond.c and the Linux kernel driver f71882fg.
Tested on a HP 500-319na (Memphis-S / IPM87-MP).
Signed-off-by: Ravi Mistry rvstry@protonmail.com Change-Id: I2b2b98c62f9305c6f4885c2ce3b1444801dcb9d5 Reviewed-on: https://review.coreboot.org/c/coreboot/+/62060 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Paul Menzel paulepanter@mailbox.org Reviewed-by: Angel Pons th3fanbus@gmail.com --- M util/superiotool/fintek.c 1 file changed, 24 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Angel Pons: Looks good to me, approved
diff --git a/util/superiotool/fintek.c b/util/superiotool/fintek.c index 2dcb131..035ec5f 100644 --- a/util/superiotool/fintek.c +++ b/util/superiotool/fintek.c @@ -545,6 +545,9 @@ hwmport = regval(port, 0x60) << 8; hwmport |= regval(port, 0x61);
+ /* HWM address register = HWM base address + 5. */ + hwmport += 5; + printf("Hardware monitor (0x%04x)\n", hwmport); dump_superio("Fintek-HWM", hwm_table, hwmport, did, LDN_SEL); }