diff mbox series

[17/20] arm64: dts: renesas: Use interrupts-extended for video decoders

Message ID 4ee2e3e1e0e174782bd4d2af44bb6150d24af551.1728045620.git.geert+renesas@glider.be (mailing list archive)
State Mainlined
Commit 53181056f063aab11db8242e4c467d958b92306d
Delegated to: Geert Uytterhoeven
Headers show
Series renesas: dts: Use interrupts-extended | expand

Commit Message

Geert Uytterhoeven Oct. 4, 2024, 12:52 p.m. UTC 
Use the more concise interrupts-extended property to fully describe the
interrupts.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm64/boot/dts/renesas/ebisu.dtsi                     | 5 ++---
 .../dts/renesas/r8a77970-eagle-function-expansion.dtso     | 7 +++----
 arch/arm64/boot/dts/renesas/salvator-common.dtsi           | 5 ++---
 3 files changed, 7 insertions(+), 10 deletions(-)

Comments

Niklas Söderlund Oct. 4, 2024, 5:41 p.m. UTC | #1 
On 2024-10-04 14:52:59 +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/arm64/boot/dts/renesas/ebisu.dtsi                     | 5 ++---
>  .../dts/renesas/r8a77970-eagle-function-expansion.dtso     | 7 +++----
>  arch/arm64/boot/dts/renesas/salvator-common.dtsi           | 5 ++---
>  3 files changed, 7 insertions(+), 10 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/renesas/ebisu.dtsi b/arch/arm64/boot/dts/renesas/ebisu.dtsi
> index b4e0347d3617a83e..ab8283656660059a 100644
> --- a/arch/arm64/boot/dts/renesas/ebisu.dtsi
> +++ b/arch/arm64/boot/dts/renesas/ebisu.dtsi
> @@ -434,10 +434,9 @@ video-receiver@70 {
>  		compatible = "adi,adv7482";
>  		reg = <0x70>;
>  
> -		interrupt-parent = <&gpio0>;
> +		interrupts-extended = <&gpio0 7 IRQ_TYPE_LEVEL_LOW>,
> +				      <&gpio0 17 IRQ_TYPE_LEVEL_LOW>;
>  		interrupt-names = "intrq1", "intrq2";
> -		interrupts = <7 IRQ_TYPE_LEVEL_LOW>,
> -			     <17 IRQ_TYPE_LEVEL_LOW>;
>  
>  		ports {
>  			#address-cells = <1>;
> diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
> index 3aa243c5f04c8022..9450d8ac94cbe977 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
> +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
> @@ -82,8 +82,7 @@ hdmi-decoder@4c {
>  		compatible = "adi,adv7612";
>  		reg = <0x4c>, <0x50>, <0x52>, <0x54>, <0x56>, <0x58>;
>  		reg-names = "main", "afe", "rep", "edid", "hdmi", "cp";
> -		interrupt-parent = <&gpio3>;
> -		interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&gpio3 2 IRQ_TYPE_LEVEL_LOW>;
>  		default-input = <0>;
>  
>  		ports {
> @@ -114,8 +113,8 @@ video-receiver@70 {
>  		       0x60 0x61 0x62 0x63 0x64 0x65>;
>  		reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater",
>  			    "infoframe", "cbus", "cec", "sdp", "txa", "txb" ;
> -		interrupt-parent = <&gpio3>;
> -		interrupts = <03 IRQ_TYPE_LEVEL_LOW>, <04 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&gpio3 3 IRQ_TYPE_LEVEL_LOW>,
> +				      <&gpio3 4 IRQ_TYPE_LEVEL_LOW>;
>  		interrupt-names = "intrq1", "intrq2";
>  
>  		ports {
> diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> index 08f4e35c414ebbcf..269cda6fae21de70 100644
> --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> @@ -544,10 +544,9 @@ video-receiver@70 {
>  		reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater",
>  			    "infoframe", "cbus", "cec", "sdp", "txa", "txb" ;
>  
> -		interrupt-parent = <&gpio6>;
> +		interrupts-extended = <&gpio6 30 IRQ_TYPE_LEVEL_LOW>,
> +				      <&gpio6 31 IRQ_TYPE_LEVEL_LOW>;
>  		interrupt-names = "intrq1", "intrq2";
> -		interrupts = <30 IRQ_TYPE_LEVEL_LOW>,
> -			     <31 IRQ_TYPE_LEVEL_LOW>;
>  
>  		ports {
>  			#address-cells = <1>;
> -- 
> 2.34.1
> 
>
Lad, Prabhakar Oct. 7, 2024, 6:24 p.m. UTC | #2 
On Fri, Oct 4, 2024 at 2:27 PM Geert Uytterhoeven
<geert+renesas@glider.be> wrote:
>
> Use the more concise interrupts-extended property to fully describe the
> interrupts.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>  arch/arm64/boot/dts/renesas/ebisu.dtsi                     | 5 ++---
>  .../dts/renesas/r8a77970-eagle-function-expansion.dtso     | 7 +++----
>  arch/arm64/boot/dts/renesas/salvator-common.dtsi           | 5 ++---
>  3 files changed, 7 insertions(+), 10 deletions(-)
>
Reviewed-by:  Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Cheers,
Prabhakar

> diff --git a/arch/arm64/boot/dts/renesas/ebisu.dtsi b/arch/arm64/boot/dts/renesas/ebisu.dtsi
> index b4e0347d3617a83e..ab8283656660059a 100644
> --- a/arch/arm64/boot/dts/renesas/ebisu.dtsi
> +++ b/arch/arm64/boot/dts/renesas/ebisu.dtsi
> @@ -434,10 +434,9 @@ video-receiver@70 {
>                 compatible = "adi,adv7482";
>                 reg = <0x70>;
>
> -               interrupt-parent = <&gpio0>;
> +               interrupts-extended = <&gpio0 7 IRQ_TYPE_LEVEL_LOW>,
> +                                     <&gpio0 17 IRQ_TYPE_LEVEL_LOW>;
>                 interrupt-names = "intrq1", "intrq2";
> -               interrupts = <7 IRQ_TYPE_LEVEL_LOW>,
> -                            <17 IRQ_TYPE_LEVEL_LOW>;
>
>                 ports {
>                         #address-cells = <1>;
> diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
> index 3aa243c5f04c8022..9450d8ac94cbe977 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
> +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
> @@ -82,8 +82,7 @@ hdmi-decoder@4c {
>                 compatible = "adi,adv7612";
>                 reg = <0x4c>, <0x50>, <0x52>, <0x54>, <0x56>, <0x58>;
>                 reg-names = "main", "afe", "rep", "edid", "hdmi", "cp";
> -               interrupt-parent = <&gpio3>;
> -               interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> +               interrupts-extended = <&gpio3 2 IRQ_TYPE_LEVEL_LOW>;
>                 default-input = <0>;
>
>                 ports {
> @@ -114,8 +113,8 @@ video-receiver@70 {
>                        0x60 0x61 0x62 0x63 0x64 0x65>;
>                 reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater",
>                             "infoframe", "cbus", "cec", "sdp", "txa", "txb" ;
> -               interrupt-parent = <&gpio3>;
> -               interrupts = <03 IRQ_TYPE_LEVEL_LOW>, <04 IRQ_TYPE_LEVEL_LOW>;
> +               interrupts-extended = <&gpio3 3 IRQ_TYPE_LEVEL_LOW>,
> +                                     <&gpio3 4 IRQ_TYPE_LEVEL_LOW>;
>                 interrupt-names = "intrq1", "intrq2";
>
>                 ports {
> diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> index 08f4e35c414ebbcf..269cda6fae21de70 100644
> --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> @@ -544,10 +544,9 @@ video-receiver@70 {
>                 reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater",
>                             "infoframe", "cbus", "cec", "sdp", "txa", "txb" ;
>
> -               interrupt-parent = <&gpio6>;
> +               interrupts-extended = <&gpio6 30 IRQ_TYPE_LEVEL_LOW>,
> +                                     <&gpio6 31 IRQ_TYPE_LEVEL_LOW>;
>                 interrupt-names = "intrq1", "intrq2";
> -               interrupts = <30 IRQ_TYPE_LEVEL_LOW>,
> -                            <31 IRQ_TYPE_LEVEL_LOW>;
>
>                 ports {
>                         #address-cells = <1>;
> --
> 2.34.1
>
>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/renesas/ebisu.dtsi b/arch/arm64/boot/dts/renesas/ebisu.dtsi
index b4e0347d3617a83e..ab8283656660059a 100644
--- a/arch/arm64/boot/dts/renesas/ebisu.dtsi
+++ b/arch/arm64/boot/dts/renesas/ebisu.dtsi
@@ -434,10 +434,9 @@  video-receiver@70 {
 		compatible = "adi,adv7482";
 		reg = <0x70>;
 
-		interrupt-parent = <&gpio0>;
+		interrupts-extended = <&gpio0 7 IRQ_TYPE_LEVEL_LOW>,
+				      <&gpio0 17 IRQ_TYPE_LEVEL_LOW>;
 		interrupt-names = "intrq1", "intrq2";
-		interrupts = <7 IRQ_TYPE_LEVEL_LOW>,
-			     <17 IRQ_TYPE_LEVEL_LOW>;
 
 		ports {
 			#address-cells = <1>;
diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
index 3aa243c5f04c8022..9450d8ac94cbe977 100644
--- a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
+++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso
@@ -82,8 +82,7 @@  hdmi-decoder@4c {
 		compatible = "adi,adv7612";
 		reg = <0x4c>, <0x50>, <0x52>, <0x54>, <0x56>, <0x58>;
 		reg-names = "main", "afe", "rep", "edid", "hdmi", "cp";
-		interrupt-parent = <&gpio3>;
-		interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
+		interrupts-extended = <&gpio3 2 IRQ_TYPE_LEVEL_LOW>;
 		default-input = <0>;
 
 		ports {
@@ -114,8 +113,8 @@  video-receiver@70 {
 		       0x60 0x61 0x62 0x63 0x64 0x65>;
 		reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater",
 			    "infoframe", "cbus", "cec", "sdp", "txa", "txb" ;
-		interrupt-parent = <&gpio3>;
-		interrupts = <03 IRQ_TYPE_LEVEL_LOW>, <04 IRQ_TYPE_LEVEL_LOW>;
+		interrupts-extended = <&gpio3 3 IRQ_TYPE_LEVEL_LOW>,
+				      <&gpio3 4 IRQ_TYPE_LEVEL_LOW>;
 		interrupt-names = "intrq1", "intrq2";
 
 		ports {
diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
index 08f4e35c414ebbcf..269cda6fae21de70 100644
--- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi
+++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
@@ -544,10 +544,9 @@  video-receiver@70 {
 		reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater",
 			    "infoframe", "cbus", "cec", "sdp", "txa", "txb" ;
 
-		interrupt-parent = <&gpio6>;
+		interrupts-extended = <&gpio6 30 IRQ_TYPE_LEVEL_LOW>,
+				      <&gpio6 31 IRQ_TYPE_LEVEL_LOW>;
 		interrupt-names = "intrq1", "intrq2";
-		interrupts = <30 IRQ_TYPE_LEVEL_LOW>,
-			     <31 IRQ_TYPE_LEVEL_LOW>;
 
 		ports {
 			#address-cells = <1>;