Re: [PATCH 09/13] thermal/drivers/hisi: Remove costly sensor inspection

2017-09-04 Thread Leo Yan
On Mon, Sep 04, 2017 at 01:29:19PM +0200, Daniel Lezcano wrote: > On 04/09/2017 02:50, Leo Yan wrote: > > [ ... ] > > >> That is something I asked myself but I finally decided to not change the > >> current behavior and let that be added in a separate patch. > > > > This is fine. > > Hi Leo, >

Re: [PATCH 09/13] thermal/drivers/hisi: Remove costly sensor inspection

2017-09-04 Thread Daniel Lezcano
On 04/09/2017 02:50, Leo Yan wrote: [ ... ] >> That is something I asked myself but I finally decided to not change the >> current behavior and let that be added in a separate patch. > > This is fine. Hi Leo, can you tag this patch if you agree with it? -- Linaro.or

Re: [PATCH 09/13] thermal/drivers/hisi: Remove costly sensor inspection

2017-09-03 Thread Leo Yan
On Sat, Sep 02, 2017 at 03:10:29PM +0200, Daniel Lezcano wrote: [...] > On 02/09/2017 05:29, Leo Yan wrote: > > On Wed, Aug 30, 2017 at 10:47:33AM +0200, Daniel Lezcano wrote: > >> The sensor is all setup, bind, resetted, acked, etc... every single second. > >> > >> That was the way to workaround

Re: [PATCH 09/13] thermal/drivers/hisi: Remove costly sensor inspection

2017-09-02 Thread Daniel Lezcano
On 02/09/2017 05:29, Leo Yan wrote: > On Wed, Aug 30, 2017 at 10:47:33AM +0200, Daniel Lezcano wrote: >> The sensor is all setup, bind, resetted, acked, etc... every single second. >> >> That was the way to workaround a problem with the interrupt bouncing again >> and >> again. >> >> With the foll

Re: [PATCH 09/13] thermal/drivers/hisi: Remove costly sensor inspection

2017-09-01 Thread Leo Yan
On Wed, Aug 30, 2017 at 10:47:33AM +0200, Daniel Lezcano wrote: > The sensor is all setup, bind, resetted, acked, etc... every single second. > > That was the way to workaround a problem with the interrupt bouncing again and > again. > > With the following changes, we fix all in one: > > - Do t

[PATCH 09/13] thermal/drivers/hisi: Remove costly sensor inspection

2017-08-30 Thread Daniel Lezcano
The sensor is all setup, bind, resetted, acked, etc... every single second. That was the way to workaround a problem with the interrupt bouncing again and again. With the following changes, we fix all in one: - Do the setup, one time, at probe time - Add the IRQF_ONESHOT, ack the interrupt in