All of lore.kernel.org
 help / color / mirror / Atom feed
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Gregory Clement <gregory.clement@bootlin.com>,
	Jason Cooper <jason@lakedaemon.net>, Andrew Lunn <andrew@lunn.ch>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	Zhang Rui <rui.zhang@intel.com>,
	Eduardo Valentin <edubezval@gmail.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
	devicetree@vger.kernel.org, linux-pm@vger.kernel.org,
	Antoine Tenart <antoine.tenart@bootlin.com>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will.deacon@arm.com>,
	Maxime Chevallier <maxime.chevallier@bootlin.com>,
	Nadav Haklai <nadavh@marvell.com>,
	David Sniatkiwicz <davidsn@marvell.com>,
	Rob Herring <robh+dt@kernel.org>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Miquel Raynal <miquel.raynal@bootlin.com>,
	linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 00/23] Add multi-channel support to Armada thermal driver
Date: Mon, 25 Jun 2018 17:12:16 +0200	[thread overview]
Message-ID: <20180625151239.20976-1-miquel.raynal@bootlin.com> (raw)

The only capability of the Armada thermal driver is currently just to
read one sensor (the default one) per AP and one per CP. Actually,
there is one sensor per core in the AP806 plus one sensor in the
thermal IP itself. The CP110 just features one thermal sensor in its
own thermal IP.

This series first improves the readability of this driver, then adds
support for multi-channel thermal IPs. The bindings and the
device-trees are updated accordingly.

Eduardo, this is a barely modified version of my previous series
adding overheat interrupt support. All the interrupt-related patches
have been dropped for now. It is now only rewording, reorganization
and multi-sensor support being added here; there is no more
dependency. As I did not receive any feedback since April, 24th I
would like it to be merged for 4.19.

Thank you,
Miquèl

Changes since v1:
=================
* Added register ranges in the thermal nodes.
* Added these ranges in the documentation also.
* In the bindings, used the "Must be one of" formula before listing the
  compatibles.
* Removed the overheat interrupt support that depends on another
  series. This work is independent.
* Rebased on top of v4.18-rc1.


Miquel Raynal (23):
  thermal: armada: remove useless register accesses
  thermal: armada: remove misleading comments
  thermal: armada: rename the initialization routine
  thermal: armada: dissociate a380 and cp110 ->init() hooks
  thermal: armada: average over samples to avoid glitches
  thermal: armada: convert driver to syscon register accesses
  thermal: armada: use the resource managed registration helper
    alternative
  thermal: armada: add multi-channel sensors support
  thermal: armada: remove sensors validity from the IP initialization
  thermal: armada: move validity check out of the read function
  thermal: armada: get rid of the ->is_valid() pointer
  dt-bindings: cp110: rename cp110 syscon file
  dt-bindings: ap806: prepare the syscon file to list other syscons
    nodes
  dt-bindings: cp110: prepare the syscon file to list other syscons
    nodes
  dt-bindings: ap806: add the thermal node in the syscon file
  dt-bindings: cp110: update documentation since DT de-duplication
  dt-bindings: cp110: add the thermal node in the syscon file
  dt-bindings: thermal: armada: add reference to new bindings
  arm64: dts: marvell: rename ap806 syscon node
  arm64: dts: marvell: move AP806/CP110 thermal nodes into a new syscon
  arm64: dts: marvell: add thermal-zone node in ap806 DTSI file
  arm64: dts: marvell: add macro to make distinction between node names
  arm64: dts: marvell: add thermal-zone node in cp110 DTSI file

 .../arm/marvell/ap806-system-controller.txt        |  50 +-
 ...controller0.txt => cp110-system-controller.txt} |  61 ++-
 .../devicetree/bindings/thermal/armada-thermal.txt |   5 +
 arch/arm64/boot/dts/marvell/armada-ap806.dtsi      |  76 ++-
 arch/arm64/boot/dts/marvell/armada-common.dtsi     |   1 +
 arch/arm64/boot/dts/marvell/armada-cp110.dtsi      |  37 +-
 drivers/thermal/armada_thermal.c                   | 510 +++++++++++++++------
 7 files changed, 583 insertions(+), 157 deletions(-)
 rename Documentation/devicetree/bindings/arm/marvell/{cp110-system-controller0.txt => cp110-system-controller.txt} (85%)

-- 
2.14.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: miquel.raynal@bootlin.com (Miquel Raynal)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 00/23] Add multi-channel support to Armada thermal driver
Date: Mon, 25 Jun 2018 17:12:16 +0200	[thread overview]
Message-ID: <20180625151239.20976-1-miquel.raynal@bootlin.com> (raw)

The only capability of the Armada thermal driver is currently just to
read one sensor (the default one) per AP and one per CP. Actually,
there is one sensor per core in the AP806 plus one sensor in the
thermal IP itself. The CP110 just features one thermal sensor in its
own thermal IP.

This series first improves the readability of this driver, then adds
support for multi-channel thermal IPs. The bindings and the
device-trees are updated accordingly.

Eduardo, this is a barely modified version of my previous series
adding overheat interrupt support. All the interrupt-related patches
have been dropped for now. It is now only rewording, reorganization
and multi-sensor support being added here; there is no more
dependency. As I did not receive any feedback since April, 24th I
would like it to be merged for 4.19.

Thank you,
Miqu?l

Changes since v1:
=================
* Added register ranges in the thermal nodes.
* Added these ranges in the documentation also.
* In the bindings, used the "Must be one of" formula before listing the
  compatibles.
* Removed the overheat interrupt support that depends on another
  series. This work is independent.
* Rebased on top of v4.18-rc1.


Miquel Raynal (23):
  thermal: armada: remove useless register accesses
  thermal: armada: remove misleading comments
  thermal: armada: rename the initialization routine
  thermal: armada: dissociate a380 and cp110 ->init() hooks
  thermal: armada: average over samples to avoid glitches
  thermal: armada: convert driver to syscon register accesses
  thermal: armada: use the resource managed registration helper
    alternative
  thermal: armada: add multi-channel sensors support
  thermal: armada: remove sensors validity from the IP initialization
  thermal: armada: move validity check out of the read function
  thermal: armada: get rid of the ->is_valid() pointer
  dt-bindings: cp110: rename cp110 syscon file
  dt-bindings: ap806: prepare the syscon file to list other syscons
    nodes
  dt-bindings: cp110: prepare the syscon file to list other syscons
    nodes
  dt-bindings: ap806: add the thermal node in the syscon file
  dt-bindings: cp110: update documentation since DT de-duplication
  dt-bindings: cp110: add the thermal node in the syscon file
  dt-bindings: thermal: armada: add reference to new bindings
  arm64: dts: marvell: rename ap806 syscon node
  arm64: dts: marvell: move AP806/CP110 thermal nodes into a new syscon
  arm64: dts: marvell: add thermal-zone node in ap806 DTSI file
  arm64: dts: marvell: add macro to make distinction between node names
  arm64: dts: marvell: add thermal-zone node in cp110 DTSI file

 .../arm/marvell/ap806-system-controller.txt        |  50 +-
 ...controller0.txt => cp110-system-controller.txt} |  61 ++-
 .../devicetree/bindings/thermal/armada-thermal.txt |   5 +
 arch/arm64/boot/dts/marvell/armada-ap806.dtsi      |  76 ++-
 arch/arm64/boot/dts/marvell/armada-common.dtsi     |   1 +
 arch/arm64/boot/dts/marvell/armada-cp110.dtsi      |  37 +-
 drivers/thermal/armada_thermal.c                   | 510 +++++++++++++++------
 7 files changed, 583 insertions(+), 157 deletions(-)
 rename Documentation/devicetree/bindings/arm/marvell/{cp110-system-controller0.txt => cp110-system-controller.txt} (85%)

-- 
2.14.1

             reply	other threads:[~2018-06-25 15:12 UTC|newest]

Thread overview: 88+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-25 15:12 Miquel Raynal [this message]
2018-06-25 15:12 ` [PATCH v2 00/23] Add multi-channel support to Armada thermal driver Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 01/23] thermal: armada: remove useless register accesses Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 02/23] thermal: armada: remove misleading comments Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 03/23] thermal: armada: rename the initialization routine Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 04/23] thermal: armada: dissociate a380 and cp110 ->init() hooks Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 05/23] thermal: armada: average over samples to avoid glitches Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 06/23] thermal: armada: convert driver to syscon register accesses Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-26  3:37   ` Baruch Siach
2018-06-26  3:37     ` Baruch Siach
2018-06-26  7:29     ` Miquel Raynal
2018-06-26  7:29       ` Miquel Raynal
2018-07-02 17:29       ` Eduardo Valentin
2018-07-02 17:29         ` Eduardo Valentin
2018-07-05 12:43         ` Miquel Raynal
2018-07-05 12:43           ` Miquel Raynal
2018-07-05 15:53           ` Miquel Raynal
2018-07-05 15:53             ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 07/23] thermal: armada: use the resource managed registration helper alternative Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 08/23] thermal: armada: add multi-channel sensors support Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 09/23] thermal: armada: remove sensors validity from the IP initialization Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 10/23] thermal: armada: move validity check out of the read function Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 11/23] thermal: armada: get rid of the ->is_valid() pointer Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 12/23] dt-bindings: cp110: rename cp110 syscon file Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 13/23] dt-bindings: ap806: prepare the syscon file to list other syscons nodes Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-07-03 21:07   ` Rob Herring
2018-07-03 21:07     ` Rob Herring
2018-06-25 15:12 ` [PATCH v2 14/23] dt-bindings: cp110: " Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-26  3:35   ` Baruch Siach
2018-06-26  3:35     ` Baruch Siach
2018-06-26  7:34     ` Miquel Raynal
2018-06-26  7:34       ` Miquel Raynal
2018-07-03 21:07   ` Rob Herring
2018-07-03 21:07     ` Rob Herring
2018-06-25 15:12 ` [PATCH v2 15/23] dt-bindings: ap806: add the thermal node in the syscon file Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-07-03 21:13   ` Rob Herring
2018-07-03 21:13     ` Rob Herring
2018-06-25 15:12 ` [PATCH v2 16/23] dt-bindings: cp110: update documentation since DT de-duplication Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 17/23] dt-bindings: cp110: add the thermal node in the syscon file Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-07-03 21:14   ` Rob Herring
2018-07-03 21:14     ` Rob Herring
2018-06-25 15:12 ` [PATCH v2 18/23] dt-bindings: thermal: armada: add reference to new bindings Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-07-03 21:30   ` Rob Herring
2018-07-03 21:30     ` Rob Herring
2018-07-05 10:05     ` Miquel Raynal
2018-07-05 10:05       ` Miquel Raynal
2018-07-05 18:03       ` Rob Herring
2018-07-05 18:03         ` Rob Herring
2018-07-06  6:54         ` Miquel Raynal
2018-07-06  6:54           ` Miquel Raynal
2018-07-06 17:51           ` Rob Herring
2018-07-06 17:51             ` Rob Herring
2018-07-06  7:49         ` Miquel Raynal
2018-07-06  7:49           ` Miquel Raynal
2018-07-06 17:56           ` Rob Herring
2018-07-06 17:56             ` Rob Herring
2018-07-16 14:32             ` Miquel Raynal
2018-07-16 14:32               ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 19/23] arm64: dts: marvell: rename ap806 syscon node Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-07-03 21:33   ` Rob Herring
2018-07-03 21:33     ` Rob Herring
2018-06-25 15:12 ` [PATCH v2 20/23] arm64: dts: marvell: move AP806/CP110 thermal nodes into a new syscon Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 21/23] arm64: dts: marvell: add thermal-zone node in ap806 DTSI file Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 22/23] arm64: dts: marvell: add macro to make distinction between node names Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal
2018-06-25 15:12 ` [PATCH v2 23/23] arm64: dts: marvell: add thermal-zone node in cp110 DTSI file Miquel Raynal
2018-06-25 15:12   ` Miquel Raynal

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180625151239.20976-1-miquel.raynal@bootlin.com \
    --to=miquel.raynal@bootlin.com \
    --cc=andrew@lunn.ch \
    --cc=antoine.tenart@bootlin.com \
    --cc=catalin.marinas@arm.com \
    --cc=davidsn@marvell.com \
    --cc=devicetree@vger.kernel.org \
    --cc=edubezval@gmail.com \
    --cc=gregory.clement@bootlin.com \
    --cc=jason@lakedaemon.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=maxime.chevallier@bootlin.com \
    --cc=nadavh@marvell.com \
    --cc=robh+dt@kernel.org \
    --cc=rui.zhang@intel.com \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=will.deacon@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.