On 8 May 2014 12:50, Jingoo Han <[email protected]> wrote: > On Thursday, May 08, 2014 3:20 PM, Rahul Sharma wrote: >> >> From: Rahul Sharma <[email protected]> >> >> Enable fimd node for snow board. >> >> Signed-off-by: Rahul Sharma <[email protected]> >> --- >> 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. >
I just followed the nodes in rest of the file for peach and snow. This looks consistent to me. But let me know if you want me to change this. Regards, Rahul Sharma. > Best regards, > Jingoo Han > >> + samsung,invert-vclk; >> + }; >> + >> }; >> -- >> 1.7.9.5 > > -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
