Messages in this thread | | | Date | Thu, 10 Mar 2016 12:26:24 +0530 | From | Laxman Dewangan <> | Subject | Re: [PATCH V2 5/6] thermal: max77620: Add thermal driver for reporting junction temp |
| |
On Thursday 10 March 2016 12:57 AM, Eduardo Valentin wrote: > * PGP Signed by an unknown key > > On Wed, Mar 09, 2016 at 06:40:09PM +0530, Laxman Dewangan wrote: >> Maxim Semiconductor Max77620 supports alarm interrupts when >> its die temperature crosses 120C and 140C. These threshold >> temperatures are not configurable. >> >> Add thermal driver to register PMIC die temperature as thermal >> zone sensor and capture the die temperature warning interrupts >> to notifying the client. >> >> Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com> >> >> --- >> Changes from V1: >> - checkpatch warning fix and simplifying the dev.of_node >> initialisation. >> >> drivers/thermal/Kconfig | 10 +++ >> drivers/thermal/Makefile | 1 + >> drivers/thermal/thermal-max77620.c | 151 +++++++++++++++++++++++++++++++++++++ >> 3 files changed, 162 insertions(+) >> create mode 100644 drivers/thermal/thermal-max77620.c >> >> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig >> index 5e7c97a..fc856eb 100644 >> --- a/drivers/thermal/Kconfig >> +++ b/drivers/thermal/Kconfig >> @@ -194,6 +194,16 @@ config IMX_THERMAL >> cpufreq is used as the cooling device to throttle CPUs when the >> passive trip is crossed. >> >> +config MAX77620_THERMAL >> + tristate "Temperature sensor driver for Maxim MAX77620 PMIC" >> + depends on MFD_MAX77620 >> + depends on OF > The previous question on compile test was more if we could add depends > on COMPILE_TEST flag. Sorry if I was not clear enough. > > > I am adding the flag here: > > > config MAX77620_THERMAL > tristate "Temperature sensor driver for Maxim MAX77620 PMIC" > - depends on MFD_MAX77620 > + depends on MFD_MAX77620 || COMPILE_TEST > + depends on HAS_IOMEM > depends on OF > help > Support for die junction temperature > warning alarm for Maxim > > > but I still cannot compile test the file because of the missing header. > drivers/thermal/thermal-max77620.c:16:32: fatal error: > linux/mfd/max77620.h: No such file or directory > #include <linux/mfd/max77620.h> > > > I wont be able to merge this as it is. We need to sort out the > dependency first. > I will update you once the mfd patches will merge as this header is on that patch.
| |