17 Dec, 2009

1 commit


16 Dec, 2009

1 commit


06 Nov, 2009

4 commits

  • Otherwise polling will continue for the thermal zone even when
    it is no longer needed, for example because forced passive cooling
    was disabled.

    Signed-off-by: Frans Pop
    Acked-by: Matthew Garrett
    Acked-by: Zhang Rui
    Signed-off-by: Andrew Morton
    Signed-off-by: Len Brown

    Frans Pop
     
  • Setting polling_delay is useless as passive_delay has priority,
    so the value shown in proc isn't the actual polling delay. It
    also gives the impression to the user that he can change the
    polling interval through proc, while in fact he can't.

    Also, unset passive_delay when the forced passive trip point is
    unbound to allow polling to be disabled.

    Signed-off-by: Frans Pop
    Acked-by: Matthew Garrett
    Acked-by: Zhang Rui
    Signed-off-by: Andrew Morton
    Signed-off-by: Len Brown

    Frans Pop
     
  • Values below 1000 milli-celsius don't make sense and can cause the
    system to go into a thermal heart attack: the actual temperature
    will always be lower and thus the system will be throttled down to
    its lowest setting.

    An additional problem is that values below 1000 will show as 0 in
    /proc/acpi/thermal/TZx/trip_points:passive.

    cat passive
    0
    echo -n 90 >passive
    bash: echo: write error: Invalid argument
    echo -n 90000 >passive
    cat passive
    90000

    Signed-off-by: Frans Pop
    Acked-by: Zhang Rui
    Signed-off-by: Andrew Morton
    Signed-off-by: Len Brown

    Frans Pop
     
  • Make the trip_point_N_type sysfs files return a string ending in EOL for
    consistency with other sysfs files.

    Signed-off-by: Amit Kucheria
    Acked-by: Zhang Rui
    Signed-off-by: Andrew Morton
    Signed-off-by: Len Brown

    Amit Kucheria
     

19 Sep, 2009

1 commit


27 Aug, 2009

1 commit

  • The return value of the get_temp function is not checked when doing a
    thermal zone update. This may lead to a critical shutdown if get_temp
    fails and the content of the temp variable is incorrectly set higher than
    the critical trip point.

    This has been observed on a system with incorrect ACPI implementation
    where the corresponding methods were not serialized and therefore
    sometimes triggered ACPI errors (AE_ALREADY_EXISTS). The following
    critical shutdowns indicated a temperature of 2097 C, which was obviously
    wrong.

    The patch adds a return value check that jumps over all trip point
    evaluations printing a warning if get_temp fails. The trip points are
    evaluated again on the next polling interval with successful get_temp
    execution.

    Signed-off-by: Michael Brunner
    Acked-by: Zhang Rui
    Cc: Len Brown
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Michael Brunner
     

16 Jun, 2009

1 commit

  • In the near future, the driver core is going to not allow direct access
    to the driver_data pointer in struct device. Instead, the functions
    dev_get_drvdata() and dev_set_drvdata() should be used. These functions
    have been around since the beginning, so are backwards compatible with
    all older kernel versions.

    Signed-off-by: Greg Kroah-Hartman

    Greg Kroah-Hartman
     

15 May, 2009

1 commit

  • This patch fixes a regression caused by commit
    b1569e99c795bf83b4ddf41c4f1c42761ab7f75e
    "ACPI: move thermal trip handling to generic thermal layer"
    which accidentally changed trip point trigger condition to
    temp > trip_temp

    This patch changes the trigger condition back to
    temp >= trip_temp

    Signed-off-by: Vladimir Zajac
    Acked-by: Zhang Rui
    Acked-by: Matthew Garrett
    Signed-off-by: Andrew Morton
    Signed-off-by: Len Brown

    Vladimir Zajac
     

28 Mar, 2009

1 commit

  • Due to poor thermal design or Linux driving hardware outside its thermal
    envelope, some systems will reach critical temperature and shut down
    under high load. This patch adds support for forcing a polling-based
    passive trip point if the firmware doesn't provide one. The assumption
    is made that the processor is the most practical means to reduce the
    dynamic heat generation, so hitting the passive thermal limit will cause
    the CPU to be throttled until the temperature stabalises around the
    defined value.

    UI is provided via a "passive" sysfs entry in the thermal zone
    directory. It accepts a decimal value in millidegrees celsius, or "0" to
    disable the functionality. Default behaviour is for this functionality
    to be disabled.

    Signed-off-by: Matthew Garrett
    Signed-off-by: Len Brown

    Matthew Garrett
     

21 Feb, 2009

1 commit


20 Feb, 2009

1 commit


07 Jan, 2009

1 commit


26 Jun, 2008

1 commit


29 Apr, 2008

3 commits


16 Apr, 2008

1 commit

  • The THERMAL_MAX_TRIPS value is set to 10. It is too few for the Compaq AP550
    machine which has 12 trip points.

    Signed-off-by: Krzysztof Helt
    Cc: Len Brown
    Cc: Zhang Rui
    Cc: "Rafael J. Wysocki"
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Krzysztof Helt
     

18 Mar, 2008

2 commits


13 Mar, 2008

1 commit


16 Feb, 2008

2 commits


08 Feb, 2008

1 commit


02 Feb, 2008

1 commit