Hi Ulrich, Thank you for the patch. On Thursday 27 Apr 2017 16:37:42 Ulrich Hecht wrote: > From: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx> > > This patch enables PWM2 for Salvator-X board on R8A7795 SoC. > > Signed-off-by: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx> > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> > --- > 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