Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm-tacho cooling device.
From: Guenter Roeck <hidden>
Date: 2017-08-09 13:30:51
Also in:
openbmc
On 08/09/2017 12:15 AM, Mykola Kostenok wrote:
quoted
-----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 [off-list ref]; devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: Jaghathiswari Rankappagounder Natarajan <redacted>; openbmc-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org; Patrick Venture [off-list ref]; Vadim Pasternak [off-list ref]; Ohad Oz [off-list ref] Subject: Re: [patch v3 resend] Documentation: dt-bindings: aspeed-pwm- tacho cooling device. On 08/03/2017 01:46 AM, Mykola Kostenok wrote:quoted
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. GuenterHi, 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