Hi, On Mon, Apr 30, 2012 at 12:14 PM, Thomas Abraham <thomas.abraham@xxxxxxxxxx> wrote: > Updated EXYNOS5 device tree source files to reflect changes in rev1 > of EXYNOS5 SoC. This includes new additions to the EXYNOS5 dtsi and > SMDK5250 dts files and few minor fixes. > > Signed-off-by: Thomas Abraham <thomas.abraham@xxxxxxxxxx> > Signed-off-by: Kukjin Kim <kgene.kim@xxxxxxxxxxx> > --- > arch/arm/boot/dts/exynos5250-smdk5250.dts | 52 +++++++++++++++++++++++++++++ > arch/arm/boot/dts/exynos5250.dtsi | 44 +++++++----------------- > 2 files changed, 65 insertions(+), 31 deletions(-) > > diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts > index 399d17b..bcc4b89 100644 > --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts > +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts > @@ -23,4 +23,56 @@ > chosen { > bootargs = "root=/dev/ram0 rw ramdisk=8192 console=ttySAC1,115200"; > }; > + > + i2c@12C60000 { > + #address-cells = <1>; > + #size-cells = <0>; Two nits: #a-c and #s-c is likely to always be 1/0, so you might as well move these to the dtsi file for the SoC. Also, we tend to use non-caps here and in other locations (so it should be 12c60000 here). -Olof -- 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