Re: [PATCH v6 1/3] MIPS: DTS: CI20: add DT node for IR sensor

[Paul Cercueil <paul@crapouillou.net>]

Hi Nikolaus,


Le ven., mars 6, 2020 at 18:29, H. Nikolaus Schaller 
<hns@goldelico.com> a écrit :
> From: Alex Smith <alex.smith@imgtec.com>
> 
> The infrared sensor on the CI20 board is connected to a GPIO and can
> be operated by using the gpio-ir-recv driver. Add a DT node for the
> sensor to allow that driver to be used.
> 
> Signed-off-by: Alex Smith <alex.smith@imgtec.com>
> Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com>
> ---
>  arch/mips/boot/dts/ingenic/ci20.dts | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/mips/boot/dts/ingenic/ci20.dts 
> b/arch/mips/boot/dts/ingenic/ci20.dts
> index c340f947baa0..e6ae08c979ea 100644
> --- a/arch/mips/boot/dts/ingenic/ci20.dts
> +++ b/arch/mips/boot/dts/ingenic/ci20.dts
> @@ -62,6 +62,11 @@
>  		enable-active-high;
>  	};
> 
> +	ir: ir-receiver {

According to Documentation/devicetree/bindings/media/rc.yaml, the node 
should be named just "ir".

Cheers,
-Paul

> +		compatible = "gpio-ir-receiver";
> +		gpios = <&gpe 3 GPIO_ACTIVE_LOW>;
> +	};
> +
>  	wlan0_power: fixedregulator@1 {
>  		compatible = "regulator-fixed";
>  		regulator-name = "wlan0_power";
> --
> 2.23.0
>