2017-12-05 02:02:19

by Eduardo Valentin

[permalink] [raw]
Subject: Re: [PATCH] thermal/drivers/hisi: disable multi alarm support for hi3660 SoC

Hello,

On Thu, Oct 19, 2017 at 09:31:24AM +0800, Wangtao (Kevin, Kirin) wrote:
>
>
> 在 2017/10/18 23:54, Daniel Lezcano 写道:
> >On 18/10/2017 11:15, Tao Wang wrote:
> >>From: Kevin Wangtao <[email protected]>
> >>
> >>multi alarm interrupt forced a re-trigger of power_allocator_throttle
> >>which changes the PID's actual sampling rate, this isn't optimal for
> >>IPA, it is best to disable multi alarm support now and sort out this
> >>issue later.
> >>
> >>Signed-off-by: Kevin Wangtao <[email protected]>
> >
> >Did you test the series with this change?
> Yes

Is this patch still valid with the latest linus master?


2017-12-05 06:57:30

by Daniel Lezcano

[permalink] [raw]
Subject: Re: [PATCH] thermal/drivers/hisi: disable multi alarm support for hi3660 SoC

On 05/12/2017 03:02, Eduardo Valentin wrote:
> Hello,
>
> On Thu, Oct 19, 2017 at 09:31:24AM +0800, Wangtao (Kevin, Kirin) wrote:
>>
>>
>> 在 2017/10/18 23:54, Daniel Lezcano 写道:
>>> On 18/10/2017 11:15, Tao Wang wrote:
>>>> From: Kevin Wangtao <[email protected]>
>>>>
>>>> multi alarm interrupt forced a re-trigger of power_allocator_throttle
>>>> which changes the PID's actual sampling rate, this isn't optimal for
>>>> IPA, it is best to disable multi alarm support now and sort out this
>>>> issue later.
>>>>
>>>> Signed-off-by: Kevin Wangtao <[email protected]>
>>>
>>> Did you test the series with this change?
>> Yes
>
> Is this patch still valid with the latest linus master?

No, it was folded in the next iteration.

All patches for hisilicon are up-to-date.

Thanks.

-- Daniel


--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog