On Thu, Apr 27, 2017 at 05:40:51PM +0300, Laurent Pinchart wrote: > 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 ? Hi Ulrich, I'm wondering if you have any plans to follow-up on this patch.