* [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
@ 2017-08-03 8:46 ` Mykola Kostenok
0 siblings, 0 replies; 12+ messages in thread
From: Mykola Kostenok @ 2017-08-03 8:46 UTC (permalink / raw)
To: Rob Herring, devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Mykola Kostenok, Guenter Roeck,
Jaghathiswari Rankappagounder Natarajan,
openbmc-uLR06cmDAlY/bJ5BZ2RsiQ, Patrick Venture, Vadim Pasternak,
Ohad Oz
It's add support for cooling device creation to
aspeed-pwm-tacho.
Cooling device could be bound to a thermal zone
for the thermal control.
v1 -> v2:
- Fix commit message.
v2 -> v3:
Pointed out by Rob Herring:
- Put cooling-levels under fan subnodes.
Signed-off-by: Mykola Kostenok <c_mykolak-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
---
Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
index cf44605..367c820 100644
--- a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
+++ b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
@@ -11,6 +11,8 @@ Required properties for pwm-tacho node:
- #size-cells : should be 1.
+- #cooling-cells: should be 2.
+
- reg : address and length of the register set for the device.
- pinctrl-names : a pinctrl state named "default" must be defined.
@@ -28,12 +30,17 @@ fan subnode format:
Under fan subnode there can upto 8 child nodes, with each child node
representing a fan. If there are 8 fans each fan can have one PWM port and
one/two Fan tach inputs.
+For PWM port can be configured cooling-levels to create cooling device.
+Cooling device could be bound to a thermal zone for the thermal control.
Required properties for each child node:
- reg : should specify PWM source port.
integer value in the range 0 to 7 with 0 indicating PWM port A and
7 indicating PWM port H.
+- cooling-levels: PWM duty cycle values in a range from 0 to 255
+ which correspond to thermal cooling states.
+
- aspeed,fan-tach-ch : should specify the Fan tach input channel.
integer value in the range 0 through 15, with 0 indicating
Fan tach channel 0 and 15 indicating Fan tach channel 15.
@@ -50,6 +57,7 @@ pwm_tacho_fixed_clk: fixedclk {
pwm_tacho: pwmtachocontroller@1e786000 {
#address-cells = <1>;
#size-cells = <1>;
+ #cooling-cells = <2>;
reg = <0x1E786000 0x1000>;
compatible = "aspeed,ast2500-pwm-tacho";
clocks = <&pwm_tacho_fixed_clk>;
@@ -58,6 +66,7 @@ pwm_tacho: pwmtachocontroller@1e786000 {
fan@0 {
reg = <0x00>;
+ cooling-levels = /bits/ 8 <125 151 177 203 229 255>;
aspeed,fan-tach-ch = /bits/ 8 <0x00>;
};
--
2.8.4
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
@ 2017-08-03 8:46 ` Mykola Kostenok
0 siblings, 0 replies; 12+ messages in thread
From: Mykola Kostenok @ 2017-08-03 8:46 UTC (permalink / raw)
To: Rob Herring, devicetree
Cc: Mykola Kostenok, Guenter Roeck,
Jaghathiswari Rankappagounder Natarajan, openbmc,
Patrick Venture, Vadim Pasternak, Ohad Oz
It's add support for cooling device creation to
aspeed-pwm-tacho.
Cooling device could be bound to a thermal zone
for the thermal control.
v1 -> v2:
- Fix commit message.
v2 -> v3:
Pointed out by Rob Herring:
- Put cooling-levels under fan subnodes.
Signed-off-by: Mykola Kostenok <c_mykolak@mellanox.com>
---
Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
index cf44605..367c820 100644
--- a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
+++ b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
@@ -11,6 +11,8 @@ Required properties for pwm-tacho node:
- #size-cells : should be 1.
+- #cooling-cells: should be 2.
+
- reg : address and length of the register set for the device.
- pinctrl-names : a pinctrl state named "default" must be defined.
@@ -28,12 +30,17 @@ fan subnode format:
Under fan subnode there can upto 8 child nodes, with each child node
representing a fan. If there are 8 fans each fan can have one PWM port and
one/two Fan tach inputs.
+For PWM port can be configured cooling-levels to create cooling device.
+Cooling device could be bound to a thermal zone for the thermal control.
Required properties for each child node:
- reg : should specify PWM source port.
integer value in the range 0 to 7 with 0 indicating PWM port A and
7 indicating PWM port H.
+- cooling-levels: PWM duty cycle values in a range from 0 to 255
+ which correspond to thermal cooling states.
+
- aspeed,fan-tach-ch : should specify the Fan tach input channel.
integer value in the range 0 through 15, with 0 indicating
Fan tach channel 0 and 15 indicating Fan tach channel 15.
@@ -50,6 +57,7 @@ pwm_tacho_fixed_clk: fixedclk {
pwm_tacho: pwmtachocontroller@1e786000 {
#address-cells = <1>;
#size-cells = <1>;
+ #cooling-cells = <2>;
reg = <0x1E786000 0x1000>;
compatible = "aspeed,ast2500-pwm-tacho";
clocks = <&pwm_tacho_fixed_clk>;
@@ -58,6 +66,7 @@ pwm_tacho: pwmtachocontroller@1e786000 {
fan@0 {
reg = <0x00>;
+ cooling-levels = /bits/ 8 <125 151 177 203 229 255>;
aspeed,fan-tach-ch = /bits/ 8 <0x00>;
};
--
2.8.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
2017-08-03 8:46 ` Mykola Kostenok
@ 2017-08-09 4:08 ` Guenter Roeck
-1 siblings, 0 replies; 12+ messages in thread
From: Guenter Roeck @ 2017-08-09 4:08 UTC (permalink / raw)
To: Mykola Kostenok, Rob Herring, devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Jaghathiswari Rankappagounder Natarajan,
openbmc-uLR06cmDAlY/bJ5BZ2RsiQ, Patrick Venture, Vadim Pasternak,
Ohad Oz
On 08/03/2017 01:46 AM, Mykola Kostenok wrote:
> It's add support for cooling device creation to
> aspeed-pwm-tacho.
> Cooling device could be bound to a thermal zone
> for the thermal control.
>
> v1 -> v2:
> - Fix commit message.
> v2 -> v3:
> Pointed out by Rob Herring:
> - Put cooling-levels under fan subnodes.
>
> Signed-off-by: Mykola Kostenok <c_mykolak-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Did Rob ever Ack this patch ?
Note you didn't copy the hwmon mailing list, which doesn't exactly help
to keep track of it, and I may have lost Rob's Ack. My apologies if that
is the case.
Guenter
> ---
> Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> index cf44605..367c820 100644
> --- a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> +++ b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> @@ -11,6 +11,8 @@ Required properties for pwm-tacho node:
>
> - #size-cells : should be 1.
>
> +- #cooling-cells: should be 2.
> +
> - reg : address and length of the register set for the device.
>
> - pinctrl-names : a pinctrl state named "default" must be defined.
> @@ -28,12 +30,17 @@ fan subnode format:
> Under fan subnode there can upto 8 child nodes, with each child node
> representing a fan. If there are 8 fans each fan can have one PWM port and
> one/two Fan tach inputs.
> +For PWM port can be configured cooling-levels to create cooling device.
> +Cooling device could be bound to a thermal zone for the thermal control.
>
> Required properties for each child node:
> - reg : should specify PWM source port.
> integer value in the range 0 to 7 with 0 indicating PWM port A and
> 7 indicating PWM port H.
>
> +- cooling-levels: PWM duty cycle values in a range from 0 to 255
> + which correspond to thermal cooling states.
> +
> - aspeed,fan-tach-ch : should specify the Fan tach input channel.
> integer value in the range 0 through 15, with 0 indicating
> Fan tach channel 0 and 15 indicating Fan tach channel 15.
> @@ -50,6 +57,7 @@ pwm_tacho_fixed_clk: fixedclk {
> pwm_tacho: pwmtachocontroller@1e786000 {
> #address-cells = <1>;
> #size-cells = <1>;
> + #cooling-cells = <2>;
> reg = <0x1E786000 0x1000>;
> compatible = "aspeed,ast2500-pwm-tacho";
> clocks = <&pwm_tacho_fixed_clk>;
> @@ -58,6 +66,7 @@ pwm_tacho: pwmtachocontroller@1e786000 {
>
> fan@0 {
> reg = <0x00>;
> + cooling-levels = /bits/ 8 <125 151 177 203 229 255>;
> aspeed,fan-tach-ch = /bits/ 8 <0x00>;
> };
>
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
@ 2017-08-09 4:08 ` Guenter Roeck
0 siblings, 0 replies; 12+ messages in thread
From: Guenter Roeck @ 2017-08-09 4:08 UTC (permalink / raw)
To: Mykola Kostenok, Rob Herring, devicetree
Cc: Jaghathiswari Rankappagounder Natarajan, openbmc,
Patrick Venture, Vadim Pasternak, Ohad Oz
On 08/03/2017 01:46 AM, Mykola Kostenok wrote:
> It's add support for cooling device creation to
> aspeed-pwm-tacho.
> Cooling device could be bound to a thermal zone
> for the thermal control.
>
> v1 -> v2:
> - Fix commit message.
> v2 -> v3:
> Pointed out by Rob Herring:
> - Put cooling-levels under fan subnodes.
>
> Signed-off-by: Mykola Kostenok <c_mykolak@mellanox.com>
Did Rob ever Ack this patch ?
Note you didn't copy the hwmon mailing list, which doesn't exactly help
to keep track of it, and I may have lost Rob's Ack. My apologies if that
is the case.
Guenter
> ---
> Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> index cf44605..367c820 100644
> --- a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> +++ b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> @@ -11,6 +11,8 @@ Required properties for pwm-tacho node:
>
> - #size-cells : should be 1.
>
> +- #cooling-cells: should be 2.
> +
> - reg : address and length of the register set for the device.
>
> - pinctrl-names : a pinctrl state named "default" must be defined.
> @@ -28,12 +30,17 @@ fan subnode format:
> Under fan subnode there can upto 8 child nodes, with each child node
> representing a fan. If there are 8 fans each fan can have one PWM port and
> one/two Fan tach inputs.
> +For PWM port can be configured cooling-levels to create cooling device.
> +Cooling device could be bound to a thermal zone for the thermal control.
>
> Required properties for each child node:
> - reg : should specify PWM source port.
> integer value in the range 0 to 7 with 0 indicating PWM port A and
> 7 indicating PWM port H.
>
> +- cooling-levels: PWM duty cycle values in a range from 0 to 255
> + which correspond to thermal cooling states.
> +
> - aspeed,fan-tach-ch : should specify the Fan tach input channel.
> integer value in the range 0 through 15, with 0 indicating
> Fan tach channel 0 and 15 indicating Fan tach channel 15.
> @@ -50,6 +57,7 @@ pwm_tacho_fixed_clk: fixedclk {
> pwm_tacho: pwmtachocontroller@1e786000 {
> #address-cells = <1>;
> #size-cells = <1>;
> + #cooling-cells = <2>;
> reg = <0x1E786000 0x1000>;
> compatible = "aspeed,ast2500-pwm-tacho";
> clocks = <&pwm_tacho_fixed_clk>;
> @@ -58,6 +66,7 @@ pwm_tacho: pwmtachocontroller@1e786000 {
>
> fan@0 {
> reg = <0x00>;
> + cooling-levels = /bits/ 8 <125 151 177 203 229 255>;
> aspeed,fan-tach-ch = /bits/ 8 <0x00>;
> };
>
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
2017-08-09 4:08 ` Guenter Roeck
@ 2017-08-09 7:15 ` Mykola Kostenok
-1 siblings, 0 replies; 12+ messages in thread
From: Mykola Kostenok @ 2017-08-09 7:15 UTC (permalink / raw)
To: Guenter Roeck, Rob Herring, devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Jaghathiswari Rankappagounder Natarajan,
openbmc-uLR06cmDAlY/bJ5BZ2RsiQ, Patrick Venture, Vadim Pasternak,
Ohad Oz
> -----Original Message-----
> From: Guenter Roeck [mailto:linux@roeck-us.net]
> Sent: Wednesday, August 9, 2017 7:09 AM
> To: Mykola Kostenok <c_mykolak@mellanox.com>; Rob Herring
> <robh+dt@kernel.org>; devicetree@vger.kernel.org
> Cc: Jaghathiswari Rankappagounder Natarajan <jaghu@google.com>;
> openbmc@lists.ozlabs.org; Patrick Venture <venture@google.com>; Vadim
> Pasternak <vadimp@mellanox.com>; Ohad Oz <ohado@mellanox.com>
> Subject: Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-
> tacho cooling device.
>
> On 08/03/2017 01:46 AM, Mykola Kostenok wrote:
> > It's add support for cooling device creation to aspeed-pwm-tacho.
> > Cooling device could be bound to a thermal zone for the thermal
> > control.
> >
> > v1 -> v2:
> > - Fix commit message.
> > v2 -> v3:
> > Pointed out by Rob Herring:
> > - Put cooling-levels under fan subnodes.
> >
> > Signed-off-by: Mykola Kostenok <c_mykolak@mellanox.com>
>
> Did Rob ever Ack this patch ?
>
> Note you didn't copy the hwmon mailing list, which doesn't exactly help to
> keep track of it, and I may have lost Rob's Ack. My apologies if that is the
> case.
>
> Guenter
Hi, Guenter.
No, Rob Herring did not ack patch yet. Just comment.
Yes, since I split initial patch to 2 patches, I have sent one to you and hwmon mail list,
and another to Rob and devicetree mail list. And added you both to cc.
Am I need to add both mail lists to both patches in such cases?
Best regards, Mykola Kostenok.
^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
@ 2017-08-09 7:15 ` Mykola Kostenok
0 siblings, 0 replies; 12+ messages in thread
From: Mykola Kostenok @ 2017-08-09 7:15 UTC (permalink / raw)
To: Guenter Roeck, Rob Herring, devicetree
Cc: Jaghathiswari Rankappagounder Natarajan, openbmc,
Patrick Venture, Vadim Pasternak, Ohad Oz
> -----Original Message-----
> From: Guenter Roeck [mailto:linux@roeck-us.net]
> Sent: Wednesday, August 9, 2017 7:09 AM
> To: Mykola Kostenok <c_mykolak@mellanox.com>; Rob Herring
> <robh+dt@kernel.org>; devicetree@vger.kernel.org
> Cc: Jaghathiswari Rankappagounder Natarajan <jaghu@google.com>;
> openbmc@lists.ozlabs.org; Patrick Venture <venture@google.com>; Vadim
> Pasternak <vadimp@mellanox.com>; Ohad Oz <ohado@mellanox.com>
> Subject: Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-
> tacho cooling device.
>
> On 08/03/2017 01:46 AM, Mykola Kostenok wrote:
> > It's add support for cooling device creation to aspeed-pwm-tacho.
> > Cooling device could be bound to a thermal zone for the thermal
> > control.
> >
> > v1 -> v2:
> > - Fix commit message.
> > v2 -> v3:
> > Pointed out by Rob Herring:
> > - Put cooling-levels under fan subnodes.
> >
> > Signed-off-by: Mykola Kostenok <c_mykolak@mellanox.com>
>
> Did Rob ever Ack this patch ?
>
> Note you didn't copy the hwmon mailing list, which doesn't exactly help to
> keep track of it, and I may have lost Rob's Ack. My apologies if that is the
> case.
>
> Guenter
Hi, Guenter.
No, Rob Herring did not ack patch yet. Just comment.
Yes, since I split initial patch to 2 patches, I have sent one to you and hwmon mail list,
and another to Rob and devicetree mail list. And added you both to cc.
Am I need to add both mail lists to both patches in such cases?
Best regards, Mykola Kostenok.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
2017-08-09 7:15 ` Mykola Kostenok
@ 2017-08-09 13:30 ` Guenter Roeck
-1 siblings, 0 replies; 12+ messages in thread
From: Guenter Roeck @ 2017-08-09 13:30 UTC (permalink / raw)
To: Mykola Kostenok, Rob Herring, devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: Jaghathiswari Rankappagounder Natarajan,
openbmc-uLR06cmDAlY/bJ5BZ2RsiQ, Patrick Venture, Vadim Pasternak,
Ohad Oz
On 08/09/2017 12:15 AM, Mykola Kostenok wrote:
>> -----Original Message-----
>> From: Guenter Roeck [mailto:linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org]
>> Sent: Wednesday, August 9, 2017 7:09 AM
>> To: Mykola Kostenok <c_mykolak-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>; Rob Herring
>> <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>; devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
>> Cc: Jaghathiswari Rankappagounder Natarajan <jaghu-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>;
>> openbmc-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org; Patrick Venture <venture-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>; Vadim
>> Pasternak <vadimp-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>; Ohad Oz <ohado-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
>> Subject: Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-
>> tacho cooling device.
>>
>> On 08/03/2017 01:46 AM, Mykola Kostenok wrote:
>>> It's add support for cooling device creation to aspeed-pwm-tacho.
>>> Cooling device could be bound to a thermal zone for the thermal
>>> control.
>>>
>>> v1 -> v2:
>>> - Fix commit message.
>>> v2 -> v3:
>>> Pointed out by Rob Herring:
>>> - Put cooling-levels under fan subnodes.
>>>
>>> Signed-off-by: Mykola Kostenok <c_mykolak-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
>>
>> Did Rob ever Ack this patch ?
>>
>> Note you didn't copy the hwmon mailing list, which doesn't exactly help to
>> keep track of it, and I may have lost Rob's Ack. My apologies if that is the
>> case.
>>
>> Guenter
>
> Hi, Guenter.
>
> No, Rob Herring did not ack patch yet. Just comment.
>
> Yes, since I split initial patch to 2 patches, I have sent one to you and hwmon mail list,
> and another to Rob and devicetree mail list. And added you both to cc.
> Am I need to add both mail lists to both patches in such cases?
>
Normally the dt patch would still be pushed through the same subsystem as the driver,
so, yes, it would be useful to have hwmon copied. Otherwise it won't be in the hwmon
patchwork and can easily get lost.
Guenter
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
@ 2017-08-09 13:30 ` Guenter Roeck
0 siblings, 0 replies; 12+ messages in thread
From: Guenter Roeck @ 2017-08-09 13:30 UTC (permalink / raw)
To: Mykola Kostenok, Rob Herring, devicetree
Cc: Jaghathiswari Rankappagounder Natarajan, openbmc,
Patrick Venture, Vadim Pasternak, Ohad Oz
On 08/09/2017 12:15 AM, Mykola Kostenok wrote:
>> -----Original Message-----
>> From: Guenter Roeck [mailto:linux@roeck-us.net]
>> Sent: Wednesday, August 9, 2017 7:09 AM
>> To: Mykola Kostenok <c_mykolak@mellanox.com>; Rob Herring
>> <robh+dt@kernel.org>; devicetree@vger.kernel.org
>> Cc: Jaghathiswari Rankappagounder Natarajan <jaghu@google.com>;
>> openbmc@lists.ozlabs.org; Patrick Venture <venture@google.com>; Vadim
>> Pasternak <vadimp@mellanox.com>; Ohad Oz <ohado@mellanox.com>
>> Subject: Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-
>> tacho cooling device.
>>
>> On 08/03/2017 01:46 AM, Mykola Kostenok wrote:
>>> It's add support for cooling device creation to aspeed-pwm-tacho.
>>> Cooling device could be bound to a thermal zone for the thermal
>>> control.
>>>
>>> v1 -> v2:
>>> - Fix commit message.
>>> v2 -> v3:
>>> Pointed out by Rob Herring:
>>> - Put cooling-levels under fan subnodes.
>>>
>>> Signed-off-by: Mykola Kostenok <c_mykolak@mellanox.com>
>>
>> Did Rob ever Ack this patch ?
>>
>> Note you didn't copy the hwmon mailing list, which doesn't exactly help to
>> keep track of it, and I may have lost Rob's Ack. My apologies if that is the
>> case.
>>
>> Guenter
>
> Hi, Guenter.
>
> No, Rob Herring did not ack patch yet. Just comment.
>
> Yes, since I split initial patch to 2 patches, I have sent one to you and hwmon mail list,
> and another to Rob and devicetree mail list. And added you both to cc.
> Am I need to add both mail lists to both patches in such cases?
>
Normally the dt patch would still be pushed through the same subsystem as the driver,
so, yes, it would be useful to have hwmon copied. Otherwise it won't be in the hwmon
patchwork and can easily get lost.
Guenter
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
2017-08-03 8:46 ` Mykola Kostenok
@ 2017-08-10 17:11 ` Rob Herring
-1 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2017-08-10 17:11 UTC (permalink / raw)
To: Mykola Kostenok
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, Guenter Roeck,
Jaghathiswari Rankappagounder Natarajan,
openbmc-uLR06cmDAlY/bJ5BZ2RsiQ, Patrick Venture, Vadim Pasternak,
Ohad Oz
On Thu, Aug 03, 2017 at 11:46:50AM +0300, Mykola Kostenok wrote:
> It's add support for cooling device creation to
> aspeed-pwm-tacho.
> Cooling device could be bound to a thermal zone
> for the thermal control.
>
> v1 -> v2:
> - Fix commit message.
> v2 -> v3:
> Pointed out by Rob Herring:
> - Put cooling-levels under fan subnodes.
>
> Signed-off-by: Mykola Kostenok <c_mykolak-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
> ---
> Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt | 9 +++++++++
> 1 file changed, 9 insertions(+)
Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
@ 2017-08-10 17:11 ` Rob Herring
0 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2017-08-10 17:11 UTC (permalink / raw)
To: Mykola Kostenok
Cc: devicetree, Guenter Roeck,
Jaghathiswari Rankappagounder Natarajan, openbmc,
Patrick Venture, Vadim Pasternak, Ohad Oz
On Thu, Aug 03, 2017 at 11:46:50AM +0300, Mykola Kostenok wrote:
> It's add support for cooling device creation to
> aspeed-pwm-tacho.
> Cooling device could be bound to a thermal zone
> for the thermal control.
>
> v1 -> v2:
> - Fix commit message.
> v2 -> v3:
> Pointed out by Rob Herring:
> - Put cooling-levels under fan subnodes.
>
> Signed-off-by: Mykola Kostenok <c_mykolak@mellanox.com>
> ---
> Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt | 9 +++++++++
> 1 file changed, 9 insertions(+)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [v3, resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
2017-08-03 8:46 ` Mykola Kostenok
@ 2017-08-13 13:47 ` Guenter Roeck
-1 siblings, 0 replies; 12+ messages in thread
From: Guenter Roeck @ 2017-08-13 13:47 UTC (permalink / raw)
To: Mykola Kostenok
Cc: Rob Herring, devicetree-u79uwXL29TY76Z2rM5mHXA,
Jaghathiswari Rankappagounder Natarajan, Ohad Oz,
Vadim Pasternak, Patrick Venture, openbmc-uLR06cmDAlY/bJ5BZ2RsiQ
On Thu, Aug 03, 2017 at 11:46:50AM +0300, Mykola Kostenok wrote:
> It's add support for cooling device creation to
> aspeed-pwm-tacho.
> Cooling device could be bound to a thermal zone
> for the thermal control.
>
> v1 -> v2:
> - Fix commit message.
> v2 -> v3:
> Pointed out by Rob Herring:
> - Put cooling-levels under fan subnodes.
>
> Signed-off-by: Mykola Kostenok <c_mykolak-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
> Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Applied to hwmon-next (removed version information from description)
Thanks,
Guenter
> ---
> Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> index cf44605..367c820 100644
> --- a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> +++ b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> @@ -11,6 +11,8 @@ Required properties for pwm-tacho node:
>
> - #size-cells : should be 1.
>
> +- #cooling-cells: should be 2.
> +
> - reg : address and length of the register set for the device.
>
> - pinctrl-names : a pinctrl state named "default" must be defined.
> @@ -28,12 +30,17 @@ fan subnode format:
> Under fan subnode there can upto 8 child nodes, with each child node
> representing a fan. If there are 8 fans each fan can have one PWM port and
> one/two Fan tach inputs.
> +For PWM port can be configured cooling-levels to create cooling device.
> +Cooling device could be bound to a thermal zone for the thermal control.
>
> Required properties for each child node:
> - reg : should specify PWM source port.
> integer value in the range 0 to 7 with 0 indicating PWM port A and
> 7 indicating PWM port H.
>
> +- cooling-levels: PWM duty cycle values in a range from 0 to 255
> + which correspond to thermal cooling states.
> +
> - aspeed,fan-tach-ch : should specify the Fan tach input channel.
> integer value in the range 0 through 15, with 0 indicating
> Fan tach channel 0 and 15 indicating Fan tach channel 15.
> @@ -50,6 +57,7 @@ pwm_tacho_fixed_clk: fixedclk {
> pwm_tacho: pwmtachocontroller@1e786000 {
> #address-cells = <1>;
> #size-cells = <1>;
> + #cooling-cells = <2>;
> reg = <0x1E786000 0x1000>;
> compatible = "aspeed,ast2500-pwm-tacho";
> clocks = <&pwm_tacho_fixed_clk>;
> @@ -58,6 +66,7 @@ pwm_tacho: pwmtachocontroller@1e786000 {
>
> fan@0 {
> reg = <0x00>;
> + cooling-levels = /bits/ 8 <125 151 177 203 229 255>;
> aspeed,fan-tach-ch = /bits/ 8 <0x00>;
> };
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [v3, resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
@ 2017-08-13 13:47 ` Guenter Roeck
0 siblings, 0 replies; 12+ messages in thread
From: Guenter Roeck @ 2017-08-13 13:47 UTC (permalink / raw)
To: Mykola Kostenok
Cc: Rob Herring, devicetree, Jaghathiswari Rankappagounder Natarajan,
Ohad Oz, Vadim Pasternak, Patrick Venture, openbmc
On Thu, Aug 03, 2017 at 11:46:50AM +0300, Mykola Kostenok wrote:
> It's add support for cooling device creation to
> aspeed-pwm-tacho.
> Cooling device could be bound to a thermal zone
> for the thermal control.
>
> v1 -> v2:
> - Fix commit message.
> v2 -> v3:
> Pointed out by Rob Herring:
> - Put cooling-levels under fan subnodes.
>
> Signed-off-by: Mykola Kostenok <c_mykolak@mellanox.com>
> Acked-by: Rob Herring <robh@kernel.org>
Applied to hwmon-next (removed version information from description)
Thanks,
Guenter
> ---
> Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> index cf44605..367c820 100644
> --- a/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> +++ b/Documentation/devicetree/bindings/hwmon/aspeed-pwm-tacho.txt
> @@ -11,6 +11,8 @@ Required properties for pwm-tacho node:
>
> - #size-cells : should be 1.
>
> +- #cooling-cells: should be 2.
> +
> - reg : address and length of the register set for the device.
>
> - pinctrl-names : a pinctrl state named "default" must be defined.
> @@ -28,12 +30,17 @@ fan subnode format:
> Under fan subnode there can upto 8 child nodes, with each child node
> representing a fan. If there are 8 fans each fan can have one PWM port and
> one/two Fan tach inputs.
> +For PWM port can be configured cooling-levels to create cooling device.
> +Cooling device could be bound to a thermal zone for the thermal control.
>
> Required properties for each child node:
> - reg : should specify PWM source port.
> integer value in the range 0 to 7 with 0 indicating PWM port A and
> 7 indicating PWM port H.
>
> +- cooling-levels: PWM duty cycle values in a range from 0 to 255
> + which correspond to thermal cooling states.
> +
> - aspeed,fan-tach-ch : should specify the Fan tach input channel.
> integer value in the range 0 through 15, with 0 indicating
> Fan tach channel 0 and 15 indicating Fan tach channel 15.
> @@ -50,6 +57,7 @@ pwm_tacho_fixed_clk: fixedclk {
> pwm_tacho: pwmtachocontroller@1e786000 {
> #address-cells = <1>;
> #size-cells = <1>;
> + #cooling-cells = <2>;
> reg = <0x1E786000 0x1000>;
> compatible = "aspeed,ast2500-pwm-tacho";
> clocks = <&pwm_tacho_fixed_clk>;
> @@ -58,6 +66,7 @@ pwm_tacho: pwmtachocontroller@1e786000 {
>
> fan@0 {
> reg = <0x00>;
> + cooling-levels = /bits/ 8 <125 151 177 203 229 255>;
> aspeed,fan-tach-ch = /bits/ 8 <0x00>;
> };
>
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2017-08-13 13:47 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-03 8:46 [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device Mykola Kostenok
2017-08-03 8:46 ` Mykola Kostenok
[not found] ` <20170803084650.22298-1-c_mykolak-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2017-08-09 4:08 ` Guenter Roeck
2017-08-09 4:08 ` Guenter Roeck
[not found] ` <3c427ddd-aab6-13d4-bfb2-ae22848c0d13-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
2017-08-09 7:15 ` Mykola Kostenok
2017-08-09 7:15 ` Mykola Kostenok
[not found] ` <DB5PR05MB122451EAD6E4A470ED2FCFBEEF8B0-8IvNv+8VlcDncoQxOOXcUNqRiQSDpxhJvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2017-08-09 13:30 ` Guenter Roeck
2017-08-09 13:30 ` Guenter Roeck
2017-08-10 17:11 ` Rob Herring
2017-08-10 17:11 ` Rob Herring
2017-08-13 13:47 ` [v3, " Guenter Roeck
2017-08-13 13:47 ` Guenter Roeck
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.