devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

  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).