On Thursday, May 08, 2014 3:20 PM, Rahul Sharma wrote: > > From: Rahul Sharma <Rahul.Sharma@xxxxxxxxxxx> > > Enable fimd node for snow board. > > Signed-off-by: Rahul Sharma <Rahul.Sharma@xxxxxxxxxxx> > --- > arch/arm/boot/dts/exynos5250-snow.dts | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts > index 1ce1088..796954c 100644 > --- a/arch/arm/boot/dts/exynos5250-snow.dts > +++ b/arch/arm/boot/dts/exynos5250-snow.dts > @@ -206,4 +206,10 @@ > clock-frequency = <24000000>; > }; > }; > + > + fimd@14400000 { > + status = "okay"; (+cc Sachin Kamat, Tomasz Figa) For readability, how about moving 'status' to the last entry of the node? The same comment applies to other patches, too. Best regards, Jingoo Han > + samsung,invert-vclk; > + }; > + > }; > -- > 1.7.9.5 -- 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