[media] exynos4-is: Correct clock properties description at the DT binding documentation
authorSylwester Nawrocki <s.nawrocki@samsung.com>
Tue, 2 Apr 2013 09:45:59 +0000 (06:45 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Thu, 4 Apr 2013 22:56:08 +0000 (19:56 -0300)
The 'camera' DT node needs to have sclk_cam0/1 and pxl_async0/1 clocks
specified, while 'fimc' nodes should have only "fimc" and "sclk_fimc".
"mux" and "parent" are leftovers from early versions of patches adding
DT support, when the IP bus clock parent clock was being set by the
driver. A better solution is needed to have e.g. clocks driver setting
all required parent clocks, before clock consumers start using the
clocks. Currently this binding doesn't describe parent clocks setup,
it needs to be specified and handled somewhere else.

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Documentation/devicetree/bindings/media/samsung-fimc.txt

index 2a63ddd23d164036832e72cebbd9ee02c6ad91eb..51c776b7f7a311339432e03e1cdf52bf4df12863 100644 (file)
@@ -17,9 +17,9 @@ Required properties:
 
 - compatible   : must be "samsung,fimc", "simple-bus"
 - clocks       : list of clock specifiers, corresponding to entries in
-                 clock-names property;
-- clock-names  : must contain "fimc", "sclk_fimc" entries, matching entries
-                 in the clocks property.
+                 the clock-names property;
+- clock-names  : must contain "sclk_cam0", "sclk_cam1", "pxl_async0",
+                 "pxl_async1" entries, matching entries in the clocks property.
 
 The pinctrl bindings defined in ../pinctrl/pinctrl-bindings.txt must be used
 to define a required pinctrl state named "default" and optional pinctrl states:
@@ -32,7 +32,6 @@ way around.
 
 The 'camera' node must include at least one 'fimc' child node.
 
-
 'fimc' device nodes
 -------------------
 
@@ -44,8 +43,7 @@ Required properties:
 - interrupts: should contain FIMC interrupt;
 - clocks: list of clock specifiers, must contain an entry for each required
   entry in clock-names;
-- clock-names: must include "fimc", "sclk_fimc", "mux" entries and optionally
-  "parent" entry.
+- clock-names: must contain "fimc", "sclk_fimc" entries.
 - samsung,pix-limits: an array of maximum supported image sizes in pixels, for
   details refer to Table 2-1 in the S5PV210 SoC User Manual; The meaning of
   each cell is as follows: