Hi Shaik,
On 07/16/2012 07:33 AM, Shaik Ameer Basha wrote:
+++ b/Documentation/devicetree/bindings/media/exynos5-gsc.txt
@@ -0,0 +1,16 @@
+* Samsung Exynos5 Gscaler device
+
+Gscaler is used for scaling and color space conversion on EXYNOS5 SoCs.
+
+Required properties:
+- compatible: should be "samsung,exynos-gsc"
Shouldn't there rather be a specific "compatible" property for each SoC,
e.g.
- compatible: should be one of:
"samsung5250,exynos-gsc",
"samsung5450,exynos-gsc",
etc. ?
+- reg: should contian Gsaler physical address location and length.
contian -> contain, Gsaler -> Gscaler
+- interrupts: should contain Gscaler interrupt number
+
+Example:
+
+gsc_0: gsc@0x13e00000 {
+ compatible = "samsung,exynos-gsc";
compatible = "samsung5250,exynos-gsc";
+ reg =<0x13e00000 0x1000>;
+ interrupts =<0 85 0>;
+};
The aliases are still undocumented. How about adding something
along the lines of:
"Each Gscaler node should have a numbered alias in the aliases node,
in the form of gscN, N = 0...3."
?
diff --git a/arch/arm/boot/dts/exynos5250.dtsi
b/arch/arm/boot/dts/exynos5250.dtsi
index 4272b29..b945c00 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -23,6 +23,13 @@
compatible = "samsung,exynos5250";
interrupt-parent =<&gic>;
+ aliases {
+ gsc0 =&gsc_0;
+ gsc1 =&gsc_1;
+ gsc2 =&gsc_2;
+ gsc3 =&gsc_3;
+ };
+
gic:interrupt-controller@10481000 {
compatible = "arm,cortex-a9-gic";
#interrupt-cells =<3>;
--
Regards,
Sylwester
--
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
