| Message ID | 1381920987-11095-1-git-send-email-l.krishna@samsung.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Leela, On Wednesday 16 of October 2013 16:26:26 Leela Krishna Amudala wrote: > Adds G-Scaler devices to the DT device list > > Signed-off-by: Leela Krishna Amudala <l.krishna@samsung.com> > --- > arch/arm/boot/dts/exynos5420.dtsi | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) Now as I think of it, patch subject should be rather something like: ARM: dts: Exynos5420: Add device nodes for GScaler blocks because the patch alone does not have anything to do with adding device tree support to GScaler driver. Also please see remaining comments inline. > diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi > index 09aa06c..c4210dc 100644 > --- a/arch/arm/boot/dts/exynos5420.dtsi > +++ b/arch/arm/boot/dts/exynos5420.dtsi > @@ -31,6 +31,8 @@ > i2c1 = &i2c_1; > i2c2 = &i2c_2; > i2c3 = &i2c_3; > + gsc0 = &gsc_0; > + gsc1 = &gsc_1; > }; > > cpus { > @@ -310,4 +312,22 @@ > clocks = <&clock 431>, <&clock 143>; > clock-names = "mixer", "sclk_hdmi"; > }; > + > + gsc_0: g-scaler@13e00000 { coding style: There should be just one space after the label. coding style 2: Please name the node simply video-scaler. Node name is not supposed to tell what exactly the device is. The name should be mostly used for readability, so a person reading the sources can find that this node represents some video scaler and then by looking at compatible value he would be able to state that it is an GScaler block compatible with this on Exynos 5 SoCs. Best regards, Tomasz -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi index 09aa06c..c4210dc 100644 --- a/arch/arm/boot/dts/exynos5420.dtsi +++ b/arch/arm/boot/dts/exynos5420.dtsi @@ -31,6 +31,8 @@ i2c1 = &i2c_1; i2c2 = &i2c_2; i2c3 = &i2c_3; + gsc0 = &gsc_0; + gsc1 = &gsc_1; }; cpus { @@ -310,4 +312,22 @@ clocks = <&clock 431>, <&clock 143>; clock-names = "mixer", "sclk_hdmi"; }; + + gsc_0: g-scaler@13e00000 { + compatible = "samsung,exynos5-gsc"; + reg = <0x13e00000 0x1000>; + interrupts = <0 85 0>; + clocks = <&clock 465>; + clock-names = "gscl"; + samsung,power-domain = <&gsc_pd>; + }; + + gsc_1: g-scaler@13e10000 { + compatible = "samsung,exynos5-gsc"; + reg = <0x13e10000 0x1000>; + interrupts = <0 86 0>; + clocks = <&clock 466>; + clock-names = "gscl"; + samsung,power-domain = <&gsc_pd>; + }; };
Adds G-Scaler devices to the DT device list Signed-off-by: Leela Krishna Amudala <l.krishna@samsung.com> --- arch/arm/boot/dts/exynos5420.dtsi | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+)