[PATCH] ARM: dts: Exynos5420: Add dt support for gscaler

2013-10-16 Thread Leela Krishna Amudala 
Adds G-Scaler devices to the DT device list

Signed-off-by: Leela Krishna Amudala l.kris...@samsung.com
---
 arch/arm/boot/dts/exynos5420.dtsi |   20 
 1 file changed, 20 insertions(+)

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@13e0 {
+   compatible = samsung,exynos5-gsc;
+   reg = 0x13e0 0x1000;
+   interrupts = 0 85 0;
+   clocks = clock 465;
+   clock-names = gscl;
+   samsung,power-domain = gsc_pd;
+   };
+
+   gsc_1:  g-scaler@13e1 {
+   compatible = samsung,exynos5-gsc;
+   reg = 0x13e1 0x1000;
+   interrupts = 0 86 0;
+   clocks = clock 466;
+   clock-names = gscl;
+   samsung,power-domain = gsc_pd;
+   };
 };
-- 
1.7.9.5

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] ARM: dts: Exynos5420: Add dt support for gscaler

2013-10-16 Thread Tomasz Figa 
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.kris...@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@13e0 {

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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html