From 458b15b3f6c2a0e0883c544a10e0e4f5dac83560 Mon Sep 17 00:00:00 2001 From: Matt DeVillier Date: Sat, 31 Jan 2026 18:50:48 -0600 Subject: [PATCH] util/superiotool/nuvoton: Add HWM banked register dump for NCT6791D/NCT6796D MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Add extra_dump (-e) support for NCT6791D and NCT6796D to dump all HWM banked registers: global, SYSFAN, CPUFAN, AUXFAN0, PECI calibration, PECI setup, AUXFAN1–3 (+AUXFAN4 for -96D). Uses index port at base+5 and data port at base+6, per datasheets. TEST=build/dump SIO registers for out-of-tree board Erying SRMJ4 with NCT6796D. Change-Id: I689374826bc1e38efaa3d68013610a8fa2052b1f Signed-off-by: Matt DeVillier Reviewed-on: https://review.coreboot.org/c/coreboot/+/91098 Reviewed-by: Paul Menzel Reviewed-by: Alicja Michalska Tested-by: build bot (Jenkins) --- util/superiotool/nuvoton.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/util/superiotool/nuvoton.c b/util/superiotool/nuvoton.c index 90c7a93f7d..8f9a029aea 100644 --- a/util/superiotool/nuvoton.c +++ b/util/superiotool/nuvoton.c @@ -1098,6 +1098,16 @@ extra: for (i = 0; i < 10; i++) dump_data(iobase + 5, i); break; + case 0xc800: /* NCT6791D */ + printf("HWM banked registers:\n"); + for (i = 0; i <= 0xa; i++) + dump_data(iobase + 5, i); + break; + case 0xd420: /* NCT6796D */ + printf("HWM banked registers:\n"); + for (i = 0; i <= 0xb; i++) + dump_data(iobase + 5, i); + break; case 0xd590: /* NCT6687D-W */ dump_nct6687d_gpios(port); /* One can use the APCI/BIOS register set, although the