[coreboot-gerrit] Change in coreboot[master]: asus/f2a85-m*: Enable automatic fancontrol

[Tobias Diedrich (Code Review)]

Hello Paul Menzel, build bot (Jenkins),

I'd like you to reexamine a change.  Please visit

    https://review.coreboot.org/19319

to look at the new patch set (#2).

Change subject: asus/f2a85-m*: Enable automatic fancontrol
......................................................................

asus/f2a85-m*: Enable automatic fancontrol

Enable automatic fancontrol for the F2A85-M and F2A85-M LE with the ASUS
'normal' register settings.

Only the main CPU fan can be speed controlled on this board.

The three asus presets only differ in the start and full speed
temperatures (in degrees celsius):
normal: start @20, full @70
silent: start @30, full @70
turbo:  start @10, full @60

Before this change the fan speed is stuck at a default value of 50%
PWM duty cycle.

In particular the following fan control registers are involved:

reg 0x0b (0x40, default 0x00)
Fan PWM Smoothing Step Frequency Selection Register
bits 7-6 control sampling rate, this changes the sampling rate from 1Hz (Default) to 16Hz (ASUS).

reg 0x15 (0x80, default 0x00)
FAN_CTL1 PWM Control Register
bit7 enables automatic control

reg 0x60 (0x80, default 0x7f)
FAN_CTL1 SmartGuardian Automatic Mode Temperature Limit of OFF Register
With the default value of 0x7f the fan is never turned on in auto mode, with 0x80 (-128) it is never turned off.

reg 0x61 (0x14, default 0x7f)
FAN_CTL1 SmartGuardian Automatic Mode Temperature Limit of Fan Start Register
Minimum PWM reached at 20 degrees (vs default of 127 degrees)

reg 0x62 (0x46, default 0x7f)
FAN_CTL1 SmartGuardian Automatic Mode Temperature Limit of Full Speed Register
100% threshold at 70 degress (vs default of 127 degrees)

reg 0x64 (0xa0, default 0x00)
FAN_CTL1 SmartGuardian Automatic Mode Control Register
bit7 enables data smoothing
bit6-0 set the slope of the control curve (32)

reg 0x65 (0x40, default 0x00)
FAN_CTL1 SmartGuardian Automatic Mode △-Temperature Registers
bit6 enable full duty cycle if any thermal limit is reached

Change-Id: I24c2d88bc54234bb7286ec2c0255549cdf3fb911
Signed-off-by: Tobias Diedrich <ranma+coreboot at tdiedrich.de>
---
M src/mainboard/asus/f2a85-m/devicetree_f2a85-m.cb
M src/mainboard/asus/f2a85-m/devicetree_f2a85-m_le.cb
2 files changed, 17 insertions(+), 7 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/19/19319/2
-- 
To view, visit https://review.coreboot.org/19319
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I24c2d88bc54234bb7286ec2c0255549cdf3fb911
Gerrit-PatchSet: 2
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Tobias Diedrich <ranma+coreboot at tdiedrich.de>
Gerrit-Reviewer: Felix Held <felix-coreboot at felixheld.de>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: Rudolf Marek <r.marek at assembler.cz>
Gerrit-Reviewer: build bot (Jenkins)