Re: patch: acpitz: active cooling and notify 0x81

2016-04-19 Thread Mike Larkin
On Tue, Apr 19, 2016 at 11:18:33AM +0200, Sebastien Marie wrote: > Hi, > > As I got some feedback for this old thread, I would like to ressurect it > and ask for wider testing: does your fans still works correctly with the > patch ? > > I personnally ran with it since September 2014. > > > The

patch: acpitz: active cooling and notify 0x81

2016-04-19 Thread Sebastien Marie
Hi, As I got some feedback for this old thread, I would like to ressurect it and ask for wider testing: does your fans still works correctly with the patch ? I personnally ran with it since September 2014. The acpitz problem was with active cooling (the ACPI subsystem that plays with fans depen

Re: patch: acpitz: active cooling and notify 0x81

2014-09-19 Thread Sébastien Marie
Hi Mike, On Tue, Sep 16, 2014 at 12:43:11AM -0700, Mike Larkin wrote: > On Mon, Sep 15, 2014 at 04:57:24PM +0200, S??bastien Marie wrote: > > ping ? > > > > Tihs patch is very conservative: it just allow to switch fan OFF if > > state is unknown. > > I finally read through the relevant parts of

Re: patch: acpitz: active cooling and notify 0x81

2014-09-16 Thread Mike Larkin
On Mon, Sep 15, 2014 at 04:57:24PM +0200, S??bastien Marie wrote: > ping ? > > Tihs patch is very conservative: it just allow to switch fan OFF if > state is unknown. I finally read through the relevant parts of the spec. The problem here is that it looks like we are implementing thermal zone hys

Re: patch: acpitz: active cooling and notify 0x81

2014-09-15 Thread Sébastien Marie
ping ? Tihs patch is very conservative: it just allow to switch fan OFF if state is unknown. Thanks. -- Sébastien Marie On Wed, Aug 27, 2014 at 02:51:20PM +0200, Sébastien Marie wrote: > Hi Jonathan, > > First, thanks for your feedback and for your patch. > > On Wed, Aug 27, 2014 at 02:42:43A

Re: patch: acpitz: active cooling and notify 0x81

2014-08-27 Thread Sébastien Marie
Hi Jonathan, First, thanks for your feedback and for your patch. On Wed, Aug 27, 2014 at 02:42:43AM +1000, Jonathan Gray wrote: > Rather than calling acpi_setfan() again would it make > sense to remove the cached state value and move > the state reading to just before the method is called? > > T

Re: patch: acpitz: active cooling and notify 0x81

2014-08-26 Thread Jonathan Gray
Rather than calling acpi_setfan() again would it make sense to remove the cached state value and move the state reading to just before the method is called? That said I'm not familiar with the acpitz code and haven't gone off to look at the spec. Index: acpitz.c ==

Re: patch: acpitz: active cooling and notify 0x81 (patch v2)

2014-08-21 Thread Sébastien Marie
On Thu, Aug 21, 2014 at 10:44:36AM +0200, Sébastien Marie wrote: > Hi, > > Another possibility (not tested) should be to change active cooling code > to permit call acpitz_setfan(OFF) when sc_ac_stat == -1. > Next is the patch that implement the other possibility (the code is running, currently

patch: acpitz: active cooling and notify 0x81

2014-08-21 Thread Sébastien Marie
Hi, I have encounter problem with acpitz: basically with acpitz enable, a fan keep running at max speed (with lot of noise). I have traced the problem (see http://marc.info/?l=openbsd-bugs&m=140517557620686&w=2 for reference). As I haven't received any indications, I go to the ACPI spec in orde