Hi Jaewon, On Thu, Jun 12, 2014 at 1:53 PM, Jaewon Kim <jaewon02.kim@xxxxxxxxxxx> wrote: > pwm-cells should be 3. Third cell is optional PWM flags. > And This flag supported by this binding is PWM_POLARITY_INVERTED > > Signed-off-by: Jaewon Kim <jaewon02.kim@xxxxxxxxxxx> > --- > arch/arm/boot/dts/exynos4.dtsi | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi > index b8ece4b..b687e53 100644 > --- a/arch/arm/boot/dts/exynos4.dtsi > +++ b/arch/arm/boot/dts/exynos4.dtsi > @@ -548,13 +548,13 @@ > status = "disabled"; > }; > > - pwm@139D0000 { > + pwm: pwm@139D0000 { unrelated change. > compatible = "samsung,exynos4210-pwm"; > reg = <0x139D0000 0x1000>; > interrupts = <0 37 0>, <0 38 0>, <0 39 0>, <0 40 0>, <0 41 0>; > clocks = <&clock CLK_PWM>; > clock-names = "timers"; > - #pwm-cells = <2>; > + #pwm-cells = <3>; > status = "disabled"; > }; Otherwise looks good. Reviewed-by: Sachin Kamat <sachin.kamat@xxxxxxxxxxx> Regards, Sachin -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html