Hello,
Perhaps you may have sent this email to the wrong Eduardo as I do not recognize
the topic you are discussing.
Regards,
Sent from my iPhone
> On Jul 24, 2014, at 11:15 AM, "Zhang, Rui" wrote:
>
> Hi, Laxman,
>
> As Eduardo is the of thermal author and maintainer, I will take your patc
Hello,
On Mon, Jun 09, 2014 at 09:22:49AM -0400, Nicolas Pitre wrote:
> On Mon, 9 Jun 2014, Morten Rasmussen wrote:
>
> > On Sat, Jun 07, 2014 at 03:33:58AM +0100, Nicolas Pitre wrote:
> > > On Fri, 6 Jun 2014, Ingo Molnar wrote:
> > >
> > > > In any case, even with turbo frequencies, switching
> derive specific power actors, such as a cpu power actor.
>
> Cc: Zhang Rui
> Cc: Eduardo Valentin
> Signed-off-by: Javi Merino
> ---
> Documentation/thermal/power_actor.txt | 38 ++
> drivers/thermal/Kconfig | 3 ++
> drivers/th
On Wed, Jun 11, 2014 at 12:42:18PM +0100, Morten Rasmussen wrote:
> On Wed, Jun 11, 2014 at 12:02:51PM +0100, Eduardo Valentin wrote:
> > Hello,
> >
> > On Mon, Jun 09, 2014 at 09:22:49AM -0400, Nicolas Pitre wrote:
> > > On Mon, 9 Jun 2014, Morten Rasmussen wrote:
> cpufreq) as well as load and OPPs for the power calculations. The
> cpus must have registered their OPPs in the OPP library.
>
> Cc: Zhang Rui
> Cc: Eduardo Valentin
> Signed-off-by: Punit Agrawal
> Signed-off-by: Javi Merino
> ---
> Documentation/thermal/power_ac
I am updating my contact email address. For now on, patches
on non ACPI / SoC thermal must be sent to edubez...@gmail.com.
Signed-off-by: Eduardo Valentin
Signed-off-by: Eduardo Valentin
---
MAINTAINERS | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/MAINTAINERS b
Hello Viresh,
On Thu, May 22, 2014 at 11:07:25AM +0530, Viresh Kumar wrote:
> cpufreq-cpu0 uses thermal framework to register a cooling device, but doesn't
> depend on it as there are dummy calls provided by thermal layer when
> CONFIG_THERMAL=n. So, we don't really need to mention thermal as a de
Hello Javi,
On Fri, May 16, 2014 at 12:16:08PM +0100, Javi Merino wrote:
> Document struct struct thermal_zone_device and struct thermal_governor
> fields and their use by the thermal framework code.
>
> Cc: Zhang Rui
> Cc: Eduardo Valentin
> Signed-off-by: Javi Merino
>
Hi Viresh,
On Fri, May 23, 2014 at 10:03:27AM +0530, Viresh Kumar wrote:
> On 22 May 2014 20:22, Eduardo Valentin wrote:
> > However, on CPUs that needs thermal managment, it makes sense to have
> > such dependency, from functional perspective. Mainly because scaling
> > fre
Hello Javi,
On Fri, May 23, 2014 at 10:35:52AM +0100, Javi Merino wrote:
> On Thu, May 22, 2014 at 04:27:30PM +0100, Eduardo Valentin wrote:
> > Hello Javi,
> >
> > On Fri, May 16, 2014 at 12:16:08PM +0100, Javi Merino wrote:
> >
> > > Document struct s
On Sun, Jun 01, 2014 at 01:33:51PM +0200, Rickard Strandqvist wrote:
> There is a risk for memory leak in when something unexpected happens
> and the function returns.
I don't think there is a risk of memory leak, but wrong address access,
in this case.
>
> This was largely found by using a stat
l/2014/4/2/644
Maybe it's fallen into the cracks.
Rui, any idea what is the status of that patch?
> Cc: Eduardo Valentin
> Signed-off-by: Lee Jones
> ---
> MAINTAINERS | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/MAINTAINERS b/MAI
Hello Amit, Javi,
On Mon, Jun 02, 2014 at 11:20:48AM +0100, Javi Merino wrote:
> On Mon, Jun 02, 2014 at 10:21:48AM +0100, Amit Kachhap wrote:
> > Hi Javi,
> >
> > On 5/29/14, Javi Merino wrote:
> > > Hi Amit,
> > >
> > > On Thu, May 29, 2014 at 09:15:34AM +0100, Amit Daniel Kachhap wrote:
> > >
On Mon, Jun 02, 2014 at 02:54:04PM +0530, Amit Kachhap wrote:
> On 5/29/14, Javi Merino wrote:
> > Hi Amit,
> >
> > One minor comment.
> >
> > On Thu, May 29, 2014 at 09:15:30AM +0100, Amit Daniel Kachhap wrote:
> >> This patch allows the caller of cpufreq cooling APIs to register along
> >> with
seconds. Let me know if you'd like to see the graphs.
Yeah, I would like to see it. But what I was more interested in seeing
is how long does it take to offline a CPU?
> Regards, Zoran
>
> On 29 November 2013 06:08, Eduardo Valentin wrote:
>> Hello Zoran,
>>
>> On
Lukasz Majewski
> Signed-off-by: Myungjoo Ham
Acked-by: Eduardo Valentin
Looks better to me now. You might consider refactoring this driver to be
frequency independent, if possible. But that is a different subject
other than this patch.
>
> ---
> Changes for v11:
> -
On 13-11-2013 05:42, Mark Rutland wrote:
> Hi,
>
> On Tue, Nov 12, 2013 at 08:14:41PM +, Eduardo Valentin wrote:
>> On 12-11-2013 15:59, Olof Johansson wrote:
>>> On Tue, Nov 12, 2013 at 11:46 AM, Eduardo Valentin
>>> wrote:
>>>> After discussion
On 13-11-2013 05:42, Mark Rutland wrote:
> Hi,
>
> On Tue, Nov 12, 2013 at 08:14:41PM +, Eduardo Valentin wrote:
>> On 12-11-2013 15:59, Olof Johansson wrote:
>>> On Tue, Nov 12, 2013 at 11:46 AM, Eduardo Valentin
>>> wrote:
>>>> After discussion
ranging max_level properly.
Reported-by: Carlos Hernandez
Signed-off-by: Eduardo Valentin
---
drivers/thermal/cpu_cooling.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c
index d179028..d0f8f8b5 100644
--- a
On 13-11-2013 14:11, Eduardo Valentin wrote:
> As per Documentation/thermal/sysfs-api.txt, max_level
> is an index, not a counter. Thus, in case a CPU has
> 3 valid frequencies, max_level is expected to be 2, for instance.
>
> The current code makes max_level == number of va
On 13-11-2013 12:57, Tomasz Figa wrote:
> Hi Eduardo,
>
Hello Tomaz
> On Tuesday 12 of November 2013 15:46:04 Eduardo Valentin wrote:
>> This patch introduces a device tree bindings for
>> describing the hardware thermal behavior and limits.
>> Also a parser to read
On 12-11-2013 15:46, Eduardo Valentin wrote:
> This patch changes the cpufreq-cpu0 driver to consider if
> a cpu needs cooling (with cpufreq). In case the cooling is needed,
> the cpu0 device tree node needs to be properly configured
> with cooling device properties.
>
> In cas
thermal domain.
Cc: Rob Herring
Cc: Pawel Moll
Cc: Mark Rutland
Cc: Stephen Warren
Cc: Ian Campbell
Signed-off-by: Eduardo Valentin
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 3438384..57fbf81 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
Hello Tomasz,
On 14-11-2013 09:40, Tomasz Figa wrote:
> On Thursday 14 of November 2013 07:31:04 Eduardo Valentin wrote:
>> On 13-11-2013 12:57, Tomasz Figa wrote:
>>> Hi Eduardo,
>>>
>>
>> Hello Tomaz
>>
>>> On Tuesday 12 of November
nging 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
Hello Jean,
I will try to complement what Rui's already commented.
On 18-11-2013 02:04, Zhang Rui wrote:
> On 五, 2013-11-15 at 09:07 +0100, Jean Delvare wrote:
>> Hi Eduardo,
>>
>> On Tue, 12 Nov 2013 15:46:04 -0400, Eduardo Valentin wrote:
>>> This patch in
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 l
os_tmu_data.c | 47
>>>>>>> + 1 file changed, 47 insertions(+)
>>>>>>>
>>>>>>> diff --git a/drivers/thermal/samsung/exynos_tmu_data.c
>>>>>>> b/drivers/thermal/samsung/exynos_tmu_data.c index
>>>>>>> 073c292..9346926 100644 ---
>>>>>>> a/drivers/thermal/samsung/exynos_tmu_data.c +++
>>>>>>> b/drivers/thermal/samsung/exynos_tmu_data.c @@ -167,13 +167,60
>>>>>>> @@ static const struct exynos_tmu_registers
>>>>>>> exynos4412_tmu_registers = { .features = (TMU_SUPPORT_EMULATION
>>>>>>> | TMU_SUPPORT_TRIM_RELOAD | \ TMU_SUPPORT_FALLING_TRIP |
>>>>>>> TMU_SUPPORT_READY_STATUS | \ TMU_SUPPORT_EMUL_TIME) +
>>>>>>> +#define EXYNOS4412_TMU_DATA_BOOST \
>>>>>>> + .threshold_falling = 10, \
>>>>>>> + .trigger_levels[0] = 70, \
>>>>>>> + .trigger_levels[1] = 85, \
>>>>>>> + .trigger_levels[2] = 103, \
>>>>>>> + .trigger_levels[3] = 110, \
>>>>>>> + .trigger_enable[0] = true, \
>>>>>>> + .trigger_enable[1] = true, \
>>>>>>> + .trigger_enable[2] = true, \
>>>>>>> + .trigger_enable[3] = true, \
>>>>>>> + .trigger_type[0] = THROTTLE_ACTIVE, \
>>>>>>> + .trigger_type[1] = THROTTLE_ACTIVE, \
>>>>>>> + .trigger_type[2] = THROTTLE_ACTIVE, \
>>>>>>> + .trigger_type[3] = SW_TRIP, \
>>>>>>> + .max_trigger_level = 4, \
>>>>>>> + .gain = 8, \
>>>>>>> + .reference_voltage = 16, \
>>>>>>> + .noise_cancel_mode = 4, \
>>>>>>> + .cal_type = TYPE_ONE_POINT_TRIMMING, \
>>>>>>> + .efuse_value = 55, \
>>>>>>> + .min_efuse_value = 40, \
>>>>>>> + .max_efuse_value = 100, \
>>>>>>> + .first_point_trim = 25, \
>>>>>>> + .second_point_trim = 85, \
>>>>>>> + .default_temp_offset = 50, \
>>>>>>> + .freq_tab[0] = { \
>>>>>>> + .freq_clip_max = 1400 * 1000, \
>>>>>>> + .temp_level = 70, \
>>>>>>> + }, \
>>>>>>> + .freq_tab[1] = { \
>>>>>>> + .freq_clip_max = 800 * 1000, \
>>>>>>> + .temp_level = 85, \
>>>>>>> + }, \
>>>>>>> + .freq_tab[2] = { \
>>>>>>> + .freq_clip_max = 200 * 1000, \
>>>>>>> + .temp_level = 103, \
>>>>>>> + }, \
>>>>>>> + .freq_tab_count = 3, \
>>>>>>> + .registers = &exynos4412_tmu_registers, \
>>>>>>> + .features = (TMU_SUPPORT_EMULATION |
>>>>>>> TMU_SUPPORT_TRIM_RELOAD | \
>>>>>>> + TMU_SUPPORT_FALLING_TRIP |
>>>>>>> TMU_SUPPORT_READY_STATUS | \
>>>>>>> + TMU_SUPPORT_EMUL_TIME)
>>>>>>> #endif
>>>>>>>
>>>>>>> #if defined(CONFIG_SOC_EXYNOS4412)
>>>>>>> struct exynos_tmu_init_data const exynos4412_default_tmu_data
>>>>>>> = { .tmu_data = {
>>>>>>> {
>>>>>>> +#ifdef CONFIG_CPU_FREQ_BOOST_SW
>>>>>>> + EXYNOS4412_TMU_DATA_BOOST,
>>>>>>> +#else
>>>>>>> EXYNOS4412_TMU_DATA,
>>>>>>> +#endif
>>>>>>> .type = SOC_ARCH_EXYNOS4412,
>>>>>>> .test_mux = EXYNOS4412_MUX_ADDR_VALUE,
>>>>>>> },
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>
>>>>
>>>
>>>
>>>
>>
>>
>
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
On 13-11-2013 14:11, Eduardo Valentin wrote:
> As per Documentation/thermal/sysfs-api.txt, max_level
> is an index, not a counter. Thus, in case a CPU has
> 3 valid frequencies, max_level is expected to be 2, for instance.
>
> The current code makes max_level == number of va
On 02-01-2014 00:02, Zhang Rui wrote:
> On Fri, 2013-12-06 at 09:52 -0400, Eduardo Valentin wrote:
>> On 13-11-2013 14:11, Eduardo Valentin wrote:
>>> As per Documentation/thermal/sysfs-api.txt, max_level
>>> is an index, not a counter. Thus, in case a CPU has
>>&g
patch series has been runtime tested on DRA7-evm.
All best
Eduardo Valentin (7):
arm: dts: add dra7 DSPEVE thermal data
arm: dts: add dra7 IVA thermal data
arm: dts: dra7: add bandgap entry
arm: dts: add cooling properties on dra7 cpu node
arm: dts: dra7: add thermal data
arm: dts
Cc: devicet...@vger.kernel.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/dra7-dspeve-thermal.dtsi | 28
1 file changed, 28 insertions(+)
create mode 100644 arch/arm/boot/dts/dra7
: devicet...@vger.kernel.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/dra7-iva-thermal.dtsi | 28
1 file changed, 28 insertions(+)
create mode 100644 arch/arm/boot/dts/dra7-iva
@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/dra7.dtsi | 12
1 file changed, 12 insertions(+)
diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index d0df4c4..8f21157 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm/boot/dts/dra7
: Stephen Warren
Cc: Ian Campbell
Cc: Russell King
Cc: devicet...@vger.kernel.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/dra7.dtsi | 9 +
1 file changed, 9 insertions(+)
diff --git a/arch/arm/boot/dts
Rutland
Cc: Stephen Warren
Cc: Ian Campbell
Cc: Russell King
Cc: devicet...@vger.kernel.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/dra7.dtsi | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm
...@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/dra7-evm.dts | 7 +++
1 file changed, 7 insertions(+)
diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
index 5babba0..36e2c9e 100644
--- a/arch/arm/boot/dts
: Eduardo Valentin
---
arch/arm/boot/dts/dra7-evm.dts | 15 +++
1 file changed, 15 insertions(+)
diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
index 36e2c9e..1ea9c3d 100644
--- a/arch/arm/boot/dts/dra7-evm.dts
+++ b/arch/arm/boot/dts/dra7-evm.dts
@@ -95,6
Cc: devicet...@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/thermal/imx_thermal.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/thermal/imx_thermal.c b/drivers/thermal/imx_thermal.c
index 77a962f..43d2f77 100644
--- a/drivers/thermal/imx_thermal.c
Cc: Pavel Machek
Cc: "Rafael J. Wysocki"
Cc: Len Brown
Cc: Greg Kroah-Hartman
Cc: linux...@vger.kernel.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/thermal/Kconfig | 12 +
drivers/thermal/Makefile
On 06-01-2014 10:46, Zhang Rui wrote:
> On Mon, 2014-01-06 at 08:36 -0400, Eduardo Valentin wrote:
>> Hello all,
>>
>> Now that the basic thermal DT support is merged into Rui's tree,
>> I am resending this series, which introduces the DT node entries
>>
ameters can also change at any time.
>
> I'd prefer to not have governer details described in the dt, and the
> choice of governer and configuration of its tuning parameters should be
> made at runtime somehow.
Agreed.
>
> Thanks,
> Mark.
>
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
rs).
No issues, I also have been there.
>
> However, I have one substantive criticism.
>
> On 11/12/2013 11:46 AM, Eduardo Valentin wrote:
>> +* Thermal zone nodes
>> +
>> +The thermal zone node is the node containing all the required info
>> +for describing a
Hello Juha-Matti,
On Thu, Aug 21, 2014 at 03:20:11PM +0300, Juha-Matti Tilli wrote:
> On Thu, Aug 21, 2014 at 01:17:18PM +0300, Mikko Perttunen wrote:
> > this series adds support for the thermal monitoring features of the
> > soctherm unit on the Tegra124 SoC.
>
> For all patches in this series
Stephen,
On Wed, Aug 20, 2014 at 01:44:00PM -0600, Stephen Warren wrote:
> On 08/06/2014 04:25 AM, Mikko Perttunen wrote:
> > This adds critical trip points to the Jetson TK1 device tree.
> > The device will do a controlled shutdown when either the CPU, GPU
> > or MEM thermal zone reaches 101 degr
Hello Stephen,
On Wed, Aug 20, 2014 at 01:50:48PM -0600, Stephen Warren wrote:
> On 08/06/2014 04:25 AM, Mikko Perttunen wrote:
> > This adds support for the Tegra SOCTHERM thermal sensing and management
> > system found in the Tegra124 system-on-chip. This initial driver supports
> > temperature
.c| 5 +-
drivers/thermal/thermal_core.c | 7 ++
include/trace/events/thermal.h | 83
7 files changed, 200 insertions(+), 15 deletions(-)
create mode 100644 include/trace/events/thermal.h
BR,
Eduardo Valentin
--
To unsubscrib
Hello Wei Ni,
On Mon, Aug 25, 2014 at 02:29:47PM +0800, Wei Ni wrote:
> Add more description for the "polling-delay" property.
> Set "trips" and "cooling maps" as optional property, because
> if missing these two sub-nodes, the thermal zone device still
> work properly.
>
> Signed-off-by: Wei Ni
On Mon, Aug 25, 2014 at 02:29:48PM +0800, Wei Ni wrote:
> From: lightning314
>
> Add dt node to describe the thermal zone for the nct1008.
>
> Signed-off-by: Wei Ni
> ---
> arch/arm/boot/dts/tegra114-dalmore.dts | 20 +++-
> 1 file changed, 19 insertions(+), 1 deletion(-)
>
>
On Mon, Aug 25, 2014 at 02:29:48PM +0800, Wei Ni wrote:
> From: lightning314
>
> Add dt node to describe the thermal zone for the nct1008.
>
> Signed-off-by: Wei Ni
> ---
> arch/arm/boot/dts/tegra114-dalmore.dts | 20 +++-
> 1 file changed, 19 insertions(+), 1 deletion(-)
>
>
Hi,
On Tue, Aug 26, 2014 at 11:03:21AM +0800, Wei Ni wrote:
> On 08/25/2014 07:07 PM, Eduardo Valentin wrote:
> > Hello Wei Ni,
> >
> > On Mon, Aug 25, 2014 at 02:29:47PM +0800, Wei Ni wrote:
> >> Add more description for the "polling-delay" property
On Tue, Aug 26, 2014 at 10:17:29AM +0800, Wei Ni wrote:
> On 08/25/2014 07:07 PM, Eduardo Valentin wrote:
> > Hello Wei Ni,
> >
> > On Mon, Aug 25, 2014 at 02:29:47PM +0800, Wei Ni wrote:
> >> Add more description for the "polling-delay" property.
> &g
On Tue, Aug 26, 2014 at 10:27:43AM +0800, Wei Ni wrote:
> On 08/25/2014 08:23 PM, Mikko Perttunen wrote:
> > FWIW, please fix the authorship information for next version.
>
> Sorry, I didn't get your point, did you mean I should remove the line
> "From: lightning314 " ? Yes I made a mistake on it,
Hello Rui,
On Wed, Aug 27, 2014 at 03:41:53PM +0800, Zhang Rui wrote:
> On Fri, 2014-08-22 at 08:46 -0400, Eduardo Valentin wrote:
> > Hello Rui,
> >
> > Please pull these changes. Here we have three new features: IMX driver has
> > now support to i.mx6sx, the of-therm
Hello Wei,
On Wed, Aug 27, 2014 at 10:54:07AM +0800, Wei Ni wrote:
> On 08/26/2014 08:12 PM, Eduardo Valentin wrote:
> > On Tue, Aug 26, 2014 at 10:17:29AM +0800, Wei Ni wrote:
> >> On 08/25/2014 07:07 PM, Eduardo Valentin wrote:
> >>> Hello Wei Ni,
> >>>
Hello Wei,
On Thu, Aug 28, 2014 at 09:50:01AM +0800, Wei Ni wrote:
> On 08/27/2014 09:32 PM, Eduardo Valentin wrote:
> > Hello Wei,
> >
> > On Wed, Aug 27, 2014 at 10:54:07AM +0800, Wei Ni wrote:
> >> On 08/26/2014 08:12 PM, Eduardo Valentin wrote:
> >>>
Ceasar and Arnd,
On Thu, Aug 28, 2014 at 10:48:23AM +0200, Arnd Bergmann wrote:
> On Thursday 28 August 2014 08:59:19 Caesar Wang wrote:
> > Thermal is TS-ADC Controller module supports user-defined mode and
> > automatic mode.
> >
> > User-defined mode refers,TSADC all the control signals enti
On Fri, Mar 14, 2014 at 02:19:53PM -0700, Courtney Cavin wrote:
> On Thu, Feb 27, 2014 at 07:53:34AM +0100, Zhang Rui wrote:
> > On Wed, 2014-02-05 at 17:43 -0800, Courtney Cavin wrote:
> > > This driver is a generic method for using IIO ADC channels as thermal
> > > sensors.
> > >
> > > Signed-of
Amit,
On Thu, Jul 31, 2014 at 07:10:58PM +0200, Bartlomiej Zolnierkiewicz wrote:
> Hi,
>
> This patch series contains various cleanups for EXYNOS thermal
> driver. Overall it decreases driver's LOC by 9%. It is based
> on next-20140731 kernel. It should not cause any functionality
> changes.
>
-off-by: Chanwoo Choi
> > Acked-by: Kyungmin Park
> > Cc: Zhang Rui
> > Cc: Eduardo Valentin
> > Cc: Amit Daniel Kachhap
> > Reviewed-by: Amit Daniel Kachhap
> > ---
> > drivers/thermal/samsung/exynos_tmu.c | 4 +---
> > 1 file changed, 1 insert
Hello,
Sorry for the waaay too long answer. In any case, I believe it is worth
giving answers to why this patch was not merged.
On Wed, Feb 05, 2014 at 05:43:27PM -0800, Courtney Cavin wrote:
> This driver is a generic method for using IIO ADC channels as thermal
> sensors.
In fact, this seams
Hello
On Tue, Sep 09, 2014 at 01:35:31PM +0200, Heiko Stübner wrote:
> Am Dienstag, 9. September 2014, 10:27:17 schrieb Zhang Rui:
> > On Thu, 2014-09-04 at 09:02 +0800, Caesar Wang wrote:
> > > 在 2014年09月03日 16:07, Heiko Stübner 写道:
> > > > Am Mittwoch, 3. September 2014, 10:10:37 schrieb Caesar
Hello Caesar,
On Wed, Sep 10, 2014 at 12:39:07PM +0800, Caesar Wang wrote:
> Dear Eduardo,
>
> I'm sorry for it.
> I just received this message.Maybe my mailbox has a problem.
No problems. You can take your time.
>
> Thank you for your comments.
>
> 在 2014年08月3
Hello Wei,
On Wed, Sep 10, 2014 at 04:14:02PM +0800, Wei Ni wrote:
> >
> > Have you tried the patch set from Javi? the power allocator schem
> > includes a PID controller governor. Might be worth having a common
> > solution. I think it is easier if I merge his changes into a branch and
> >
have
given it a try on the boards I have available, which are mainline OMAP
and Exynos based.
Cheers,
Eduardo Valentin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.
From: Eduardo Valentin
This patch introduces a new thermal cooling device based on common clock
framework. The original motivation to write this cooling device is to be
able to cool down thermal zones using clocks that feed co-processors, such
as GPUs, DSPs, Image Processing Co-processors, etc
w much power to give each cooling device based
> on the performance they are requesting. The PID controller ensures
> that the total power budget does not exceed the control temperature.
>
> Cc: Zhang Rui
> Cc: Eduardo Valentin
> Signed-off-by: Punit Agrawal
> Signed-o
orrupt (line wrapped?)
#61: FILE: include/linux/thermal.h:159:
=20
ERROR: spaces required around that '=' (ctx:ExV)
#110: FILE: include/linux/thermal.h:218:
+=09struct mutex lock;
^
>
> Cc: Zhang Rui
> Cc: Eduardo Valentin
> Signed-off-by: Javi Merino
> ---
s how much power to give each cooling device based
> on the performance they are requesting. The PID controller ensures
> that the total power budget does not exceed the control temperature.
>
> Cc: Zhang Rui
> Cc: Eduardo Valentin
> Signed-off-by: Punit Agrawal
> Signed-o
Hello Bartlomiej,
On Wed, Aug 20, 2014 at 02:04:18PM +0200, Bartlomiej Zolnierkiewicz wrote:
> Currently these SoCs claim TRIM_RELOAD support but don't have
> triminfo_ctrl register address defined in their struct
> exynos_tmu_registers entries. This causes incorrect write of
> value "1" to data-
Hi,
On Wed, Sep 03, 2014 at 12:09:01PM +0900, Chanwoo Choi wrote:
> This patchset add the support of TRIMINFO_RELOAD feature for Exynos3250.
> But Exynos3250 has two TRIMINFO_CTRL register instead other Exynos has only
> one
> TRIMINFO_CTRL register. So, this patchset support the some Exynos SoC
ote:
> >>> On 08/28/2014 09:21 PM, Eduardo Valentin wrote:
> >>>> Hello Wei,
> >>>>
> >>>> On Thu, Aug 28, 2014 at 09:50:01AM +0800, Wei Ni wrote:
> >>>>> On 08/27/2014 09:32 PM, Eduardo Valentin wrote:
> >>>>>&g
ed_irq(&pdev->dev, data->irq, NULL,
> + rockchip_thermal_alarm_irq_thread,
> + IRQF_ONESHOT, "rockchip_thermal", data);
> + if (ret < 0) {
> + dev_err(&pdev->dev,
> +
On Mon, Sep 01, 2014 at 07:05:27AM -0400, edubez...@gmail.com wrote:
> Amit
>
> On Mon, Sep 1, 2014 at 6:53 AM, amit daniel kachhap
> wrote:
> > On Thu, Aug 28, 2014 at 8:19 PM, Eduardo Valentin
> > wrote:
> >> Amit,
> >>
> >> On Thu, Jul 31,
Chanwoo,
On Fri, Aug 29, 2014 at 08:50:41AM +0900, Chanwoo Choi wrote:
> Dear Eduardo,
>
> On 08/28/2014 11:53 PM, Eduardo Valentin wrote:
> > On Thu, Aug 28, 2014 at 01:25:14PM +0900, Chanwoo Choi wrote:
> >> Dear Eduardo,
> >>
> >> This patch is wrong.
Hello Anson,
On Wed, Aug 06, 2014 at 03:12:09PM +0800, Anson Huang wrote:
> i.MX6SX has some new features of thermal interrupt function,
> there are LOW, HIGH and PANIC irq for thermal sensor, so add
> platform data to separate different thermal version;
>
> The reset value of LOW ALARM is 0 whic
Hello Mikko,
It follows more small comments.
On Wed, Aug 6, 2014 at 6:25 AM, Mikko Perttunen wrote:
> This adds support for the Tegra SOCTHERM thermal sensing and management
> system found in the Tegra124 system-on-chip. This initial driver supports
> temperature polling for four thermal zones.
Hello Anson,
On Tue, Aug 05, 2014 at 05:12:43PM +0800, Anson Huang wrote:
> i.MX6SX has some new features of thermal interrupt function,
> there are LOW, HIGH and PANIC irq for thermal sensor, so add
> platform data to separate different thermal version;
>
> The reset value of LOW ALARM is 0 whic
Mikko,
On Tue, Aug 05, 2014 at 03:18:54PM +0300, Mikko Perttunen wrote:
> This adds binding documentation and headers for the Tegra124
> SOCTHERM device tree node.
>
> Signed-off-by: Mikko Perttunen
> ---
> .../devicetree/bindings/thermal/tegra-soctherm.txt | 35
> ++
> inc
Hello Mikko,
On Tue, Aug 05, 2014 at 03:18:57PM +0300, Mikko Perttunen wrote:
> This adds support for the Tegra SOCTHERM thermal sensing and management
> system found in the Tegra124 system-on-chip. This initial driver supports
> the four thermal zones with hardware-tracked trip points.
>
> Sign
Hello Yao,
On Tue, Oct 21, 2014 at 11:34:37AM +0800, Yao Dongdong wrote:
>
> On 2014/10/20 20:10, Eduardo Valentin wrote:
> > Hello Yao
> >
> > On Mon, Oct 20, 2014 at 04:27:59PM +0800, Yao Dongdong wrote:
> >> result is always zero when comes here.
> >>
APIs.
They are based on your next branch.
BR,
Eduardo Valentin
The following changes since commit 6ceaf58abe25e86292152005c51169796bad3407:
Merge branch 'int340x-thermal' of .git into next (2014-10-17 14:30:58 +0800)
are available in the git repository at:
git://git.kernel.o
Hello Rui,
On Thu, Oct 23, 2014 at 01:15:45PM +0800, Zhang Rui wrote:
> On 二, 2014-10-21 at 09:49 -0400, Eduardo Valentin wrote:
> > Hello Rui,
> >
> > Here are a couple of changes that are in my tree. They include:
> > - a few code refactoring improving the Exynos c
ELOAD);
. Documentation of RCAR thermal;
. Fix in the of-thermal, regarding the proper usage
of of-APIs; and
. Fixes on thermal-core, removal of unreachable code.
BR,
Eduardo Valentin
Bartlomiej Zolnierkiewicz (10):
th
ufreq_cooling_device for policy's cpu from cpufreq_dev_list
> when there is any policy update.
OK. Please give real examples of when the current code fails to catch
the event.
BR,
Eduardo Valentin
>
> Signed-off-by: Yadwinder Singh Brar
> ---
> drivers/thermal/cpu_coo
Hello Yadwinder,
On Thu, Nov 06, 2014 at 04:26:27PM +0530, Yadwinder Singh Brar wrote:
> Hello Eduardo Valentin,
>
> On Thursday, November 06, 2014 2:17 AM, Eduardo Valentin wrote:
> > Hello Yadwinder,
> >
> > On Wed, Nov 05, 2014 at 05:46:25PM +0530, Yadwinder Singh
Bartlomiej,
On Thu, Sep 18, 2014 at 05:27:17PM +0200, Bartlomiej Zolnierkiewicz wrote:
> Hi,
>
> This patch series replaces the hardware registers abstractions in
> the Exynos thermal driver by the usage of per-SoC type operations.
> Such solution provides simpler, easier to understand code and
>
Hello Rui,
On Thu, Oct 23, 2014 at 07:27:24AM -0400, Eduardo Valentin wrote:
> Hello Rui,
>
> On Thu, Oct 23, 2014 at 01:15:45PM +0800, Zhang Rui wrote:
> > On 二, 2014-10-21 at 09:49 -0400, Eduardo Valentin wrote:
> > > Hello Rui,
> > >
> > > Here a
Hello Ivan
On Wed, Oct 01, 2014 at 06:53:37PM +0300, Ivan T. Ivanov wrote:
> Add support for the temperature alarm peripheral found inside
> Qualcomm plug-and-play (QPNP) PMIC chips. The temperature alarm
> peripheral outputs a pulse on an interrupt line whenever the
> thermal over temperature st
rom travel, or you can
> help me send the pull request for 3.18-rc.
>
I would be glad to help you sending out what ever patches are pending.
Do you have 3.18-rc outstanding patches or are all of them in tree for
now?
> thanks,
> rui
All the best,
Eduardo Valentin
signature.asc
Description: Digital signature
On Tue, Oct 28, 2014 at 04:11:00PM +0800, Yao Dongdong wrote:
> Hello Eduardo,
>
> I'm sorry for that my patches listed below have format problems, so i resend
> them.
>
> https://lkml.org/lkml/2014/10/20/85
> https://lkml.org/lkml/2014/10/20/84
No issues. Thanks. I will apply them in my tree.
On Fri, Oct 03, 2014 at 10:26:28AM +0200, Lukasz Majewski wrote:
> Hi Eduardo,
>
> > Hello Lukasz,
> >
> > On Wed, Sep 24, 2014 at 10:27:11AM +0200, Lukasz Majewski wrote:
> > > Pointer to the uninitialized max_state variable is passed to get the
> > > maximal cooling state.
> > > For CPU cooling
Hello Pramod,
On Fri, Sep 19, 2014 at 11:35:09PM +0530, Pramod Gurav wrote:
> This change switches to managed resource APIs to allocated resources
> such as irq, clock. Hence does away with release statements of the
> same resorces in error lables and remove function.
>
> Cc: E
Hello Lukasz,
On Wed, Sep 24, 2014 at 10:27:10AM +0200, Lukasz Majewski wrote:
> It turns out that some boards can have instance->lower greater than 0 and
> when thermal trend is dropping it results with next_target equal to -1.
>
> Since the next_target is defined as unsigned long it is interpre
On Wed, Sep 24, 2014 at 10:27:12AM +0200, Lukasz Majewski wrote:
> Without this check it was possible to execute cooling device binding code
> even when wrong max cooling state was read.
>
> Signed-off-by: Lukasz Majewski
Acked-by: Eduardo Valentin
Rui, are you taking these pat
Hello Lukasz,
On Wed, Sep 24, 2014 at 10:27:11AM +0200, Lukasz Majewski wrote:
> Pointer to the uninitialized max_state variable is passed to get the
> maximal cooling state.
> For CPU cooling device (cpu_cooling.c) the cpufreq_get_max_state() is
> called, which even when error occurs will return
8,7 +38,7 @@
>
> /**
> * PID (Proportional-Integral-Derivative) controller is commonly used in
> - * linear control system, consider the the process.
> + * linear control system, consider the process.
On Tmon
On Thu, Oct 30, 2014 at 03:55:45PM +0200, Ivan T. Ivanov wrote:
>
> On Wed, 2014-10-29 at 17:32 +0200, Ivan T. Ivanov wrote:
> >
> +
> > > > + chip->tz_dev = thermal_zone_device_register(node->name,
> > > > TRIP_NUM, 0,
> > > > + chip, &qpnp_tz_ops, NULL,
> > >
> > > Have you consid
Hello Caesar,
On Sat, Nov 01, 2014 at 10:44:29AM +0800, Caesar Wang wrote:
> Thermal is TS-ADC Controller module supports
> user-defined mode and automatic mode.
>
> User-defined mode refers,TSADC all the control signals entirely by
> software writing to register for direct control.
>
> Automaic
Terve Mikko,
On Wed, Oct 15, 2014 at 01:05:19PM +0300, Mikko Perttunen wrote:
> Eduardo: ping
>
I had no objections with the driver at this point. Neither with the DT
part. I decided to include it in my -linus queue, which means, it will
hopefully be queued in the next major merge windown (v3.1
901 - 1000 of 1922 matches
Mail list logo
