Regards,
Igal Liberman.
> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Wednesday, April 15, 2015 8:36 PM
> To: Liberman Igal-B31950
> Cc: devicet...@vger.kernel.org; linuxppc-dev@lists.ozlabs.org
> Subject: Re: [v3] dt/bindings: qoriq-clock: Add binding for FMan clock mux
>
> On Tue, 2015-04-14 at 13:56 +0300, Igal.Liberman wrote:
> > From: Igal Liberman <igal.liber...@freescale.com>
> >
> > v3: Addressed feedback from Scott:
> > - Removed clock specifier description.
> >
> > v2: Addressed feedback from Scott:
> > - Moved the "fman-clk-mux" clock provider details
> > under "clocks" property.
> >
> > Signed-off-by: Igal Liberman <igal.liber...@freescale.com>
> > ---
> > .../devicetree/bindings/clock/qoriq-clock.txt | 17 +++++++++++++++--
> > 1 file changed, 15 insertions(+), 2 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > index b0d7b73..2bb3b38 100644
> > --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > @@ -65,9 +65,10 @@ Required properties:
> > It takes parent's clock-frequency as its clock.
> > * "fsl,qoriq-platform-pll-1.0" for the platform PLL clock (v1.0)
> > * "fsl,qoriq-platform-pll-2.0" for the platform PLL clock (v2.0)
> > + * "fsl,fman-clk-mux" for the Frame Manager clock.
> > - #clock-cells: From common clock binding. The number of cells in a
> > - clock-specifier. Should be <0> for "fsl,qoriq-sysclk-[1,2].0"
> > - clocks, or <1> for "fsl,qoriq-core-pll-[1,2].0" clocks.
> > + clock-specifier. Should be <0> for "fsl,qoriq-sysclk-[1,2].0" and
> > + "fsl,fman-clk-mux" clocks or <1> for "fsl,qoriq-core-pll-[1,2].0".
> > For "fsl,qoriq-core-pll-1.0" clocks, the single
> > clock-specifier cell may take the following values:
> > * 0 - equal to the PLL frequency
> > @@ -145,6 +146,18 @@ Example for clock block and clock provider:
> > clocks = <&sysclk>;
> > clock-output-names = "platform-pll", "platform-pll-
> div2";
> > };
> > +
> > + fm0clk: fm0-clk-mux {
> > + #clock-cells = <0>;
> > + reg = <0x10 4>
> > + compatible = "fsl,fman-clk-mux";
> > + clocks = <&pll0 0>, <&pll0 1>, <&pll0 2>, <&pll0 3>,
> > + <&platform_pll 0>, <&pll1 1>, <&pll1 2>;
> > + clock-names = "pll0", "pll0-div2", "pll0-div3",
> > + "pll0-div4", "platform-pll", "pll1-div2",
> > + "pll1-div3";
> > + clock-output-names = "fm0-clk";
> > + };
> > };
> > };
> >
>
> I don't see this register in the manuals for older DPAA chips, such as
> p4080 or p3041. Is it present but undocumented? Should I be looking
> somewhere other than "Clocking Memory Map"?
>
It's available only in part of the new chips (T4, T2, B4).
In T1024/T1040 there's only one option for FMan clock so this register is not
available.
> -Scott
>
Igal.
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
