Re: [PATCH 3/3] thermal: max77620: Add thermal driver for reporting junction temp

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Wednesday 09 March 2016 02:54 AM, Eduardo Valentin wrote:


On Fri, Mar 04, 2016 at 07:10:10PM +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.
Are any of these critical?

Datasheet says that two alarm interrupt at 120 and 140degC. 165 degC is shutdown temp on which PMIC get shutdown.
So just says as the warning interrupt.



Signed-off-by: Laxman Dewangan <ldewangan@xxxxxxxxxx>
---
  drivers/thermal/Kconfig            |   9 +++
  drivers/thermal/Makefile           |   1 +
  drivers/thermal/thermal-max77620.c | 151 +++++++++++++++++++++++++++++++++++++
Given that it is a DT based driver, please add also a binding entry
under Documentation/devicetree/bindings/thermal/

There is no new DT property and so did not added. But will add the doc saying the mandatory properties from thermal framework i.e. #thermal-sensor-cells


+	depends on MFD_MAX77620
Can this be COMPILE_TEST'ed?

Yes, I compile and tested .

+	depends on OF
+	help
+	  Support for die junction temperature warning alarm for Maxim
+	  Semiconductor PMIC MAX77620 device. Device generates alert
+	  signal/interrupt when die temperature cross its threshold.
+
Somehow checkpatch.pl --strict is complaining about this entry. Can you
please check?

The help should be minimum 4 lines otherwise warning is generated. I made it for next patch.

Second warning is:
/**
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#57:
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 174 lines checked

0001-thermal-max77620-Add-thermal-driver-for-reporting-ju.patch has style problems, please review.
**/

This is because new file get added and I think we can ignore it.

+	if (val & MAX77620_IRQ_TJALRM2_MASK)
+		*temp = MAX77620_TJALARM2_TEMP;
+	else if (val & MAX77620_IRQ_TJALRM1_MASK)
+		*temp = MAX77620_TJALARM1_TEMP;
+	else
+		*temp = MAX77620_NORMAL_OPERATING_TEMP;
So, no way at all to get a temp?

yaah, there is no way other than getting the bit for whether temp crossed threshold or not.


+
+	if (!pdev->dev.of_node)
+		pdev->dev.of_node = pdev->dev.parent->of_node;
Why is this needed?

This driver is sub mfd devices and it is registered without device node pointer. The DT binding doc for the mfdmax77620 is flat, does not have thermal sub node.

hence to get the of_node for sensor, I am making the device node as node pointer for thermal sensor.
I can overwrite as
pdev->dev.of_node = pdev->dev.parent->of_node also without check for simplification.



--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux