Add documentation of the function for reading alert thresholds
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/omap-bandgap.c | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/drivers/staging/omap-thermal/omap-bandgap.c
b/drivers/staging/omap
Add documentation for the function to setup TSHUT handling
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/omap-bandgap.c | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/drivers/staging/omap-thermal/omap-bandgap.c
b/drivers/staging/omap
Document function that sets talert handling up.
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/omap-bandgap.c | 12 +++-
1 files changed, 11 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/omap-thermal/omap-bandgap.c
b/drivers/staging/omap-thermal/omap
Document function to build omap_bandgap structure
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/omap-bandgap.c |9 +
1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/drivers/staging/omap-thermal/omap-bandgap.c
b/drivers/staging/omap-thermal/omap
is
required to device the data structures that maps the registers
and their bitfields. The DT compatible list must also be updated.
CC: Santosh Shilimkar
CC: Vaibhav Bedia
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/Kconfig |2 +-
drivers/staging/omap-thermal/TODO
Add myself as maintainer of the TI bandgap and thermal driver.
CC: Santosh Shilimkar
CC: Zhang Rui
Signed-off-by: Eduardo Valentin
---
MAINTAINERS |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 6672165..199bb4b 100644
--- a
there are registers shared among the sensors, this lock
is global, not per sensor.
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/TODO |1 -
drivers/staging/omap-thermal/omap-bandgap.c | 18 ++
drivers/staging/omap-thermal/omap-bandgap.h |4
It is hard to use regmap because benefit of using regmap cache
may not be applicable as there is a specific sequence to
restore the bandgap context.
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/TODO |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a
Not all APIs exposed today are used. However all unused APIs
will be required once the thermal layer allows IRQ based
policies.
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/TODO |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/omap
Document the helper to validate a struct omap_bandgap and a sensor id.
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/omap-bandgap.c |8
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/drivers/staging/omap-thermal/omap-bandgap.c
b/drivers/staging
Hello Dan,
On 15-03-2013 17:09, Dan Carpenter wrote:
On Fri, Mar 15, 2013 at 08:59:55AM -0400, Eduardo Valentin wrote:
This patch introduce a macro to read, update, write bitfields.
It will be specific to bandgap data structures.
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap
Hey Dan,
On 15-03-2013 17:22, Dan Carpenter wrote:
On Fri, Mar 15, 2013 at 08:59:57AM -0400, Eduardo Valentin wrote:
Change the way the omap_bandgap_power is written so that it has only
one exit entry (Documentation/CodingStyle).
It's only if there is an unlock or something that you s
On 16-03-2013 04:59, Dan Carpenter wrote:
On Fri, Mar 15, 2013 at 09:00:35AM -0400, Eduardo Valentin wrote:
Because there is a need to lock inside IRQ handler, this patch
changes the locking mechanism inside the omap-bandgap.[c,h] to
spinlocks. Now this lock is used to protect omap_bandgap
Hello Dan,
On 16-03-2013 05:05, Dan Carpenter wrote:
I've reviewed this set.
I hate to make people redo whole patchset sets, and I hate
re-reviewing code. Obviously, I don't really like the bunny hop
patches and I'm trying to discourage that going forward. ;P But
I wouldn't say it's a "Redo
On 16-03-2013 04:39, Dan Carpenter wrote:
On Fri, Mar 15, 2013 at 09:00:21AM -0400, Eduardo Valentin wrote:
if (ret) {
dev_err(bg_ptr->dev, "failed to read thot\n");
- return -EIO;
+ ret = -EIO;
+
From: J Keerthy
Removes checkpatch warnings on omap-bandgap.c.
Signed-off-by: J Keerthy
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/omap-bandgap.c | 15 ---
1 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/omap-thermal/omap
/omap/omap-thermal-fixes
All best,
Eduardo Valentin (3):
staging: omap-thermal: remove checkpatch.pl warnings on data files
staging: omap-thermal: fix polling period settings
staging: omap-thermal: improve conf data handling and initialization
J Keerthy (1):
staging: omap-thermal: Correct
.
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/omap-bandgap.c|6 ++--
drivers/staging/omap-thermal/omap-thermal-common.c | 38 +---
2 files changed, 36 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/omap-thermal/omap-bandgap.c
b/drivers
While registering the omap thermal zones we need to
properly specify TC1 and TC2, as long as the proper
passive polling period and monitor period.
This patch fixes the parameters passed while registering
the thermal zone.
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/omap
Simple checkpatch.pl clean ups.
Signed-off-by: Eduardo Valentin
---
drivers/staging/omap-thermal/omap4-thermal.c | 54 +-
drivers/staging/omap-thermal/omap5-thermal.c | 38 +-
2 files changed, 46 insertions(+), 46 deletions(-)
diff --git a/drivers
On 23-08-2013 19:39, Guenter Roeck wrote:
> On Fri, Aug 23, 2013 at 07:15:44PM -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.
>>
>> Th
On 26-08-2013 00:42, Viresh Kumar wrote:
> On 24 August 2013 04:45, Eduardo Valentin wrote:
>> diff --git a/drivers/cpufreq/cpufreq-cpu0.c b/drivers/cpufreq/cpufreq-cpu0.c
>> index ad1fde2..ede6487 100644
>> --- a/drivers/cpufreq/cpufreq-cpu0.c
>> +++ b/drivers/cpufreq
On 23-08-2013 19:50, Guenter Roeck wrote:
> On Fri, Aug 23, 2013 at 07:15:44PM -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.
>>
>> Th
Hey Mark,
On 27-08-2013 05:29, Mark Rutland wrote:
> On Sat, Aug 24, 2013 at 12:15:42AM +0100, 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,
>> it can be flagged at
t; On Sat, Aug 24, 2013 at 12:15:43AM +0100, Eduardo Valentin wrote:
>> In order to be able to build thermal policies
>> based on generic sensors, like I2C device, that
>> can be places in different points on different boards,
>> there is a need to have a way to feed board d
On 27-08-2013 12:23, Mark Rutland wrote:
> On Tue, Aug 27, 2013 at 02:44:40PM +0100, Eduardo Valentin wrote:
>> Hello Mark,
>>
>>
>> First of all, thanks for taking the time to review in such level of
>> detail. Lets try to align and have a common understanding. An
On 23-08-2013 19:08, Rafael J. Wysocki wrote:
> On Friday, August 23, 2013 06:03:14 PM Eduardo Valentin wrote:
>> When registering a new thermal_device, the thermal framework
>> will always add a hwmon sysfs interface.
>>
>> This patch adds a flag to make this beh
On 27-08-2013 20:13, Zhang Rui wrote:
> On 二, 2013-08-27 at 23:17 +0200, Rafael J. Wysocki wrote:
>> On Tuesday, August 27, 2013 02:26:41 PM Eduardo Valentin wrote:
>>> On 23-08-2013 19:08, Rafael J. Wysocki wrote:
>>>> On Friday, August 23, 2013 06:03:14 PM Edua
been runtime tested on DRA7-evm.
All best,
[1] - https://lkml.org/lkml/2013/9/26/787
[2] - git://github.com/rrnayak/linux.git out-of-tree/dra-integrated-v3
Eduardo Valentin (7):
arm: dts: add dra7 DSPEVE thermal data
arm: dts: add dra7 IVA thermal data
arm: dts: dra7: add bandgap entry
arm
This patch changes a dtsi file to contain the thermal data
for IVA domain on DRA7 and later SoCs. This data will
enable a thermal shutdown at 125C.
This thermal data can be reused across TI SoC devices.
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/dra7-iva-thermal.dtsi | 28
This patch changes a dtsi file to contain the thermal data
for DSPEVE domain on DRA7 and later SoCs. This data will
enable a thermal shutdown at 125C.
This thermal data can be reused across TI SoC devices.
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/dra7-dspeve-thermal.dtsi | 28
DRA7 devices can reach high temperatures and thus
needs to have cpufreq-cooling on systems running on it.
This patch adds the required cooling device properties
so that cpufreq-cpu0 driver loads the cooling device.
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/dra7.dtsi | 6 +-
1
This patch adds bandgap IP entry on DRA7 dtsi device tree file.
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 3a746c2..4e9b159 100644
--- a/arch
This patch adds thermal data for DRA7 SoCs. Cpufreq cooling
will be enabled for MPU domain whenever it crosses 100C.
GPU, CORE, DSPEVE, IVA will be exposed and system shall
shutdown when they cross 125C.
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/dra7.dtsi | 9 +
1 file
On dra7-evm there is an tmp102 temperature sensor on i2c bus 1.
This patch adds its device tree node.
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/dra7-evm.dts | 8
1 file changed, 8 insertions(+)
diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
Add simple thermal zone on tmp102.
Signed-off-by: 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 3b6c16a..70e1a12 100644
--- a/arch/arm/boot/dts/dra7
On 01-10-2013 18:33, Nishanth Menon wrote:
> On 14:32-20131001, Eduardo Valentin wrote:
> minor comments follow
>> This patch changes a dtsi file to contain the thermal data
> s/changes/introduces?
>> for IVA domain on DRA7 and later SoCs. This data will
>> enable
On 01-10-2013 18:46, Nishanth Menon wrote:
> On 14:32-20131001, Eduardo Valentin wrote:
>> This patch adds bandgap IP entry on DRA7 dtsi device tree file.
>>
>> Signed-off-by: Eduardo Valentin
>> ---
>> arch/arm/boot/dts/dra7.dtsi | 12
>> 1 fi
On 01-10-2013 18:37, Nishanth Menon wrote:
> On 14:32-20131001, Eduardo Valentin wrote:
>> This patch changes a dtsi file to contain the thermal data
>^^ introduces?
>> for DSPEVE domain on DRA7 and later SoCs. This data will
>> enable a thermal shutdown at
On 01-10-2013 18:51, Nishanth Menon wrote:
> On 14:32-20131001, Eduardo Valentin wrote:
>> On dra7-evm there is an tmp102 temperature sensor on i2c bus 1.
>> This patch adds its device tree node.
>>
>> Signed-off-by: Eduardo Valentin
>> ---
>> arch/arm/b
On 01-10-2013 18:58, Nishanth Menon wrote:
> On 14:32-20131001, Eduardo Valentin wrote:
>> Hello all,
>>
>> This is a complementary patch series with themal DT support for DRA7.
>>
>> Although this work depends on the thermal dt parser work [1], I decided
&g
o, my question before acking is, shall we remove this check altogether?
> return;
>
> if (th_zone->bind == false) {
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
FIG_SOC_EXYNOS5250) || defined(CONFIG_SOC_EXYNOS4412))
> +#if defined(CONFIG_SOC_EXYNOS4412)
> +extern struct exynos_tmu_init_data const exynos4412_default_tmu_data;
> +#define EXYNOS4412_TMU_DRV_DATA (&exynos4412_default_tmu_data)
> +#else
> +#define EXYNOS4412_TMU_DRV_DATA (NULL)
> +#endif
> +
> +#if defined(CONFIG_SOC_EXYNOS5250)
> extern struct exynos_tmu_init_data const exynos5250_default_tmu_data;
> #define EXYNOS5250_TMU_DRV_DATA (&exynos5250_default_tmu_data)
> #else
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
_tmu_data.h
> @@ -95,6 +95,10 @@
>
> #define EXYNOS_MAX_TRIGGER_PER_REG 4
>
> +/* Exynos4412 specific */
> +#define EXYNOS4412_MUX_ADDR_VALUE 6
> +#define EXYNOS4412_MUX_ADDR_SHIFT 20
> +
> /*exynos5440 specific registers*/
> #define EXYNOS
183,7 +183,7 @@ struct exynos_tmu_init_data const
> exynos4412_default_tmu_data = {
> struct exynos_tmu_init_data const exynos5250_default_tmu_data = {
> .tmu_data = {
> { EXYNOS4412_TMU_DATA,
> - .type = SOC_ARCH_EXYNOS,
> + .ty
rmal/samsung/exynos_tmu_data.c index bd08093..188223f
>>>> 100644 --- a/drivers/thermal/samsung/exynos_tmu_data.c
>>>> +++ b/drivers/thermal/samsung/exynos_tmu_data.c
>>>> @@ -183,7 +183,7 @@ struct exynos_tmu_init_data const
>>>> exynos4412_default_tmu_data = { struct exynos_tmu_init_data const
>>>> exynos5250_default_tmu_data = { .tmu_data = {
>>>>{ EXYNOS4412_TMU_DATA,
>>>> - .type = SOC_ARCH_EXYNOS,
>>>> + .type = SOC_ARCH_EXYNOS5250,
>>>>},
>>>>},
>>>>.tmu_count = 1,
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pm" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
return -EPROBE_DEFER;
> }
> +#endif
I would prefer not to have this ifdef here.
>
> /* Register cooling device */
> data->cool_dev = cpufreq_cooling_register(cpu_present_mask);
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
to depend on an arch, but the arch code is expected to select
ARCH_HAS_BANDGAP.
> source "drivers/thermal/ti-soc-thermal/Kconfig"
> endmenu
>
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
nt id)
> -{
> - return 0;
> -}
> -
> -static inline
> -int ti_thermal_register_cpu_cooling(struct ti_bandgap *bgp, int id)
> -{
> - return 0;
> -}
> -
> -static inline
> -int ti_thermal_unregister_cpu_cooling(struct ti_bandgap *bgp, int id)
> -{
> - return 0;
> -}
> -#endif
> #endif
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
On 04-10-2013 14:22, Eduardo Valentin wrote:
> On 04-10-2013 08:35, Bartlomiej Zolnierkiewicz wrote:
>> Menu for Texas Instruments thermal support is visible on all
>> platforms and TI_SOC_THERMAL + TI_THERMAL config options can
>> be selected also on EXYNOS platform (on w
/devicetree/bindings/thermal/exynos-thermal.txt
> delete mode 100644 drivers/thermal/exynos_thermal.c
> create mode 100644 drivers/thermal/samsung/Kconfig
> create mode 100644 drivers/thermal/samsung/Makefile
> create mode 100644 drivers/thermal/samsung/exynos_thermal_common.c
> create mode 100644 drivers/thermal/samsung/exynos_thermal_common.h
> create mode 100644 drivers/thermal/samsung/exynos_tmu.c
> create mode 100644 drivers/thermal/samsung/exynos_tmu.h
> create mode 100644 drivers/thermal/samsung/exynos_tmu_data.c
> create mode 100644 drivers/thermal/samsung/exynos_tmu_data.h
> delete mode 100644 include/linux/platform_data/exynos_thermal.h
>
>
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
.../devicetree/bindings/thermal/thermal.txt| 92 +
drivers/thermal/Kconfig| 13 +
drivers/thermal/Makefile | 1 +
drivers/thermal/thermal_dt.c
thermal locking.
Cc: Zhang Rui
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/thermal/Kconfig | 9 ++
drivers/thermal/Makefile| 3 +
drivers/thermal/thermal_core.c | 255 +-
drivers
On 09-07-2013 12:04, R, Durgadoss wrote:
> Hi Eduardo,
>
>> -Original Message-
>> From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm-
>> ow...@vger.kernel.org] On Behalf Of Eduardo Valentin
>> Sent: Tuesday, July 09, 2013 7:30 PM
>> To: linux...@vge
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/thermal/ti-soc-thermal/ti-thermal-common.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/thermal/ti-soc-thermal/ti-thermal-common.c
b/drivers/thermal/ti-soc-thermal/ti-thermal
and is composed by the cpufreq cooling device.
Cc: "Rafael J. Wysocki"
Cc: Viresh Kumar
Cc: Grant Likely
Cc: Rob Herring
Cc: cpuf...@vger.kernel.org
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: devicetree-disc...@lists.ozlabs.org
Signed-off-by: Eduard
ay to upstream.
All best,
Eduardo Valentin (16):
drivers: thermal: allow registering without .get_temp
drivers: thermal: introduce device tree parser
drivers: thermal: cpu_cooling: introduce of_cpufreq_cooling_register
cpufreq: cpufreq-cpu0: add dt node parsing for cooling device
prope
.
Cc: Zhang Rui
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/thermal/Kconfig | 1 +
drivers/thermal/cpu_cooling.c | 31 +++
include/linux/cpu_cooling.h | 25 +
3 files changed
file mode 100644
index 000..dd69788
--- /dev/null
+++ b/arch/arm/boot/dts/omap4-cpu-thermal.dtsi
@@ -0,0 +1,41 @@
+/*
+ * Device Tree Source for OMAP4/5 SoC CPU thermal
+ *
+ * Copyright (C) 2013 Texas Instruments Incorporated - http://www.ti.com/
+ * Contact: Eduardo Valentin
+ *
+ * This file i
nel.org
Cc: devicetree-disc...@lists.ozlabs.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/omap443x.dtsi | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/omap443x.d
behavior
will be the same.
Cc: Jean Delvare
Cc: Guenter Roeck
Cc: lm-sens...@lm-sensors.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/hwmon/lm75.c | 46 +-
1 file changed, 41 insertions(+), 5 deletions(-)
diff --git a
behavior
will be the same.
Cc: Jean Delvare
Cc: Guenter Roeck
Cc: lm-sens...@lm-sensors.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/hwmon/tmp102.c | 28
1 file changed, 28 insertions(+)
diff --git a/drivers/hwmon/tmp102.c b/drivers
rk Rutland
Cc: Stephen Warren
Cc: Ian Campbell
Cc: Russell King
Cc: linux-o...@vger.kernel.org
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/omap4460.dtsi | 10 --
1 file
c: linux-o...@vger.kernel.org
Cc: devicetree-disc...@lists.ozlabs.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
arch/arm/boot/dts/omap443x.dtsi | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm/boot/dts/omap443x.dtsi
l Moll
Cc: Mark Rutland
Cc: Stephen Warren
Cc: Ian Campbell
Cc: Russell King
Cc: linux-o...@vger.kernel.org
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/omap4460.dtsi | 5 +++
rk Rutland
Cc: Stephen Warren
Cc: Ian Campbell
Cc: Russell King
Cc: linux-o...@vger.kernel.org
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/omap5-core-thermal
n
Cc: Rob Herring
Cc: Pawel Moll
Cc: Mark Rutland
Cc: Stephen Warren
Cc: Ian Campbell
Cc: Russell King
Cc: linux-o...@vger.kernel.org
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/omap5
l Moll
Cc: Mark Rutland
Cc: Stephen Warren
Cc: Ian Campbell
Cc: Russell King
Cc: linux-o...@vger.kernel.org
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/omap5.dtsi | 3 +++
1 file
rk Rutland
Cc: Stephen Warren
Cc: Ian Campbell
Cc: Russell King
Cc: linux-o...@vger.kernel.org
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/omap5-gpu-thermal
thermal zone,
it was required to allow changing the thermal zone
.get_temp callback. For this reason, this patch
also removes the 'const' modifier from the .ops
field of thermal zone devices.
Cc: Zhang Rui
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduard
interface
will be returning -EINVAL on any temperature read operation.
Cc: Zhang Rui
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/thermal/thermal_core.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a
device. Loading the thermal data from
DT, the driver assumes someone else will add the cpufreq
cooling device, like the cpufreq driver.
Cc: Zhang Rui
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/thermal/ti-soc-thermal/ti-thermal
On 15-09-2013 19:33, Guenter Roeck wrote:
> On 09/15/2013 03:02 PM, Eduardo Valentin wrote:
>> This patch adds to tmp102 temperature sensor the possibility
>> to expose itself as thermal zone device, registered on the
>> thermal framework.
>>
>> The thermal zone is
behavior
will be the same.
Cc: Jean Delvare
Cc: Guenter Roeck
Cc: lm-sens...@lm-sensors.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/hwmon/tmp102.c | 20
1 file changed, 20 insertions(+)
---
Guenter,
Here is a version without the warnings
behavior
will be the same.
Cc: Jean Delvare
Cc: Guenter Roeck
Cc: lm-sens...@lm-sensors.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/hwmon/lm75.c | 36 +++-
1 file changed, 31 insertions(+), 5 deletions(-)
---
Guenter,
Here is a
On 18-09-2013 07:06, Guenter Roeck wrote:
> On Tue, Sep 17, 2013 at 06:34:05PM -0400, Eduardo Valentin wrote:
>> This patch adds to tmp102 temperature sensor the possibility
>> to expose itself as thermal zone device, registered on the
>> thermal framework.
>>
>>
behavior
will be the same.
Cc: Jean Delvare
Cc: Guenter Roeck
Cc: lm-sens...@lm-sensors.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/hwmon/tmp102.c | 19 +++
1 file changed, 19 insertions(+)
---
Hi Guenter,
Very minor change, removing comment
behavior
will be the same.
Cc: Jean Delvare
Cc: Guenter Roeck
Cc: lm-sens...@lm-sensors.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin
---
drivers/hwmon/lm75.c | 35 ++-
1 file changed, 30 insertions(+), 5 deletions(-)
---
Hi Guenter,
Very minor
thermal zone,
it was required to allow changing the thermal zone
.get_temp callback. For this reason, this patch
also removes the 'const' modifier from the .ops
field of thermal zone devices.
Cc: Zhang Rui
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduard
On 18-09-2013 13:08, Guenter Roeck wrote:
> On Wed, Sep 18, 2013 at 12:03:09PM -0400, Eduardo Valentin wrote:
>> This patch introduces a device tree bindings for
>> describing the hardware thermal behavior and limits.
>> Also a parser to read and interpret the data and feed
thermal zone,
it was required to allow changing the thermal zone
.get_temp callback. For this reason, this patch
also removes the 'const' modifier from the .ops
field of thermal zone devices.
Cc: Zhang Rui
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduard
Hello Joe,
Thanks for reviewing this code. Couple of replies.
18-09-2013 15:11, Joe Perches wrote:
> On Wed, 2013-09-18 at 15:02 -0400, Eduardo Valentin wrote:
>> This patch introduces a device tree bindings for
>> describing the hardware thermal behavior and limits.
>>
NULL, 0, sensor->ops->register_irq ? 0 : 1);
> +
> + if (IS_ERR(sensor->th_dev)) {
> + dev_err(dev, "unable to register thermal zone device\n");
> + ret = PTR_ERR(sensor->th_dev);
> + goto turn_off_exit;
> + }
> +
> + if (sensor->ops->register_irq) {
> + ret = sensor->ops->register_irq(sensor);
> + if (ret) {
> + dev_err(dev, "unable to register irq\n");
> + goto unregister_thzone;
> + }
> + }
> +
> + platform_set_drvdata(pdev, sensor);
> + dev_info(dev, start_banner);
> + return 0;
> +
> +unregister_thzone:
> + thermal_zone_device_unregister(sensor->th_dev);
> +turn_off_exit:
> + st_thermal_sensor_off(sensor);
> +out:
> + return ret;
> +}
> +
> +static int st_thermal_remove(struct platform_device *pdev)
> +{
> + struct st_thermal_sensor *sensor = platform_get_drvdata(pdev);
> +
> + st_thermal_sensor_off(sensor);
> + thermal_zone_device_unregister(sensor->th_dev);
> + return 0;
> +}
> +
> +#ifdef CONFIG_PM
> +static int st_thermal_suspend(struct device *dev)
> +{
> + struct platform_device *pdev = to_platform_device(dev);
> + struct st_thermal_sensor *sensor = platform_get_drvdata(pdev);
> +
> + return st_thermal_sensor_off(sensor);
> +}
> +
> +static int st_thermal_resume(struct device *dev)
> +{
> + int ret;
> + struct platform_device *pdev = to_platform_device(dev);
> + struct st_thermal_sensor *sensor = platform_get_drvdata(pdev);
> +
> + ret = st_thermal_sensor_on(sensor);
> + if (ret)
> + return ret;
> +
> + if (sensor->ops->enable_irq) {
> + ret = sensor->ops->enable_irq(sensor);
> + if (ret)
> + return ret;
> + }
> + return 0;
> +}
> +#endif
> +
> +static SIMPLE_DEV_PM_OPS(st_thermal_pm_ops, st_thermal_suspend,
> + st_thermal_resume);
> +
> +MODULE_DEVICE_TABLE(of, st_thermal_of_match);
> +static struct platform_driver st_thermal_driver = {
> + .driver = {
> + .name = DRIVER_NAME,
> + .owner = THIS_MODULE,
> + .of_match_table = st_thermal_of_match,
> + .pm = &st_thermal_pm_ops,
> + },
> + .probe = st_thermal_probe,
> + .remove = st_thermal_remove,
> +};
> +
> +module_platform_driver(st_thermal_driver);
> +MODULE_AUTHOR("STMicroelectronics (R&D) Limited");
> +MODULE_DESCRIPTION("STMicroelectronics STIH41x SOC thermal sensor driver");
> +MODULE_LICENSE("GPL");
GPL v2?
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
On 18-09-2013 15:59, Joe Perches wrote:
> On Wed, 2013-09-18 at 15:44 -0400, Eduardo Valentin wrote:
>> Hello Joe,
>
> Hello Eduardo.
>
>> 18-09-2013 15:11, Joe Perches wrote:
>>> On Wed, 2013-09-18 at 15:02 -0400, Eduardo Valentin wrote:
>>>> Thi
thermal zone,
it was required to allow changing the thermal zone
.get_temp callback. For this reason, this patch
also removes the 'const' modifier from the .ops
field of thermal zone devices.
Cc: Zhang Rui
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduard
On 18-09-2013 16:44, Joe Perches wrote:
> On Wed, 2013-09-18 at 16:31 -0400, Eduardo Valentin wrote:
>
>> +/**
>> + * of_parse_thermal_zones - parse device tree thermal data
>> + *
>> + * Initialization function that can be called by machine initialization
>>
thermal zone,
it was required to allow changing the thermal zone
.get_temp callback. For this reason, this patch
also removes the 'const' modifier from the .ops
field of thermal zone devices.
Cc: Zhang Rui
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduard
On 18-09-2013 17:00, Joe Perches wrote:
> On Wed, 2013-09-18 at 16:52 -0400, Eduardo Valentin wrote:
>> On 18-09-2013 16:44, Joe Perches wrote:
>>> On Wed, 2013-09-18 at 16:31 -0400, Eduardo Valentin wrote:
>>>
>>>> +/**
>>>> + * of_p
On 18-09-2013 17:04, Joe Perches wrote:
> On Wed, 2013-09-18 at 16:57 -0400, Eduardo Valentin wrote:
>> This patch introduces a device tree bindings for
>> describing the hardware thermal behavior and limits.
>> Also a parser to read and interpret the data and feed
>>
thermal zone,
it was required to allow changing the thermal zone
.get_temp callback. For this reason, this patch
also removes the 'const' modifier from the .ops
field of thermal zone devices.
Cc: Zhang Rui
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduard
Hi Mark, Stephen and Pawel,
On 03-09-2013 09:15, Mark Rutland wrote:
> On Fri, Aug 30, 2013 at 12:19:43AM +0100, Eduardo Valentin wrote:
> I think that the above can describe that, but I'd like to see a binding
> document so we can consider it in more detail.
Find below anothe
On 08-09-2013 17:58, Regid Ichira wrote:
> Applied to the HEAD of linux.git,
>VERSION = 3 PATCHLEVEL = 11 SUBLEVEL = 0
> Signed-off-by: Regid Ichira
Acked-by: Eduardo Valentin
> ---
> drivers/thermal/Kconfig | 2 +-
> 1 file changed, 1 insert
hey just describe thermal shutdown by software, thermal
zones with only CRITICAL trip points, for instance. For this reason,
thermal zones must be probed properly without cooling-maps. I need to
double check the documentation in this case, I believe.
>
> Thanks,
> -Wendy
>
On 07-10-2013 10:52, Eduardo Valentin wrote:
> On 04-10-2013 18:17, Wendy Ng wrote:
>> Hello Eduardo,
>>
>
> Hello Wendy,
>
>> I have rebased my work on the top of yours and tried out the new way of
>> registering the thermal zone.The new method is certain
On 07-10-2013 13:52, Wendy Ng wrote:
>
> On 10/7/2013 9:12 AM, Eduardo Valentin wrote:
>> On 07-10-2013 10:52, Eduardo Valentin wrote:
>>> On 04-10-2013 18:17, Wendy Ng wrote:
>>>> Hello Eduardo,
>>>>
>>> Hello Wendy,
>>>
>>&g
On 08-10-2013 04:45, Hongbo Zhang wrote:
> On 09/25/2013 10:15 PM, Eduardo Valentin wrote:
>> On 25-09-2013 03:13, Hongbo Zhang wrote:
>>> On 09/19/2013 05:35 AM, Eduardo Valentin wrote:
>>>>[...]
>>>> +
>>>> +/*** sensor API ***
2013 11:57:16 AM Bartlomiej Zolnierkiewicz wrote:
>>>>
>>>> Hi,
>>>>
>>>> On Friday, October 04, 2013 02:22:30 PM Eduardo Valentin wrote:
>>>>> On 04-10-2013 08:35, Bartlomiej Zolnierkiewicz wrote:
>>>>>> Menu for Texas I
ged, 1831 insertions(+), 1 deletion(-)
> create mode 100644 Documentation/ABI/testing/sysfs-class-thermal
> create mode 100644 Documentation/thermal/sysfs-api2.txt
> create mode 100644 drivers/thermal/thermal_test.c
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
signature.asc
Description: OpenPGP digital signature
-off-by: Eduardo Valentin
---
Documentation/thermal/sysfs-api.txt | 5 +
drivers/thermal/thermal_core.c | 8 +---
include/linux/thermal.h | 8
3 files changed, 18 insertions(+), 3 deletions(-)
---
Hello all,
So, here is then a different way of implementing same
Mark, Pawel and Stephen,
On 27-08-2013 14:17, Eduardo Valentin wrote:
> On 27-08-2013 12:23, Mark Rutland wrote:
>> On Tue, Aug 27, 2013 at 02:44:40PM +0100, Eduardo Valentin wrote:
>>> Hello Mark,
I believe now we need to align on thermal bindings, before I propose
ne
101 - 200 of 1922 matches
Mail list logo
