[PATCH v4 0/6] Add hw overheat IRQ support to Marvell thermal driver

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

 



Hello,

This is the last batch of patches about the thermal driver that was
suspended, waiting for the ICU/SEI series to be merged. Now that
everything is ready in mainline, let's add hardware overheat interrupt
support to this driver.

Bindings and DT are updated accordingly. The interrupt will only be
triggered if the platform goes above ~102°C (threshold set to 100°C,
hysteresis to > 2°C). The interrupt property is of course not
mandatory.

In the mean time, I add myself to the MAINTAINERS file to receive and
review possible fixes/new features.

Thanks,
Miquèl

Changes since v3:
=================
* Added Rob's Reviewed-by on cp110 bindings.
* Addressed Rob comment on ap806 bindings by removing the reference to
  the interrupt parent.

Changes since v2:
=================
* Remove useless goto statement.
* Move the hysteresis array out of the function using it and declare
  the array statically instead that doing so on the stack.
* Remove unnecessary cast.
* On error, avoid returning a value not compatible with irqreturn_t in
  the IRQ handler (and when doing so, re-enable interrupts
  first). This is simply handled with a goto statement.
* 0 is not a valid interrupt, take this into account during variables
  validation.

Changes *not* addressed since v2:
================================
* Overheat interrupt configuration has certain limitations on which
  Marc asked questions. I don't know if/what I need to change so
  keeping it like this for this version.
* Eduardo pointed that once a critical level has been reached, there
  is no way back, the system will shutdown. I designed the handler to
  recover from that and to enable the IRQ again if needed, but maybe
  this is useless and should be removed. While waiting for his answer,
  I am sending another version and will correct in a v4 if needed.

Changes since v1:
=================
* Use a threaded IRQ handler to avoid a potential lock depency when
  notifying the core of an overheat situation.


Miquel Raynal (6):
  thermal: armada: add overheat interrupt support
  MAINTAINERS: thermal: add entry for Marvell MVEBU thermal driver
  dt-bindings: ap806: document the thermal interrupt capabilities
  dt-bindings: cp110: document the thermal interrupt capabilities
  arm64: dts: marvell: add interrupt support to ap806 thermal node
  arm64: dts: marvell: add interrupt support to cp110 thermal node

 .../arm/marvell/ap806-system-controller.txt   |   7 +
 .../arm/marvell/cp110-system-controller.txt   |   9 +
 MAINTAINERS                                   |   5 +
 arch/arm64/boot/dts/marvell/armada-ap806.dtsi |  18 +-
 arch/arm64/boot/dts/marvell/armada-cp110.dtsi |  15 +-
 drivers/thermal/armada_thermal.c              | 270 +++++++++++++++++-
 6 files changed, 317 insertions(+), 7 deletions(-)

-- 
2.19.1




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux