Hi, Shawn,
On Tue, Mar 8, 2011 at 12:22 AM, Shawn Guo <shawn....@linaro.org> wrote:
> The patch is to add all gpt, uart related dt clock nodes for babbage.
> It sticks to the clock name used in clock-mx51-mx53.c, so that
> everything gets consistent to Reference Manual. For example, the
> numbering in clock name usually starts from 1, while 'reg' property
> numbering starts from 0 to easy clock binding.
>
> Besides the generally used clock bindings, the following properties
> are proposed in this patch.
>
> * clock-alias
> Like clock-outputs to reflect cl->dev_id, property clock-alias is
> defined to reflect cl->con_id.
>
> * clock-depend
> The mxc 'struct clk' has the member 'secondary' to refer to the clock
> that the 'clk' has dependency on. This 'secondary' clock needs to be
> on whenever the 'clk' is set to on. This clock-depend property is
> defined to reflect this 'secondary' clock.
>
> Signed-off-by: Shawn Guo <shawn....@linaro.org>
> ---
> arch/arm/boot/dts/babbage.dts | 162
> +++++++++++++++++++++++++++++++++++++++--
> 1 files changed, 156 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm/boot/dts/babbage.dts b/arch/arm/boot/dts/babbage.dts
> index 46a3071..1774cec 100644
> --- a/arch/arm/boot/dts/babbage.dts
> +++ b/arch/arm/boot/dts/babbage.dts
> @@ -35,19 +35,169 @@
> #address-cells = <1>;
> #size-cells = <0>;
>
> - uart0_clk: uart@0 {
> + ckil_clk: clkil {
> + compatible = "fixed-clock";
> + #frequency-cells = <1>;
> + clock-outputs = "clil";
> + clock-frequency = <32768>;
> + };
> +
> + ckih_clk: ckih {
> + compatible = "fixed-clock";
> + #frequency-cells = <1>;
> + clock-outputs = "ckih";
> + clock-frequency = <22579200>;
> + };
> +
> + osc_clk: soc {
> + compatible = "fixed-clock";
> + #frequency-cells = <1>;
> + clock-outputs = "osc";
> + clock-frequency = <24000000>;
> + };
> +
> + pll1_main_clk: pll1_main {
> + compatible = "clock";
> + reg = <0>;
> + clock-outputs = "pll1_main";
> + clock-source = <&osc_clk>;
> + };
> +
> + pll1_sw_clk: pll_switch@0 {
> + compatible = "clock";
> + reg = <0>;
> + clock-outputs = "pll1_sw";
> + clock-source = <&pll1_main_clk>;
> + };
> +
> + pll2_sw_clk: pll_switch@1 {
> + compatible = "clock";
> + reg = <1>;
> + clock-outputs = "pll2_sw";
> + clock-source = <&osc_clk>;
> + };
>
It seems that you mis-used the reg property, it need fixed globally.
BR,
Jason
>
> --
> 1.7.1
>
>
> _______________________________________________
> linaro-dev mailing list
> linaro-dev@lists.linaro.org
> http://lists.linaro.org/mailman/listinfo/linaro-dev
>
_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev
