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

Groups > linux.kernel > #1332789 > unrolled thread

Re: [PATCH V5 07/11] of: Add bindings of hw-trip-points for soctherm

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH V5 07/11] of: Add bindings of hw-trip-points for soctherm Rob Herring <robh@kernel.org> - 2016-02-12 15:50 +0100
    Re: [PATCH V5 07/11] of: Add bindings of hw-trip-points for soctherm Wei Ni <wni@nvidia.com> - 2016-02-15 08:20 +0100
      Re: [PATCH V5 07/11] of: Add bindings of hw-trip-points for soctherm Wei Ni <wni@nvidia.com> - 2016-02-15 08:30 +0100
      Re: [PATCH V5 07/11] of: Add bindings of hw-trip-points for soctherm Wei Ni <wni@nvidia.com> - 2016-02-17 10:50 +0100
        Re: [PATCH V5 07/11] of: Add bindings of hw-trip-points for soctherm Matt Longnecker <mlongnecker@nvidia.com> - 2016-02-17 22:00 +0100

#1332789 — Re: [PATCH V5 07/11] of: Add bindings of hw-trip-points for soctherm

On Fri, Jan 29, 2016 at 2:47 AM, Wei Ni <wni@nvidia.com> wrote:
> Add hw-trips sub-node for soctherm, which is
> used to describe the hardware trip points for
> each soctherm sensors.

I still don't understand why you are doing something custom here.
What's wrong with what is defined in:

Documentation/devicetree/bindings/thermal/thermal.txt

Rob

[toc] | [next] | [standalone]

#1334202

Hi, Rob

On 2016年02月12日 22:42, Rob Herring wrote:
> On Fri, Jan 29, 2016 at 2:47 AM, Wei Ni <wni@nvidia.com> wrote:
>> Add hw-trips sub-node for soctherm, which is
>> used to describe the hardware trip points for
>> each soctherm sensors.
> 
> I still don't understand why you are doing something custom here.
> What's wrong with what is defined in:
> 
> Documentation/devicetree/bindings/thermal/thermal.txt

In current thermal framework, it support to set trip points for the thermal
zones, but this trip point is handled by SW, doesn't have callbacks to program
the hardware.
There had patches:
[PATCH 12/16] thermal: thermal: Add support for hardware-tracked trip points,
http://lists.infradead.org/pipermail/linux-mediatek/2015-May/000866.html,
which can support to program hardware to set trip points, but it seems this
series didn't be taken.
So I add this hw-trip-points sub-node for soctherm.

Hi, Sascha Hauer

Thanks.
Wei.

> 
> Rob
> 

[toc] | [prev] | [next] | [standalone]

#1334203

On 2016年02月15日 15:15, Wei Ni wrote:
> Hi, Rob
> 
> On 2016年02月12日 22:42, Rob Herring wrote:
>> On Fri, Jan 29, 2016 at 2:47 AM, Wei Ni <wni@nvidia.com> wrote:
>>> Add hw-trips sub-node for soctherm, which is
>>> used to describe the hardware trip points for
>>> each soctherm sensors.
>>
>> I still don't understand why you are doing something custom here.
>> What's wrong with what is defined in:
>>
>> Documentation/devicetree/bindings/thermal/thermal.txt
> 
> In current thermal framework, it support to set trip points for the thermal
> zones, but this trip point is handled by SW, doesn't have callbacks to program
> the hardware.
> There had patches:
> [PATCH 12/16] thermal: thermal: Add support for hardware-tracked trip points,
> http://lists.infradead.org/pipermail/linux-mediatek/2015-May/000866.html,
> which can support to program hardware to set trip points, but it seems this
> series didn't be taken.
> So I add this hw-trip-points sub-node for soctherm.
> 
> Hi, Sascha Hauer

Hi, Sasha
Sorry, I forgot to type messages in previous mail.

I noticed that you submitted "[PATCH 12/16] thermal: thermal: Add support for
hardware-tracked trip points" long time ago, I think it's very useful, do you
know why it didn't be taken?

> 
> Thanks.
> Wei.
> 
>>
>> Rob
>>

[toc] | [prev] | [next] | [standalone]

#1336177

Hi, Rob

On 2016年02月15日 15:15, Wei Ni wrote:
> Hi, Rob
> 
> On 2016年02月12日 22:42, Rob Herring wrote:
>> On Fri, Jan 29, 2016 at 2:47 AM, Wei Ni <wni@nvidia.com> wrote:
>>> Add hw-trips sub-node for soctherm, which is
>>> used to describe the hardware trip points for
>>> each soctherm sensors.
>>
>> I still don't understand why you are doing something custom here.
>> What's wrong with what is defined in:
>>
>> Documentation/devicetree/bindings/thermal/thermal.txt

I think it's better to add a .set_trip_temp for of-thermal's
thermal_zone_of_device_ops{}, so that we can set trip points on hardware. I'm
preparing patches, and will send out next version.
Thanks for your comments.

> 
> In current thermal framework, it support to set trip points for the thermal
> zones, but this trip point is handled by SW, doesn't have callbacks to program
> the hardware.
> There had patches:
> [PATCH 12/16] thermal: thermal: Add support for hardware-tracked trip points,
> http://lists.infradead.org/pipermail/linux-mediatek/2015-May/000866.html,
> which can support to program hardware to set trip points, but it seems this
> series didn't be taken.
> So I add this hw-trip-points sub-node for soctherm.
> 
> Hi, Sascha Hauer
> 
> Thanks.
> Wei.
> 
>>
>> Rob
>>

[toc] | [prev] | [next] | [standalone]

#1336753

(cc += linux-pm)

Rob,

The Linux thermal framework makes a sharp distinction between "thermal 
zones" and "cooling devices". thermal_zones are aware of temperatures 
and have trip points. cooling_devices are unaware of temperatures -- 
they just have a state variable. The thermal framework binds thermal 
zones at particular temperatures to particular cooling devices. It does 
so *in software*

The SOC_THERM IP block doesn't fit neatly in that paradigm. Yes, it 
provides the ability to read temperatures and raise interrupts at 
temperatures. However, it also has the ability to take cooling action 
_in hardware_ at particular temperatures. It can throttle the CPU & GPU 
clock and it can shut down the SOC. Neither of those are graceful 
actions, but they are valuable in practice.

In previous incarnations of this driver we have implemented an unholy 
mash-up of a cooling device and a thermal zone for configuring the 
hardware thermal actions. It was a mess. This patchset from Wei takes a 
simpler approach -- allow the thermal hardware cooling actions to be 
configured directly via DT. It's simple but it's good enough in 
practice. I like it.

-Matt

On 02/17/2016 01:46 AM, Wei Ni wrote:
> On 2016年02月15日 15:15, Wei Ni wrote:
>> >Hi, Rob
>> >
>> >On 2016年02月12日 22:42, Rob Herring wrote:
>>> >>On Fri, Jan 29, 2016 at 2:47 AM, Wei Ni<wni@nvidia.com>  wrote:
>>>> >>>Add hw-trips sub-node for soctherm, which is
>>>> >>>used to describe the hardware trip points for
>>>> >>>each soctherm sensors.
>>> >>
>>> >>I still don't understand why you are doing something custom here.
>>> >>What's wrong with what is defined in:
>>> >>
>>> >>Documentation/devicetree/bindings/thermal/thermal.txt
> I think it's better to add a .set_trip_temp for of-thermal's
> thermal_zone_of_device_ops{}, so that we can set trip points on hardware. I'm
> preparing patches, and will send out next version.
> Thanks for your comments.
>

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web