Re: [PATCH 05/20] ARM: dts: renesas: Use interrupts-extended for touchpanels

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

On 2024-10-04 14:52:47 +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/iwg20d-q7-common.dtsi       | 3 +--
>  arch/arm/boot/dts/renesas/r8a7740-armadillo800eva.dts | 3 +--
>  arch/arm/boot/dts/renesas/r8a7742-iwg21d-q7.dts       | 3 +--
>  arch/arm/boot/dts/renesas/sh73a0-kzm9g.dts            | 3 +--
>  4 files changed, 4 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/renesas/iwg20d-q7-common.dtsi b/arch/arm/boot/dts/renesas/iwg20d-q7-common.dtsi
> index 4351c5a02fa596de..2cc2908b48ca1b9f 100644
> --- a/arch/arm/boot/dts/renesas/iwg20d-q7-common.dtsi
> +++ b/arch/arm/boot/dts/renesas/iwg20d-q7-common.dtsi
> @@ -219,8 +219,7 @@ sgtl5000: codec@a {
>  	touch: touchpanel@38 {
>  		compatible = "edt,edt-ft5406";
>  		reg = <0x38>;
> -		interrupt-parent = <&gpio2>;
> -		interrupts = <12 IRQ_TYPE_EDGE_FALLING>;
> +		interrupts-extended = <&gpio2 12 IRQ_TYPE_EDGE_FALLING>;
>  		vcc-supply = <&vcc_3v3_tft1>;
>  	};
>  };
> diff --git a/arch/arm/boot/dts/renesas/r8a7740-armadillo800eva.dts b/arch/arm/boot/dts/renesas/r8a7740-armadillo800eva.dts
> index 9ec0f098bf6eb5d7..3d32cf068abc20f1 100644
> --- a/arch/arm/boot/dts/renesas/r8a7740-armadillo800eva.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7740-armadillo800eva.dts
> @@ -238,8 +238,7 @@ eeprom@50 {
>  	touchscreen@55 {
>  		compatible = "sitronix,st1232";
>  		reg = <0x55>;
> -		interrupt-parent = <&irqpin1>;
> -		interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> +		interrupts-extended = <&irqpin1 2 IRQ_TYPE_LEVEL_LOW>;
>  		pinctrl-0 = <&st1232_pins>;
>  		pinctrl-names = "default";
>  		gpios = <&pfc 166 GPIO_ACTIVE_LOW>;
> diff --git a/arch/arm/boot/dts/renesas/r8a7742-iwg21d-q7.dts b/arch/arm/boot/dts/renesas/r8a7742-iwg21d-q7.dts
> index 64102b664055b475..6a8a0d2113b020c7 100644
> --- a/arch/arm/boot/dts/renesas/r8a7742-iwg21d-q7.dts
> +++ b/arch/arm/boot/dts/renesas/r8a7742-iwg21d-q7.dts
> @@ -202,8 +202,7 @@ sgtl5000: codec@a {
>  	touch: touchpanel@38 {
>  		compatible = "edt,edt-ft5406";
>  		reg = <0x38>;
> -		interrupt-parent = <&gpio0>;
> -		interrupts = <24 IRQ_TYPE_EDGE_FALLING>;
> +		interrupts-extended = <&gpio0 24 IRQ_TYPE_EDGE_FALLING>;
>  		/* GP1_29 is also shared with audio codec reset pin */
>  		reset-gpios = <&gpio1 29 GPIO_ACTIVE_LOW>;
>  		vcc-supply = <&vcc_3v3_tft1>;
> diff --git a/arch/arm/boot/dts/renesas/sh73a0-kzm9g.dts b/arch/arm/boot/dts/renesas/sh73a0-kzm9g.dts
> index 15d89c20618a16b7..fe96ea07628779c6 100644
> --- a/arch/arm/boot/dts/renesas/sh73a0-kzm9g.dts
> +++ b/arch/arm/boot/dts/renesas/sh73a0-kzm9g.dts
> @@ -296,8 +296,7 @@ &i2c1 {
>  	touchscreen@55 {
>  		compatible = "sitronix,st1232";
>  		reg = <0x55>;
> -		interrupt-parent = <&irqpin1>;
> -		interrupts = <0 IRQ_TYPE_EDGE_FALLING>;
> +		interrupts-extended = <&irqpin1 0 IRQ_TYPE_EDGE_FALLING>;
>  	};
>  };
>  
> -- 
> 2.34.1
> 
> 

-- 
Kind Regards,
Niklas Söderlund