Attention is currently required from: Harrie Paijmans, Paul Menzel.
Hello Angel Pons, Felix Held, Harrie Paijmans, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/87273?usp=email
to look at the new patch set (#4).
Change subject: superio/fintek/f81866d: Fix HWM port address ......................................................................
superio/fintek/f81866d: Fix HWM port address
The HWM port is +5 to the base address stored in LDN 0x4 at index 0x60/0x61. Take this rule into account when configuring the monitor, as it was done for Fintek SIO chips in the superiotool utility [1].
[1] commit d92745b
TEST=Run coreboot on the motherboard with the Fintek F81966 chip (which is architecturally compatible) with pnp_write_hwm5_index() in the HWM initialization code: - the fans are regulated correctly; - superiotool prints the values of the configuration registers updated during initialization.
Change-Id: If39400e56a7d0792a5bc8f312c29dd5e98a0b2d3 Signed-off-by: Maxim Polyakov max.senia.poliak@gmail.com --- M src/superio/fintek/f81866d/f81866d_hwm.c 1 file changed, 16 insertions(+), 16 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/73/87273/4