2014-07-16 22:41:03

by Peter Kästle

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

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;


2014-07-17 08:46:37

by Borislav Petkov

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

On Thu, Jul 17, 2014 at 12:34:32AM +0200, Peter Feuerer wrote:
> 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.

Yeah, I'll wait for your latest submission and give it a run here.

Thanks.

--
Regards/Gruss,
Boris.

Sent from a fat crate under my desk. Formatting is fine.
--