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 02/20] ARM: dts: renesas: Use interrupts-extended for Ethernet PHYs
Date: Fri, 4 Oct 2024 17:04:26 +0200	[thread overview]
Message-ID: <20241004150426.GI2071111@ragnatech.se> (raw)
In-Reply-To: <623645456e7636f43150a70f8603114b26304818.1728045620.git.geert+renesas@glider.be>

On 2024-10-04 14:52:44 +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/r8a7743-sk-rzg1m.dts    | 3 +--
>  arch/arm/boot/dts/renesas/r8a7745-sk-rzg1e.dts    | 3 +--
>  arch/arm/boot/dts/renesas/r8a77470-iwg23s-sbc.dts | 3 +--
>  arch/arm/boot/dts/renesas/r8a7790-lager.dts       | 3 +--
>  arch/arm/boot/dts/renesas/r8a7790-stout.dts       | 3 +--
>  arch/arm/boot/dts/renesas/r8a7791-koelsch.dts     | 3 +--
>  arch/arm/boot/dts/renesas/r8a7791-porter.dts      | 3 +--
>  arch/arm/boot/dts/renesas/r8a7793-gose.dts        | 3 +--
>  arch/arm/boot/dts/renesas/r8a7794-alt.dts         | 3 +--
>  arch/arm/boot/dts/renesas/r8a7794-silk.dts        | 3 +--
>  10 files changed, 10 insertions(+), 20 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/renesas/r8a7743-sk-rzg1m.dts b/arch/arm/boot/dts/renesas/r8a7743-sk-rzg1m.dts
> index ff274bfcb6646ea7..9b16fe7ce713c903 100644
> --- a/arch/arm/boot/dts/renesas/r8a7743-sk-rzg1m.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7743-sk-rzg1m.dts
> @@ -73,8 +73,7 @@ phy1: ethernet-phy@1 {
>  		compatible = "ethernet-phy-id0022.1537",
>  			     "ethernet-phy-ieee802.3-c22";
>  		reg = <1>;
> -		interrupt-parent = <&irqc>;
> -		interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&irqc 0 IRQ_TYPE_LEVEL_LOW>;
>  		micrel,led-mode = <1>;
>  		reset-gpios = <&gpio5 22 GPIO_ACTIVE_LOW>;
>  	};
> diff --git a/arch/arm/boot/dts/renesas/r8a7745-sk-rzg1e.dts b/arch/arm/boot/dts/renesas/r8a7745-sk-rzg1e.dts
> index 0a75e8c79acc0e58..571615a506207169 100644
> --- a/arch/arm/boot/dts/renesas/r8a7745-sk-rzg1e.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7745-sk-rzg1e.dts
> @@ -68,8 +68,7 @@ phy1: ethernet-phy@1 {
>  		compatible = "ethernet-phy-id0022.1537",
>  			     "ethernet-phy-ieee802.3-c22";
>  		reg = <1>;
> -		interrupt-parent = <&irqc>;
> -		interrupts = <8 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&irqc 8 IRQ_TYPE_LEVEL_LOW>;
>  		micrel,led-mode = <1>;
>  		reset-gpios = <&gpio1 24 GPIO_ACTIVE_LOW>;
>  	};
> diff --git a/arch/arm/boot/dts/renesas/r8a77470-iwg23s-sbc.dts b/arch/arm/boot/dts/renesas/r8a77470-iwg23s-sbc.dts
> index 6448022852491661..7c7a9f257567d4f8 100644
> --- a/arch/arm/boot/dts/renesas/r8a77470-iwg23s-sbc.dts
> +++ b/arch/arm/boot/dts/renesas/r8a77470-iwg23s-sbc.dts
> @@ -82,8 +82,7 @@ phy3: ethernet-phy@3 {
>  		compatible = "ethernet-phy-id0022.1622",
>  			     "ethernet-phy-ieee802.3-c22";
>  		reg = <3>;
> -		interrupt-parent = <&gpio5>;
> -		interrupts = <16 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&gpio5 16 IRQ_TYPE_LEVEL_LOW>;
>  		micrel,led-mode = <1>;
>  	};
>  };
> diff --git a/arch/arm/boot/dts/renesas/r8a7790-lager.dts b/arch/arm/boot/dts/renesas/r8a7790-lager.dts
> index 826a29064d27f663..97c13b31f476505c 100644
> --- a/arch/arm/boot/dts/renesas/r8a7790-lager.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7790-lager.dts
> @@ -692,8 +692,7 @@ phy1: ethernet-phy@1 {
>  		compatible = "ethernet-phy-id0022.1537",
>  			     "ethernet-phy-ieee802.3-c22";
>  		reg = <1>;
> -		interrupt-parent = <&irqc0>;
> -		interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&irqc0 0 IRQ_TYPE_LEVEL_LOW>;
>  		micrel,led-mode = <1>;
>  		reset-gpios = <&gpio5 31 GPIO_ACTIVE_LOW>;
>  	};
> diff --git a/arch/arm/boot/dts/renesas/r8a7790-stout.dts b/arch/arm/boot/dts/renesas/r8a7790-stout.dts
> index 683f7395fab0b696..1593609064104ae4 100644
> --- a/arch/arm/boot/dts/renesas/r8a7790-stout.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7790-stout.dts
> @@ -211,8 +211,7 @@ phy1: ethernet-phy@1 {
>  		compatible = "ethernet-phy-id0022.1537",
>  			     "ethernet-phy-ieee802.3-c22";
>  		reg = <1>;
> -		interrupt-parent = <&irqc0>;
> -		interrupts = <1 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&irqc0 1 IRQ_TYPE_LEVEL_LOW>;
>  		micrel,led-mode = <1>;
>  		reset-gpios = <&gpio3 31 GPIO_ACTIVE_LOW>;
>  	};
> diff --git a/arch/arm/boot/dts/renesas/r8a7791-koelsch.dts b/arch/arm/boot/dts/renesas/r8a7791-koelsch.dts
> index 0a8eac57135f2e81..758d8331e7af66c4 100644
> --- a/arch/arm/boot/dts/renesas/r8a7791-koelsch.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7791-koelsch.dts
> @@ -659,8 +659,7 @@ phy1: ethernet-phy@1 {
>  		compatible = "ethernet-phy-id0022.1537",
>  			     "ethernet-phy-ieee802.3-c22";
>  		reg = <1>;
> -		interrupt-parent = <&irqc0>;
> -		interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&irqc0 0 IRQ_TYPE_LEVEL_LOW>;
>  		micrel,led-mode = <1>;
>  		reset-gpios = <&gpio5 22 GPIO_ACTIVE_LOW>;
>  	};
> diff --git a/arch/arm/boot/dts/renesas/r8a7791-porter.dts b/arch/arm/boot/dts/renesas/r8a7791-porter.dts
> index 93c86e9216455577..267b3623655fbc38 100644
> --- a/arch/arm/boot/dts/renesas/r8a7791-porter.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7791-porter.dts
> @@ -329,8 +329,7 @@ phy1: ethernet-phy@1 {
>  		compatible = "ethernet-phy-id0022.1537",
>  			     "ethernet-phy-ieee802.3-c22";
>  		reg = <1>;
> -		interrupt-parent = <&irqc0>;
> -		interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&irqc0 0 IRQ_TYPE_LEVEL_LOW>;
>  		micrel,led-mode = <1>;
>  		reset-gpios = <&gpio5 22 GPIO_ACTIVE_LOW>;
>  	};
> diff --git a/arch/arm/boot/dts/renesas/r8a7793-gose.dts b/arch/arm/boot/dts/renesas/r8a7793-gose.dts
> index 1ea6c757893bc0bf..f3f62206088348c6 100644
> --- a/arch/arm/boot/dts/renesas/r8a7793-gose.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7793-gose.dts
> @@ -622,8 +622,7 @@ phy1: ethernet-phy@1 {
>  		compatible = "ethernet-phy-id0022.1537",
>  			     "ethernet-phy-ieee802.3-c22";
>  		reg = <1>;
> -		interrupt-parent = <&irqc0>;
> -		interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&irqc0 0 IRQ_TYPE_LEVEL_LOW>;
>  		micrel,led-mode = <1>;
>  		reset-gpios = <&gpio5 22 GPIO_ACTIVE_LOW>;
>  	};
> diff --git a/arch/arm/boot/dts/renesas/r8a7794-alt.dts b/arch/arm/boot/dts/renesas/r8a7794-alt.dts
> index b5ecafbb2e4de582..1e04b8630ef3f3ca 100644
> --- a/arch/arm/boot/dts/renesas/r8a7794-alt.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7794-alt.dts
> @@ -381,8 +381,7 @@ phy1: ethernet-phy@1 {
>  		compatible = "ethernet-phy-id0022.1537",
>  			     "ethernet-phy-ieee802.3-c22";
>  		reg = <1>;
> -		interrupt-parent = <&irqc0>;
> -		interrupts = <8 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&irqc0 8 IRQ_TYPE_LEVEL_LOW>;
>  		micrel,led-mode = <1>;
>  		reset-gpios = <&gpio1 24 GPIO_ACTIVE_LOW>;
>  	};
> diff --git a/arch/arm/boot/dts/renesas/r8a7794-silk.dts b/arch/arm/boot/dts/renesas/r8a7794-silk.dts
> index 595e074085eb4cd3..8857bb4a9b91d969 100644
> --- a/arch/arm/boot/dts/renesas/r8a7794-silk.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7794-silk.dts
> @@ -415,8 +415,7 @@ phy1: ethernet-phy@1 {
>  		compatible = "ethernet-phy-id0022.1537",
>  			     "ethernet-phy-ieee802.3-c22";
>  		reg = <1>;
> -		interrupt-parent = <&irqc0>;
> -		interrupts = <8 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&irqc0 8 IRQ_TYPE_LEVEL_LOW>;
>  		micrel,led-mode = <1>;
>  		reset-gpios = <&gpio1 24 GPIO_ACTIVE_LOW>;
>  	};
> -- 
> 2.34.1
> 
> 

-- 
Kind Regards,
Niklas Söderlund

  reply	other threads:[~2024-10-04 15:04 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 [this message]
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
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=20241004150426.GI2071111@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).