Re: [PATCH 2/5] thermal: devfreq_cooling: get a copy of device status

2020-12-02 Thread Ionela Voinescu
On Tuesday 01 Dec 2020 at 12:19:18 (+), Lukasz Luba wrote: > > > On 12/1/20 10:36 AM, Ionela Voinescu wrote: > > Hi, > > > > Sorry for the delay and for the noise on this older version. I first > > want to understand the code better. > > > > On Thursday 22 Oct 2020 at 11:55:28 (+0100), Luka

Re: [PATCH 2/5] thermal: devfreq_cooling: get a copy of device status

2020-12-02 Thread Lukasz Luba
On 12/1/20 10:36 AM, Ionela Voinescu wrote: Hi, Sorry for the delay and for the noise on this older version. I first want to understand the code better. On Thursday 22 Oct 2020 at 11:55:28 (+0100), Lukasz Luba wrote: [..] +{ + /* Make some space if needed */ + if (status->bu

Re: [PATCH 2/5] thermal: devfreq_cooling: get a copy of device status

2020-12-02 Thread Ionela Voinescu
Hi, Sorry for the delay and for the noise on this older version. I first want to understand the code better. On Thursday 22 Oct 2020 at 11:55:28 (+0100), Lukasz Luba wrote: [..] > > > > > > +{ > > > + /* Make some space if needed */ > > > + if (status->busy_time > 0x) { > > > + stat

Re: [PATCH 2/5] thermal: devfreq_cooling: get a copy of device status

2020-10-23 Thread Lukasz Luba
Hi Daniel, On 10/14/20 3:34 PM, Daniel Lezcano wrote: On 21/09/2020 14:20, Lukasz Luba wrote: Devfreq cooling needs to now the correct status of the device in order to operate. Do not rely on Devfreq last_status which might be a stale data and get more up-to-date values of the load. Devfreq fr

Re: [PATCH 2/5] thermal: devfreq_cooling: get a copy of device status

2020-10-23 Thread Lukasz Luba
Hi Ionela, On 10/7/20 5:11 PM, Ionela Voinescu wrote: On Monday 21 Sep 2020 at 13:20:04 (+0100), Lukasz Luba wrote: Devfreq cooling needs to now the correct status of the device in order to operate. Do not rely on Devfreq last_status which might be a stale data and get more up-to-date values o

Re: [PATCH 2/5] thermal: devfreq_cooling: get a copy of device status

2020-10-15 Thread Daniel Lezcano
On 21/09/2020 14:20, Lukasz Luba wrote: > Devfreq cooling needs to now the correct status of the device in order > to operate. Do not rely on Devfreq last_status which might be a stale data > and get more up-to-date values of the load. > > Devfreq framework can change the device status in the back

Re: [PATCH 2/5] thermal: devfreq_cooling: get a copy of device status

2020-10-07 Thread Ionela Voinescu
On Monday 21 Sep 2020 at 13:20:04 (+0100), Lukasz Luba wrote: > Devfreq cooling needs to now the correct status of the device in order > to operate. Do not rely on Devfreq last_status which might be a stale data > and get more up-to-date values of the load. > > Devfreq framework can change the dev