Re: [PATCH v3 0/6] acerhdf/thermal: adding new models, appropriate governor and minor clean up

Peter Feuerer
2014-07-16T22:41:03Z

Hi Boris,

nice to hear from you.

Borislav Petkov writes:

> Hi Peter,
>
> On Sat, May 03, 2014 at 07:59:20PM +0200, Peter Feuerer wrote:
>> This patch series is intended to:
>>
>> * Introduce "manual mode" support (Patch 1 & 2), which is needed to control
>> the fan of a few new models.
>>
>> * Add an appropriate thermal governor (Patch 3 & 4). Manipulating and
>> fiddling around with the step-wise governor has been a very fragile thing
>> in the past and as it broke again, I used the opportunity to add a two
>> point thermal governor which implements the actual fan handling required by
>> acerhdf and puts from my point of view things straight.
>>
>> * Do some minor clean up like:
>> - adding second trip point for critical temperature (Patch 5)
>> - removing _t suffix from struct which isn't typedef and replace unsigned
>> char by u8 (Patch 6)
>
> so what's the state of those?
>
> In the meantime I noticed the breakage here too, after updating the
> kernel on the aspire one - the fan remains on.
>
> Do you have a latest version which is ready for testing or should I take
> those?

I'll send out new set of patches by Sunday latestly, but you could of course
start testing with version3 of the patchset, as I just planned to do minor
fixes anyhow.

--
kind regards,
--peter;