Hi Jaewon, On Thu, Jun 19, 2014 at 7:33 AM, 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> > --- > > Changes in v2: > - Remove unnecessary handle. > > arch/arm/boot/dts/exynos4.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi > index b8ece4b..e118356 100644 > --- a/arch/arm/boot/dts/exynos4.dtsi > +++ b/arch/arm/boot/dts/exynos4.dtsi > @@ -554,7 +554,7 @@ > 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"; > }; I think I had already reviewed this patch. Anyway, Reviewed-by: Sachin Kamat <sachin.kamat@xxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html