From 44f293a509326ed72a4bdefe2b5983586f869a7a Mon Sep 17 00:00:00 2001 From: RichardG867 Date: Sun, 29 Mar 2020 19:32:37 -0300 Subject: [PATCH] Small fixes to W83781D and SPD --- src/hwm_w83781d.c | 6 +++--- src/spd.c | 1 + 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/hwm_w83781d.c b/src/hwm_w83781d.c index b80bef7d9..19de7d997 100644 --- a/src/hwm_w83781d.c +++ b/src/hwm_w83781d.c @@ -427,7 +427,7 @@ w83781d_reset(w83781d_t *dev, uint8_t initialization) * AS99127F Rev. 2 on a P4B motherboard, and they seem to work well enough. */ if (dev->local & W83781D_AS99127F) { - dev->regs[0x00] = 0xB8; /* might be connected to IN2 Low Limit in some way */ + /* 0x00 appears to mirror IN2 Low Limit */ dev->regs[0x01] = dev->regs[0x23]; /* appears to mirror IN3 */ dev->regs[0x02] = dev->regs[0x20]; /* appears to mirror IN0 */ dev->regs[0x03] = 0x60; @@ -512,7 +512,7 @@ const device_t w83781d_device = { /* - * ASUS AS99127F is a customized W83781D with no ISA interface (SMBus only), + * The ASUS AS99127F is a customized W83781D with no ISA interface (SMBus only), * added proprietary registers and different chip/vendor IDs. */ const device_t as99127f_device = { @@ -526,7 +526,7 @@ const device_t as99127f_device = { /* - * Rev. 2 changes the vendor ID back to Winbond's. + * Rev. 2 changes the vendor ID back to Winbond's and brings some other changes. */ const device_t as99127f_rev2_device = { "ASUS AS99127F Rev. 2 Hardware Monitor", diff --git a/src/spd.c b/src/spd.c index a4abca624..2cdc81abd 100644 --- a/src/spd.c +++ b/src/spd.c @@ -184,6 +184,7 @@ spd_register(uint8_t ram_type, uint8_t slot_mask, uint16_t max_module_size) spd_log("SPD: vslot %d = %d MB\n", vslot, vslots[vslot]); total_size -= vslots[vslot]; } else { + vslots[vslot] = 0; break; } }