Hi Rob,

On 19 June 2016 at 17:04, Rob Herring [off-list ref] wrote:

On Thu, Jun 16, 2016 at 11:20:22AM +0200, Gabriel Fernandez wrote:

quoted

This patch reworks the clock binding to avoid too much detail in DT.
Now we have only compatible string per type of clock
(remark from Rob https://lkml.org/lkml/2016/5/25/492)

I have no idea what the clock trees and clock controller in these chips
look like, so it's hard to say if the changes here are good. It still
looks like things are somewhat fine grained clocks in DT. I'll leave
it up to the platform maintainers to decide...

quoted

Signed-off-by: Gabriel Fernandez <redacted>
---
 .../devicetree/bindings/clock/st/st,clkgen-mux.txt |  2 +-
 .../devicetree/bindings/clock/st/st,clkgen-pll.txt | 11 ++--
 .../devicetree/bindings/clock/st/st,clkgen.txt     |  2 +-
 .../devicetree/bindings/clock/st/st,quadfs.txt     |  6 +--
 drivers/clk/st/clkgen-fsyn.c                       | 41 ++++++--------
 drivers/clk/st/clkgen-mux.c                        | 28 ++++------
 drivers/clk/st/clkgen-pll.c                        | 62 ++++++++++------------
 7 files changed, 65 insertions(+), 87 deletions(-)

diff --git a/Documentation/devicetree/bindings/clock/st/st,clkgen-mux.txt b/Documentation/devicetree/bindings/clock/st/st,clkgen-mux.txt
index 4d277d6..9a46cb1d7 100644
--- a/Documentation/devicetree/bindings/clock/st/st,clkgen-mux.txt
+++ b/Documentation/devicetree/bindings/clock/st/st,clkgen-mux.txt

@@ -10,7 +10,7 @@ This binding uses the common clock binding[1].
 Required properties:

 - compatible : shall be:
-     "st,stih407-clkgen-a9-mux",     "st,clkgen-mux"
+     "st,stih407-clkgen-a9-mux"

 - #clock-cells : from common clock binding; shall be set to 0.

diff --git a/Documentation/devicetree/bindings/clock/st/st,clkgen-pll.txt b/Documentation/devicetree/bindings/clock/st/st,clkgen-pll.txt
index c9fd674..be0b043 100644
--- a/Documentation/devicetree/bindings/clock/st/st,clkgen-pll.txt
+++ b/Documentation/devicetree/bindings/clock/st/st,clkgen-pll.txt

@@ -9,11 +9,10 @@ Base address is located to the parent node. See clock binding[2]
 Required properties:

 - compatible : shall be:
-     "st,stih407-plls-c32-a0",       "st,clkgen-plls-c32"
-     "st,stih407-plls-c32-a9",       "st,clkgen-plls-c32"
-     "sst,plls-c32-cx_0",            "st,clkgen-plls-c32"
-     "sst,plls-c32-cx_1",            "st,clkgen-plls-c32"
-     "st,stih418-plls-c28-a9",       "st,clkgen-plls-c32"

quoted

+     "st,clkgen-pll0"
+     "st,clkgen-pll0"

Repeated. Supposed to be 0 and 1? This seems a bit generic, too.

Yes you are right, it's 0 and 1.
I wait remarks from Mike or Stephen before send a V3.

Thanks Rob

BR
Gabriel

quoted

+     "st,stih407-clkgen-plla9"
+     "st,stih418-clkgen-plla9"