Re: git: acpi_thinkpad(4): make acpi_thinkpad sensors actually w
On 18 February 2010 02:51, Constantine A. Murenin
<cnst@crater.dragonflybsd.org> wrote:
>
> commit 20b3fbb5f22fcbc17e90863f7363a7918c04cbd2
> Author: Constantine A. Murenin <cnst+dfly@bugmail.mojo.ru>
> Date: ꀠThu Feb 18 02:13:00 2010 -0500
>
> ꀠ乸cpi_thinkpad(4): make acpi_thinkpad sensors actually work
>
> ꀠꀪ After this patch on my ThinkPad R51:
> ꀠꀠꀥsysctl hw.sensors.acpi_ibm0
> ꀠꀠ温w.sensors.acpi_ibm0.temp0: 69.00 degC
> ꀠꀠ温w.sensors.acpi_ibm0.temp1: 52.00 degC
> ꀠꀠ温w.sensors.acpi_ibm0.temp2: 41.00 degC
> ꀠꀠ温w.sensors.acpi_ibm0.temp3: 59.00 degC
> ꀠꀠ温w.sensors.acpi_ibm0.temp4: 36.00 degC
> ꀠꀠ温w.sensors.acpi_ibm0.temp5: invalid
> ꀠꀠ温w.sensors.acpi_ibm0.temp6: 26.00 degC
> ꀠꀠ温w.sensors.acpi_ibm0.temp7: invalid
> ꀠꀠ温w.sensors.acpi_ibm0.fan0: 3469 RPM
>
> ꀠꀪ Before this patch:
> ꀠꀠ漑 The temperature values were absolutely random, since
> ꀠꀠꀠ濳he /data/ variable was never initialised from temp[i].
> ꀠꀠ漑 The braces were missing from the fan error path of
> ꀠꀠꀠ乸n if statement.
> ꀠꀠ漑 Whitespaces were used instead of tabs.
>
> Summary of changes:
> 澵ys/dev/acpica5/acpi_thinkpad/acpi_thinkpad.c | ꀠ45 +++++++++++-------------
> ꀱ files changed, 21 insertions(+), 24 deletions(-)
>
> http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/20b3fbb5f22fcbc17e90863f7363a7918c04cbd2
>
>
> --
> DragonFly BSD source repository
Hum, and it would look that this fixes only a small part of the
breakages that were introduced to acpi_thinkpad with 10f97674 on
2009-11-08. Others include stuff like the unintended and partial
rename of acpi_thinkpad back to acpi_ibm, and change of many
freebsd.org email addresses to kfreebsd.org.
Someone with more interest and spare time should perhaps take a look
at a possible (partial?) backout.
http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/10f976749fd9ad2e8642ea80ce533f7416910a65#patch25
C.