Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1492230

Re: [PATCH] of: thermal: Fixed governor at each thermal zone

Show all headers | View raw

Hi, Javi, Lukasz and Eduardo,

thanks for your input.

thanks,
rui

On 二, 2016-09-27 at 06:22 -0700, Eduardo Valentin wrote:
> Hello, Lukasz, Inhyuk, Javi,
> 
> On Tue, Sep 27, 2016 at 12:52:04PM +0100, Lukasz Luba wrote:
> > 
> > 
> > On 27/09/16 02:46, Zhang Rui wrote:
> > > 
> > > On 一, 2016-09-19 at 10:18 +0900, Inhyuk Kang wrote:
> > > > 
> > > > It is necessary to be added governor at each thermal_zone.
> > > > Because some governors should be operated in the during the
> > > > kernel
> > > > booting
> > > > in order to avoid heating problem.
> > > > 
> > > > Default governor cannot be covered all thermal zones policy
> > > > because
> > > > some thermal zones want to apply different one.
> > > > For example, the power allocator governor operates differently
> > > > with
> > > > step wise governor.
> > > > Hence, it is better to parse governor parameter from the device
> > > > tree.
> > > > 
> > > > Signed-off-by: Inhyuk Kang <hugh.kang@lge.com>
> > > > 
> > > The patch looks okay to me.
> > > Eduardo, what do you think of this patch?
> > Hi Rui,
> > 
> > Beside the fact which Javi pointed out in his email, there is an
> > issue in
> > the patch itself.
> > The idea behind the patch is good, but the patch should have some
> > improvements, i.e:
> > - strncpy instead of strcpy,
> > - if the governor name is not found in the registered governor's
> > list by
> > __find_governor (and then null is set) we should probably switch to
> > default
> > governor,
> > - add DT documentation,
> Also, the idea of the patch is good, almost tempting to do it, but
> unfortunately, not acceptable from DT perspective. The patch
> infringes
> two of the DT conceptual and design decision of:
> (a) DT should describe hardware, not policy;
> (b) DT should describe hardware, not OS specific implementations.
> 
> As already pointed by Javi, this patch has already been proposed
> (more
> than one time by different people), but, it still continues to be
> unacceptable.
> 
> Cheers,
> 
> 
> > 
> > 
> > Regards,
> > Lukasz
> > 

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH] of: thermal: Fixed governor at each thermal zone Zhang Rui <rui.zhang@intel.com> - 2016-09-27 03:50 +0200
  Re: [PATCH] of: thermal: Fixed governor at each thermal zone Javi Merino <javi.merino@arm.com> - 2016-09-27 13:30 +0200
  Re: [PATCH] of: thermal: Fixed governor at each thermal zone Lukasz Luba <lukasz.luba@arm.com> - 2016-09-27 14:00 +0200
    Re: [PATCH] of: thermal: Fixed governor at each thermal zone Eduardo Valentin <edubezval@gmail.com> - 2016-09-27 15:30 +0200
      Re: [PATCH] of: thermal: Fixed governor at each thermal zone Zhang Rui <rui.zhang@intel.com> - 2016-09-28 03:40 +0200
        RE: [PATCH] of: thermal: Fixed governor at each thermal zone 강인혁/책임연구원/SW Platform(연)AOT팀(hugh.kang@lge.com) <hugh.kang@lge.com> - 2016-09-28 09:20 +0200

csiph-web