linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Eduardo Valentin <eduardo.valentin@ti.com>
To: Guenter Roeck <linux@roeck-us.net>
Cc: Eduardo Valentin <eduardo.valentin@ti.com>,
	Jean Delvare <khali@linux-fr.org>, <swarren@wwwdotorg.org>,
	<pawel.moll@arm.com>, <mark.rutland@arm.com>,
	<ian.campbell@citrix.com>, <rob.herring@calxeda.com>,
	<rui.zhang@intel.com>, <wni@nvidia.com>,
	<grant.likely@linaro.org>, <durgadoss.r@intel.com>,
	<linux-pm@vger.kernel.org>, <devicetree@vger.kernel.org>,
	<lm-sensors@lm-sensors.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCHv5 06/20] hwmon: lm75: expose to thermal fw via DT nodes
Date: Mon, 18 Nov 2013 12:40:53 -0400	[thread overview]
Message-ID: <528A4315.8050706@ti.com> (raw)
In-Reply-To: <20131118162532.GB15610@roeck-us.net>

[-- Attachment #1: Type: text/plain, Size: 1762 bytes --]

On 18-11-2013 12:25, Guenter Roeck wrote:
> On Mon, Nov 18, 2013 at 10:27:41AM -0400, Eduardo Valentin wrote:
>> On 15-11-2013 03:43, Jean Delvare wrote:
>>> Hi Eduardo,
>>>
>>
>> Hello Jean!
>>
>>> Sorry for joining the discussion a little late, I could never find the
>>> time to look into this patch series so far.
>>
>> Well, better late than never, that's what it is said, isn't it? :-)
>> Thanks for arranging the time to look at these patches.
>>
>> The patch series have been looked and contributed by several people and
>> a couple o maintainers now. Starts to be way better than the original
>> RFC. The core idea still holds though.
>>
>>>
>>> On Tue, 12 Nov 2013 15:46:08 -0400, Eduardo Valentin wrote:
>>>> This patch adds to lm75 temperature sensor the possibility
>>>> to expose itself as thermal zone device, registered on the
>>>> thermal framework.
>>>>
>>>> The thermal zone is built only if a device tree node
>>>> describing a thermal zone for this sensor is present
>>>> inside the lm75 DT node. Otherwise, the driver behavior
>>>> will be the same.
>>>>
>>>> Cc: Jean Delvare <khali@linux-fr.org>
>>>> Cc: lm-sensors@lm-sensors.org
>>>> Cc: linux-kernel@vger.kernel.org
>>>> Acked-by: Guenter Roeck <linux@roeck-us.net>
>>
>> On HWMON side, I got Guenter's solid contributions, as you can see.
>>
> Not an argument. I am not perfect, and Jean has a number of valid points.
> 

In fact, this is not an argument, at least not on what Jean is bringing
now. The point here was just to mention that, even though he joined
late, you have already contributed on this patch series. ;-)

> Guenter
> 
> 


-- 
You have got to be excited about what you are doing. (L. Lamport)

Eduardo Valentin


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 295 bytes --]

  reply	other threads:[~2013-11-18 16:43 UTC|newest]

Thread overview: 80+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-12 19:46 [PATCHv5 00/20] device thermal limits represented in device tree nodes (v5) Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 01/20] thermal: allow registering without .get_temp Eduardo Valentin
2013-11-12 19:46 ` [PATCHv9 02/20] thermal: introduce device tree parser Eduardo Valentin
2013-11-13 16:57   ` Tomasz Figa
2013-11-14 11:31     ` Eduardo Valentin
2013-11-14 13:40       ` Tomasz Figa
2013-11-15 13:19         ` Eduardo Valentin
2013-11-21 14:57           ` Tomasz Figa
2013-11-21 15:48             ` Eduardo Valentin
2013-11-21 16:32               ` Tomasz Figa
2013-11-22 12:33                 ` Eduardo Valentin
2013-11-25 15:31                   ` Mark Rutland
2013-11-25 15:40                     ` Eduardo Valentin
2013-11-25 15:41                     ` Eduardo Valentin
2013-11-25 15:14             ` Mark Rutland
2013-11-25 15:34               ` Eduardo Valentin
2013-11-15  8:07   ` [lm-sensors] " Jean Delvare
2013-11-18  6:04     ` Zhang Rui
2013-11-18 14:45       ` Eduardo Valentin
2013-11-19 14:43         ` Jean Delvare
2013-11-25 15:37   ` Mark Rutland
2013-11-25 15:47     ` Eduardo Valentin
2013-12-31 10:17   ` Wei Ni
2014-01-07  2:48     ` Wei Ni
2014-01-07 11:17       ` Eduardo Valentin
2014-01-08  3:19         ` Wei Ni
     [not found]           ` <CAHqbYQvchi3QSgcitUtguFyOJtXhFt5OjcoiSDZnPg9ZyiN4cg@mail.gmail.com>
2014-01-08  4:16             ` Wei Ni
2014-01-02  2:55   ` Wei Ni
2014-01-02  3:03     ` Wei Ni
2014-01-02  2:59   ` Wei Ni
2014-01-02 17:50     ` Matthew Longnecker
2014-01-06 13:51       ` Mark Rutland
2014-01-06 14:54         ` Eduardo Valentin
2014-01-07  2:44           ` Wei Ni
2014-01-07 12:02             ` Mark Rutland
2014-01-13 21:29             ` Eduardo Valentin
2014-01-14  2:54               ` Wei Ni
2014-01-14 18:48                 ` Eduardo Valentin
2014-01-13 15:37       ` Eduardo Valentin
2014-01-02 17:35   ` Matthew Longnecker
2014-01-06 18:52     ` Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 03/20] thermal: core: introduce thermal_of_cooling_device_register Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 04/20] thermal: cpu_cooling: introduce of_cpufreq_cooling_register Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 05/20] cpufreq: cpufreq-cpu0: add dt node parsing for cooling device properties Eduardo Valentin
2013-11-14 13:17   ` Eduardo Valentin
2013-11-14 22:04     ` Rafael J. Wysocki
2013-11-15  4:41   ` viresh kumar
2014-01-12 14:31   ` Zhang, Rui
2014-01-13 15:08     ` Eduardo Valentin
2014-01-14 19:07     ` Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 06/20] hwmon: lm75: expose to thermal fw via DT nodes Eduardo Valentin
2013-11-15  7:43   ` Jean Delvare
2013-11-18 14:27     ` Eduardo Valentin
2013-11-18 16:25       ` Guenter Roeck
2013-11-18 16:40         ` Eduardo Valentin [this message]
2013-11-19  9:39       ` Jean Delvare
2013-11-22 14:37         ` Eduardo Valentin
2013-11-23 18:38           ` Jean Delvare
2013-11-12 19:46 ` [PATCHv5 07/20] hwmon: tmp102: " Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 08/20] thermal: ti-soc-thermal: use thermal DT infrastructure Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 09/20] arm: dts: add omap4 CPU thermal data Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 10/20] arm: dts: add omap4430 " Eduardo Valentin
2013-11-20 12:32   ` Pavel Machek
2013-11-21 15:36     ` Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 11/20] arm: dts: add omap4460 " Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 12/20] arm: dts: add cooling properties on omap4430 cpu node Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 13/20] arm: dts: add cooling properties on omap4460 " Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 14/20] arm: dts: add omap5 GPU thermal data Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 15/20] arm: dts: add omap5 CORE " Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 16/20] arm: dts: add omap5 " Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 17/20] arm: dts: add cooling properties on omap5 cpu node Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 18/20] arm: dts: make OMAP443x bandgap node to belong to OCP Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 19/20] arm: dts: make OMAP4460 " Eduardo Valentin
2013-11-12 19:46 ` [PATCHv5 20/20] MAINTAINERS: add maintainer entry for thermal bindings Eduardo Valentin
2013-11-12 19:59   ` Olof Johansson
2013-11-12 20:14     ` Eduardo Valentin
2013-11-13  9:42       ` Mark Rutland
2013-11-13 12:17         ` Eduardo Valentin
2013-11-13 14:46         ` Eduardo Valentin
2013-11-14 13:30         ` [PATCHv6 20/20] MAINTAINERS: add thermal bindings entry in thermal domain Eduardo Valentin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=528A4315.8050706@ti.com \
    --to=eduardo.valentin@ti.com \
    --cc=devicetree@vger.kernel.org \
    --cc=durgadoss.r@intel.com \
    --cc=grant.likely@linaro.org \
    --cc=ian.campbell@citrix.com \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=lm-sensors@lm-sensors.org \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=rob.herring@calxeda.com \
    --cc=rui.zhang@intel.com \
    --cc=swarren@wwwdotorg.org \
    --cc=wni@nvidia.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).