Re: [PATCH 13/20] arm64: dts: renesas: Use interrupts-extended for HDMI bridges

["Niklas Söderlund" <niklas.soderlund@ragnatech.se>]

On 2024-10-04 14:52:55 +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/condor-common.dtsi  | 3 +--
>  arch/arm64/boot/dts/renesas/draak.dtsi          | 3 +--
>  arch/arm64/boot/dts/renesas/ebisu.dtsi          | 3 +--
>  arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts | 3 +--
>  arch/arm64/boot/dts/renesas/r8a77970-eagle.dts  | 3 +--
>  arch/arm64/boot/dts/renesas/r8a77970-v3msk.dts  | 3 +--
>  arch/arm64/boot/dts/renesas/r8a77980-v3hsk.dts  | 3 +--
>  arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi    | 3 +--
>  arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi   | 3 +--
>  arch/arm64/boot/dts/renesas/ulcb-kf.dtsi        | 3 +--
>  10 files changed, 10 insertions(+), 20 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/renesas/condor-common.dtsi b/arch/arm64/boot/dts/renesas/condor-common.dtsi
> index b2d99dfaa0cdf19d..375a56b20f267bf0 100644
> --- a/arch/arm64/boot/dts/renesas/condor-common.dtsi
> +++ b/arch/arm64/boot/dts/renesas/condor-common.dtsi
> @@ -195,8 +195,7 @@ io_expander1: gpio@21 {
>  	hdmi@39 {
>  		compatible = "adi,adv7511w";
>  		reg = <0x39>;
> -		interrupt-parent = <&gpio1>;
> -		interrupts = <20 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&gpio1 20 IRQ_TYPE_LEVEL_LOW>;
>  		avdd-supply = <&d1_8v>;
>  		dvdd-supply = <&d1_8v>;
>  		pvdd-supply = <&d1_8v>;
> diff --git a/arch/arm64/boot/dts/renesas/draak.dtsi b/arch/arm64/boot/dts/renesas/draak.dtsi
> index 402112a37d75a8c5..05712cd96d28bbdf 100644
> --- a/arch/arm64/boot/dts/renesas/draak.dtsi
> +++ b/arch/arm64/boot/dts/renesas/draak.dtsi
> @@ -367,8 +367,7 @@ hdmi-encoder@39 {
>  		compatible = "adi,adv7511w";
>  		reg = <0x39>, <0x3f>, <0x3c>, <0x38>;
>  		reg-names = "main", "edid", "cec", "packet";
> -		interrupt-parent = <&gpio1>;
> -		interrupts = <28 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&gpio1 28 IRQ_TYPE_LEVEL_LOW>;
>  
>  		avdd-supply = <&reg_1p8v>;
>  		dvdd-supply = <&reg_1p8v>;
> diff --git a/arch/arm64/boot/dts/renesas/ebisu.dtsi b/arch/arm64/boot/dts/renesas/ebisu.dtsi
> index 1aedd093fb41bf44..4d16b8f0eae5474b 100644
> --- a/arch/arm64/boot/dts/renesas/ebisu.dtsi
> +++ b/arch/arm64/boot/dts/renesas/ebisu.dtsi
> @@ -399,8 +399,7 @@ io_expander: gpio@20 {
>  	hdmi-encoder@39 {
>  		compatible = "adi,adv7511w";
>  		reg = <0x39>;
> -		interrupt-parent = <&gpio1>;
> -		interrupts = <1 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&gpio1 1 IRQ_TYPE_LEVEL_LOW>;
>  
>  		avdd-supply = <&reg_1p8v>;
>  		dvdd-supply = <&reg_1p8v>;
> diff --git a/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts b/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts
> index d42e24d9c09b9162..486688b789b8cd58 100644
> --- a/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts
> @@ -232,8 +232,7 @@ hd3ss3220_out_ep: endpoint {
>  	tda19988: tda19988@70 {
>  		compatible = "nxp,tda998x";
>  		reg = <0x70>;
> -		interrupt-parent = <&gpio1>;
> -		interrupts = <1 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&gpio1 1 IRQ_TYPE_LEVEL_LOW>;
>  
>  		video-ports = <0x234501>;
>  
> diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> index 7dd9e13cf0074442..32f07aa2731678a5 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> @@ -171,8 +171,7 @@ io_expander: gpio@20 {
>  	hdmi@39 {
>  		compatible = "adi,adv7511w";
>  		reg = <0x39>;
> -		interrupt-parent = <&gpio1>;
> -		interrupts = <20 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&gpio1 20 IRQ_TYPE_LEVEL_LOW>;
>  
>  		avdd-supply = <&d1p8>;
>  		dvdd-supply = <&d1p8>;
> diff --git a/arch/arm64/boot/dts/renesas/r8a77970-v3msk.dts b/arch/arm64/boot/dts/renesas/r8a77970-v3msk.dts
> index 0a103f93b14d71ad..118e77f4477e389c 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77970-v3msk.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77970-v3msk.dts
> @@ -148,8 +148,7 @@ hdmi@39 {
>  		compatible = "adi,adv7511w";
>  		#sound-dai-cells = <0>;
>  		reg = <0x39>;
> -		interrupt-parent = <&gpio1>;
> -		interrupts = <20 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&gpio1 20 IRQ_TYPE_LEVEL_LOW>;
>  		avdd-supply = <&vcc_d1_8v>;
>  		dvdd-supply = <&vcc_d1_8v>;
>  		pvdd-supply = <&vcc_d1_8v>;
> diff --git a/arch/arm64/boot/dts/renesas/r8a77980-v3hsk.dts b/arch/arm64/boot/dts/renesas/r8a77980-v3hsk.dts
> index a8a20c748ffcd1ed..b409a8d1737e629c 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77980-v3hsk.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77980-v3hsk.dts
> @@ -140,8 +140,7 @@ hdmi@39 {
>  		compatible = "adi,adv7511w";
>  		#sound-dai-cells = <0>;
>  		reg = <0x39>;
> -		interrupt-parent = <&gpio1>;
> -		interrupts = <20 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&gpio1 20 IRQ_TYPE_LEVEL_LOW>;
>  		avdd-supply = <&vcc1v8_d4>;
>  		dvdd-supply = <&vcc1v8_d4>;
>  		pvdd-supply = <&vcc1v8_d4>;
> diff --git a/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi b/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
> index ee3d96fdb6168b56..789f7b0b5ebcadc7 100644
> --- a/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
> +++ b/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
> @@ -64,8 +64,7 @@ adv7535: hdmi@3d {
>  		compatible = "adi,adv7535";
>  		reg = <0x3d>;
>  
> -		interrupt-parent = <&pinctrl>;
> -		interrupts = <RZG2L_GPIO(2, 1) IRQ_TYPE_EDGE_FALLING>;
> +		interrupts-extended = <&pinctrl RZG2L_GPIO(2, 1) IRQ_TYPE_EDGE_FALLING>;
>  		clocks = <&osc1>;
>  		clock-names = "cec";
>  		avdd-supply = <&reg_1p8v>;
> diff --git a/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi b/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi
> index 377849cbb462eae9..345b779e4f6015da 100644
> --- a/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi
> +++ b/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi
> @@ -86,8 +86,7 @@ adv7535: hdmi@3d {
>  		compatible = "adi,adv7535";
>  		reg = <0x3d>;
>  
> -		interrupt-parent = <&pinctrl>;
> -		interrupts = <RZG2L_GPIO(43, 1) IRQ_TYPE_EDGE_FALLING>;
> +		interrupts-extended = <&pinctrl RZG2L_GPIO(43, 1) IRQ_TYPE_EDGE_FALLING>;
>  		clocks = <&osc1>;
>  		clock-names = "cec";
>  		avdd-supply = <&reg_1p8v>;
> diff --git a/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi b/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
> index 431b37bf566192d2..5a5dd5ecb75e0e7c 100644
> --- a/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
> +++ b/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
> @@ -150,8 +150,7 @@ hdmi@3d {
>  				pinctrl-0 = <&hdmi1_pins>;
>  				pinctrl-names = "default";
>  
> -				interrupt-parent = <&gpio2>;
> -				interrupts = <14 IRQ_TYPE_LEVEL_LOW>;
> +				interrupts-extended = <&gpio2 14 IRQ_TYPE_LEVEL_LOW>;
>  
>  				clocks = <&cs2000>;
>  				clock-names = "cec";
> -- 
> 2.34.1
> 
> 

-- 
Kind Regards,
Niklas Söderlund