On Saturday 13 of October 2012 02:12:32 Rahul Sharma wrote: > This patch adds support for device tree based discovery for exynos5 > hdmiphy. > > Signed-off-by: Rahul Sharma <rahul.sharma@xxxxxxxxxxx> > --- > .../devicetree/bindings/drm/exynos/hdmiphy.txt | 12 ++++++++++++ > arch/arm/boot/dts/exynos5250-smdk5250.dts | 10 ++++++++++ > arch/arm/boot/dts/exynos5250.dtsi | 8 ++++++++ > arch/arm/mach-exynos/mach-exynos5-dt.c | 2 ++ > 4 files changed, 32 insertions(+), 0 deletions(-) > create mode 100644 > Documentation/devicetree/bindings/drm/exynos/hdmiphy.txt > > diff --git a/Documentation/devicetree/bindings/drm/exynos/hdmiphy.txt > b/Documentation/devicetree/bindings/drm/exynos/hdmiphy.txt new file > mode 100644 > index 0000000..c4face5 > --- /dev/null > +++ b/Documentation/devicetree/bindings/drm/exynos/hdmiphy.txt > @@ -0,0 +1,12 @@ > +Device-Tree bindings for hdmiphy driver > + > +Required properties: > +- compatible: value should be "samsung,exynos5-hdmiphy". > +- reg: I2C address of the hdmiphy device. > + > +Example: > + > + hdmiphy { > + compatible = "samsung,exynos5-hdmiphy"; > + reg = <0x38>; > + }; > \ No newline at end of file Please put a new line at end of file. > diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts > b/arch/arm/boot/dts/exynos5250-smdk5250.dts index 2a478ab..af9271c > 100644 > --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts > +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts > @@ -79,6 +79,16 @@ > status = "disabled"; > }; > > + i2c@12CE0000 { Wrong indentation. > + samsung,i2c-sda-delay = <100>; > + samsung,i2c-max-bus-freq = <66000>; > + > + hdmiphy { hdmiphy@38 > + compatible = "samsung,exynos5-hdmiphy"; > + reg = <0x38>; > + }; > + }; > + Rest of the patch looks fine. Best regards, Tomasz Figa -- 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