From: "Niklas Söderlund" <niklas.soderlund@ragnatech.se>
To: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Magnus Damm <magnus.damm@gmail.com>,
Wolfram Sang <wsa+renesas@sang-engineering.com>,
linux-renesas-soc@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org
Subject: Re: [PATCH 04/20] ARM: dts: renesas: Use interrupts-extended for PMICs
Date: Fri, 4 Oct 2024 17:27:03 +0200 [thread overview]
Message-ID: <20241004152703.GK2071111@ragnatech.se> (raw)
In-Reply-To: <934b9b9992dacd72dbad0f5433728aac292a3cfc.1728045620.git.geert+renesas@glider.be>
On 2024-10-04 14:52:46 +0200, Geert Uytterhoeven wrote:
> Use the more concise interrupts-extended property to fully describe the
> interrupts.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> ---
> arch/arm/boot/dts/renesas/r8a7790-lager.dts | 6 ++----
> arch/arm/boot/dts/renesas/r8a7790-stout.dts | 9 +++------
> arch/arm/boot/dts/renesas/r8a7791-koelsch.dts | 6 ++----
> arch/arm/boot/dts/renesas/r8a7791-porter.dts | 6 ++----
> arch/arm/boot/dts/renesas/r8a7792-blanche.dts | 3 +--
> arch/arm/boot/dts/renesas/r8a7793-gose.dts | 6 ++----
> arch/arm/boot/dts/renesas/r8a7794-alt.dts | 3 +--
> arch/arm/boot/dts/renesas/r8a7794-silk.dts | 3 +--
> 8 files changed, 14 insertions(+), 28 deletions(-)
>
> diff --git a/arch/arm/boot/dts/renesas/r8a7790-lager.dts b/arch/arm/boot/dts/renesas/r8a7790-lager.dts
> index 5ef87f8088c4c81c..47ffa278a0dfd79e 100644
> --- a/arch/arm/boot/dts/renesas/r8a7790-lager.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7790-lager.dts
> @@ -443,8 +443,7 @@ i2cpwr: i2c-mux4 {
> pmic@58 {
> compatible = "dlg,da9063";
> reg = <0x58>;
> - interrupt-parent = <&irqc0>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc0 2 IRQ_TYPE_LEVEL_LOW>;
> interrupt-controller;
> #interrupt-cells = <2>;
>
> @@ -460,8 +459,7 @@ watchdog {
> vdd_dvfs: regulator@68 {
> compatible = "dlg,da9210";
> reg = <0x68>;
> - interrupt-parent = <&irqc0>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc0 2 IRQ_TYPE_LEVEL_LOW>;
>
> regulator-min-microvolt = <1000000>;
> regulator-max-microvolt = <1000000>;
> diff --git a/arch/arm/boot/dts/renesas/r8a7790-stout.dts b/arch/arm/boot/dts/renesas/r8a7790-stout.dts
> index 9287724187ef3b69..d7c0a9574ce83144 100644
> --- a/arch/arm/boot/dts/renesas/r8a7790-stout.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7790-stout.dts
> @@ -342,8 +342,7 @@ &iic3 {
> pmic@58 {
> compatible = "dlg,da9063";
> reg = <0x58>;
> - interrupt-parent = <&irqc0>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc0 2 IRQ_TYPE_LEVEL_LOW>;
> interrupt-controller;
> #interrupt-cells = <2>;
>
> @@ -363,8 +362,7 @@ watchdog {
> vdd_dvfs: regulator@68 {
> compatible = "dlg,da9210";
> reg = <0x68>;
> - interrupt-parent = <&irqc0>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc0 2 IRQ_TYPE_LEVEL_LOW>;
>
> regulator-min-microvolt = <1000000>;
> regulator-max-microvolt = <1000000>;
> @@ -375,8 +373,7 @@ vdd_dvfs: regulator@68 {
> vdd: regulator@70 {
> compatible = "dlg,da9210";
> reg = <0x70>;
> - interrupt-parent = <&irqc0>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc0 2 IRQ_TYPE_LEVEL_LOW>;
>
> regulator-min-microvolt = <1000000>;
> regulator-max-microvolt = <1000000>;
> diff --git a/arch/arm/boot/dts/renesas/r8a7791-koelsch.dts b/arch/arm/boot/dts/renesas/r8a7791-koelsch.dts
> index bce93db4c9df5e18..1a0d2c6ed0e83ce7 100644
> --- a/arch/arm/boot/dts/renesas/r8a7791-koelsch.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7791-koelsch.dts
> @@ -814,8 +814,7 @@ &i2c6 {
> pmic@58 {
> compatible = "dlg,da9063";
> reg = <0x58>;
> - interrupt-parent = <&irqc0>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc0 2 IRQ_TYPE_LEVEL_LOW>;
> interrupt-controller;
> #interrupt-cells = <2>;
>
> @@ -831,8 +830,7 @@ watchdog {
> vdd_dvfs: regulator@68 {
> compatible = "dlg,da9210";
> reg = <0x68>;
> - interrupt-parent = <&irqc0>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc0 2 IRQ_TYPE_LEVEL_LOW>;
>
> regulator-min-microvolt = <1000000>;
> regulator-max-microvolt = <1000000>;
> diff --git a/arch/arm/boot/dts/renesas/r8a7791-porter.dts b/arch/arm/boot/dts/renesas/r8a7791-porter.dts
> index 92b54e043795ba08..08381498350aacde 100644
> --- a/arch/arm/boot/dts/renesas/r8a7791-porter.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7791-porter.dts
> @@ -408,8 +408,7 @@ &i2c6 {
> pmic@5a {
> compatible = "dlg,da9063l";
> reg = <0x5a>;
> - interrupt-parent = <&irqc0>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc0 2 IRQ_TYPE_LEVEL_LOW>;
> interrupt-controller;
> #interrupt-cells = <2>;
>
> @@ -421,8 +420,7 @@ watchdog {
> vdd_dvfs: regulator@68 {
> compatible = "dlg,da9210";
> reg = <0x68>;
> - interrupt-parent = <&irqc0>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc0 2 IRQ_TYPE_LEVEL_LOW>;
>
> regulator-min-microvolt = <1000000>;
> regulator-max-microvolt = <1000000>;
> diff --git a/arch/arm/boot/dts/renesas/r8a7792-blanche.dts b/arch/arm/boot/dts/renesas/r8a7792-blanche.dts
> index 69009535814406fe..a3986076d8e3e993 100644
> --- a/arch/arm/boot/dts/renesas/r8a7792-blanche.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7792-blanche.dts
> @@ -376,8 +376,7 @@ pmic@58 {
> reg = <0x58>;
> pinctrl-names = "default";
> pinctrl-0 = <&pmic_irq_pins>;
> - interrupt-parent = <&irqc>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc 2 IRQ_TYPE_LEVEL_LOW>;
> interrupt-controller;
> #interrupt-cells = <2>;
>
> diff --git a/arch/arm/boot/dts/renesas/r8a7793-gose.dts b/arch/arm/boot/dts/renesas/r8a7793-gose.dts
> index 45ef1d1900245a11..5334af25c10111c8 100644
> --- a/arch/arm/boot/dts/renesas/r8a7793-gose.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7793-gose.dts
> @@ -754,8 +754,7 @@ &i2c6 {
> pmic@58 {
> compatible = "dlg,da9063";
> reg = <0x58>;
> - interrupt-parent = <&irqc0>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc0 2 IRQ_TYPE_LEVEL_LOW>;
> interrupt-controller;
> #interrupt-cells = <2>;
>
> @@ -771,8 +770,7 @@ watchdog {
> vdd_dvfs: regulator@68 {
> compatible = "dlg,da9210";
> reg = <0x68>;
> - interrupt-parent = <&irqc0>;
> - interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&irqc0 2 IRQ_TYPE_LEVEL_LOW>;
>
> regulator-min-microvolt = <1000000>;
> regulator-max-microvolt = <1000000>;
> diff --git a/arch/arm/boot/dts/renesas/r8a7794-alt.dts b/arch/arm/boot/dts/renesas/r8a7794-alt.dts
> index 1e04b8630ef3f3ca..882644cd7c1875c1 100644
> --- a/arch/arm/boot/dts/renesas/r8a7794-alt.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7794-alt.dts
> @@ -449,8 +449,7 @@ &i2c7 {
> pmic@58 {
> compatible = "dlg,da9063";
> reg = <0x58>;
> - interrupt-parent = <&gpio3>;
> - interrupts = <31 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&gpio3 31 IRQ_TYPE_LEVEL_LOW>;
> interrupt-controller;
> #interrupt-cells = <2>;
>
> diff --git a/arch/arm/boot/dts/renesas/r8a7794-silk.dts b/arch/arm/boot/dts/renesas/r8a7794-silk.dts
> index 5ed5b426f9639775..2a0819311a3c4ef3 100644
> --- a/arch/arm/boot/dts/renesas/r8a7794-silk.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7794-silk.dts
> @@ -434,8 +434,7 @@ &i2c7 {
> pmic@58 {
> compatible = "dlg,da9063";
> reg = <0x58>;
> - interrupt-parent = <&gpio3>;
> - interrupts = <31 IRQ_TYPE_LEVEL_LOW>;
> + interrupts-extended = <&gpio3 31 IRQ_TYPE_LEVEL_LOW>;
> interrupt-controller;
> #interrupt-cells = <2>;
>
> --
> 2.34.1
>
>
--
Kind Regards,
Niklas Söderlund
next prev parent reply other threads:[~2024-10-04 15:27 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-04 12:52 [PATCH 00/20] renesas: dts: Use interrupts-extended Geert Uytterhoeven
2024-10-04 12:52 ` [PATCH 01/20] ARM: dts: renesas: Use interrupts-extended for Ethernet MACs Geert Uytterhoeven
2024-10-04 15:03 ` Niklas Söderlund
2024-10-07 17:05 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 02/20] ARM: dts: renesas: Use interrupts-extended for Ethernet PHYs Geert Uytterhoeven
2024-10-04 15:04 ` Niklas Söderlund
2024-10-07 17:02 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 03/20] ARM: dts: renesas: Use interrupts-extended for HDMI bridges Geert Uytterhoeven
2024-10-04 15:25 ` Niklas Söderlund
2024-10-07 17:04 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 04/20] ARM: dts: renesas: Use interrupts-extended for PMICs Geert Uytterhoeven
2024-10-04 15:27 ` Niklas Söderlund [this message]
2024-10-07 17:08 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 05/20] ARM: dts: renesas: Use interrupts-extended for touchpanels Geert Uytterhoeven
2024-10-04 15:48 ` Niklas Söderlund
2024-10-07 17:58 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 06/20] ARM: dts: renesas: Use interrupts-extended for video decoders Geert Uytterhoeven
2024-10-04 15:48 ` Niklas Söderlund
2024-10-07 17:58 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 07/20] ARM: dts: renesas: iwg22d-sodimm: Use interrupts-extended for port expander Geert Uytterhoeven
2024-10-04 15:49 ` Niklas Söderlund
2024-10-07 17:59 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 08/20] ARM: dts: renesas: r8a7742-iwg21m: Use interrupts-extended for RTC Geert Uytterhoeven
2024-10-04 15:49 ` Niklas Söderlund
2024-10-07 17:59 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 09/20] ARM: dts: renesas: kzm9g: Use interrupts-extended for I/O expander Geert Uytterhoeven
2024-10-04 15:51 ` Niklas Söderlund
2024-10-07 18:00 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 10/20] ARM: dts: renesas: kzm9g: Use interrupts-extended for sensors Geert Uytterhoeven
2024-10-04 16:23 ` Niklas Söderlund
2024-10-07 18:01 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 11/20] arm64: dts: renesas: Use interrupts-extended for DisplayPort bridges Geert Uytterhoeven
2024-10-04 16:31 ` Niklas Söderlund
2024-10-07 18:02 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 12/20] arm64: dts: renesas: Use interrupts-extended for Ethernet PHYs Geert Uytterhoeven
2024-10-04 16:44 ` Niklas Söderlund
2024-10-07 18:17 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 13/20] arm64: dts: renesas: Use interrupts-extended for HDMI bridges Geert Uytterhoeven
2024-10-04 16:56 ` Niklas Söderlund
2024-10-07 18:20 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 14/20] arm64: dts: renesas: Use interrupts-extended for I/O expanders Geert Uytterhoeven
2024-10-04 16:56 ` Niklas Söderlund
2024-10-07 18:22 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 15/20] arm64: dts: renesas: Use interrupts-extended for PMICs Geert Uytterhoeven
2024-10-04 17:38 ` Niklas Söderlund
2024-10-07 18:22 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 16/20] arm64: dts: renesas: Use interrupts-extended for USB muxes Geert Uytterhoeven
2024-10-04 17:39 ` Niklas Söderlund
2024-10-07 18:23 ` Lad, Prabhakar
2024-10-04 12:52 ` [PATCH 17/20] arm64: dts: renesas: Use interrupts-extended for video decoders Geert Uytterhoeven
2024-10-04 17:41 ` Niklas Söderlund
2024-10-07 18:24 ` Lad, Prabhakar
2024-10-04 12:53 ` [PATCH 18/20] arm64: dts: renesas: Use interrupts-extended for WLAN Geert Uytterhoeven
2024-10-04 17:41 ` Niklas Söderlund
2024-10-07 18:36 ` Lad, Prabhakar
2024-10-04 12:53 ` [PATCH 19/20] arm64: dts: renesas: beacon-renesom: Use interrupts-extended for touchscreen Geert Uytterhoeven
2024-10-04 17:41 ` Niklas Söderlund
2024-10-07 18:37 ` Lad, Prabhakar
2024-10-04 12:53 ` [PATCH 20/20] arm64: dts: renesas: rzg3s-smarc: Use interrupts-extended for gpio-keys Geert Uytterhoeven
2024-10-04 17:43 ` Niklas Söderlund
2024-10-07 18:38 ` Lad, Prabhakar
2024-10-09 7:46 ` claudiu beznea
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=20241004152703.GK2071111@ragnatech.se \
--to=niklas.soderlund@ragnatech.se \
--cc=devicetree@vger.kernel.org \
--cc=geert+renesas@glider.be \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=wsa+renesas@sang-engineering.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).