Re: [PATCH v2 7/8] arm64: dts: r8a7795-salvator-x: Enable PWM2

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Ulrich,

Thank you for the patch.

On Thursday 27 Apr 2017 16:37:42 Ulrich Hecht wrote:
> From: Takeshi Kihara <takeshi.kihara.df@renesas.com>
> 
> This patch enables PWM2 for Salvator-X board on R8A7795 SoC.
> 
> Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com>
> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
> ---
>  arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts index 8558b27..534b17e
> 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> @@ -381,6 +381,11 @@
>  		function = "pwm1";
>  	};
> 
> +	pwm2_pins: pwm2 {
> +		groups = "pwm2_a";
> +		function = "pwm2";
> +	};
> +
>  	sdhi0_pins: sd0 {
>  		groups = "sdhi0_data4", "sdhi0_ctrl";
>  		function = "sdhi0";
> @@ -463,6 +468,13 @@
>  	status = "okay";
>  };
> 
> +&pwm2 {
> +	/* PMIC DC/DC switching frequency synchronization */

Please pardon the stupid question, but if the PWM channel is used by the PMIC, 
is there a point in enabling it without a PMIC DT node using it ?

> +	pinctrl-0 = <&pwm2_pins>;
> +	pinctrl-names = "default";
> +	status = "okay";
> +};
> +
>  &scif1 {
>  	pinctrl-0 = <&scif1_pins>;
>  	pinctrl-names = "default";

-- 
Regards,

Laurent Pinchart