[PATCH] ARM: imx6q: support ptp and rmii clock from pad
Shawn Guo
shawn.guo at linaro.org
Mon Feb 10 01:03:23 EST 2014
On Fri, Feb 07, 2014 at 11:45:18AM +0100, Philippe De Muyter wrote:
> Thanks, Shawn
>
> your patch works perfectly. I can now use the same kernel on my sabresd
> board (RGMII, enet_ref_clock generated by the imx6q) and my custom board
> (RMII, clock generated by the PHY).
That's great.
>
> some comments. though :
>
> the patch did not apply cleanly in clk-imx6q.c on v3.13
The patch targets v3.14-rc1, and if you need it for v3.13 you will need
to back port it.
>
> don't forget to complete Documentation/devicetree/bindings/
It requires change on device tree sources but not bindings. Which part
of Documentation/devicetree/bindings/ do you think we need to update?
Shawn
>
> But, again, many thanks for the support
>
> Philippe
>
>
> On Thu, Feb 06, 2014 at 01:53:38PM +0800, Shawn Guo wrote:
> > On imx6qdl, the ENET RMII and PTP clock can come from either internal
> > ANATOP/CCM or external clock source through pad GPIO_16. But in case
> > of the external clock source, bit IOMUXC_GPR1[21] needs to be cleared.
> >
> > The patch adds the support for systems that use an external clock source
> > and distinguishes above two cases by checking if the PTP clock specified
> > in device tree is the one coming from the internal ANATOP/CCM.
> >
> > Signed-off-by: Shawn Guo <shawn.guo at linaro.org>
> > ---
> > arch/arm/mach-imx/clk-imx6q.c | 1 +
> > arch/arm/mach-imx/mach-imx6q.c | 36 +++++++++++++++++++++++++++++++++++-
> > 2 files changed, 36 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/mach-imx/clk-imx6q.c b/arch/arm/mach-imx/clk-imx6q.c
> > index dd4f1e3..45de2e5 100644
> > --- a/arch/arm/mach-imx/clk-imx6q.c
> > +++ b/arch/arm/mach-imx/clk-imx6q.c
> > @@ -437,6 +437,7 @@ static void __init imx6q_clocks_init(struct device_node *ccm_node)
> >
> > clk_register_clkdev(clk[gpt_ipg], "ipg", "imx-gpt.0");
> > clk_register_clkdev(clk[gpt_ipg_per], "per", "imx-gpt.0");
> > + clk_register_clkdev(clk[enet_ref], "enet_ref", NULL);
> >
> > if ((imx_get_soc_revision() != IMX_CHIP_REVISION_1_0) ||
> > cpu_is_imx6dl()) {
> > diff --git a/arch/arm/mach-imx/mach-imx6q.c b/arch/arm/mach-imx/mach-imx6q.c
> > index f9cbbf9..d131499b 100644
> > --- a/arch/arm/mach-imx/mach-imx6q.c
> > +++ b/arch/arm/mach-imx/mach-imx6q.c
> > @@ -182,16 +182,50 @@ static void __init imx6q_enet_phy_init(void)
> >
> > static void __init imx6q_1588_init(void)
> > {
> > + struct device_node *np;
> > + struct clk *ptp_clk;
> > + struct clk *enet_ref;
> > struct regmap *gpr;
> > + u32 clksel;
> > +
> > + np = of_find_compatible_node(NULL, NULL, "fsl,imx6q-fec");
> > + if (!np) {
> > + pr_warn("%s: failed to find fec node\n", __func__);
> > + return;
> > + }
> > +
> > + ptp_clk = of_clk_get(np, 2);
> > + if (IS_ERR(ptp_clk)) {
> > + pr_warn("%s: failed to get ptp clock\n", __func__);
> > + goto put_node;
> > + }
> > +
> > + enet_ref = clk_get_sys(NULL, "enet_ref");
> > + if (IS_ERR(enet_ref)) {
> > + pr_warn("%s: failed to get enet clock\n", __func__);
> > + goto put_ptp_clk;
> > + }
> >
> > + /*
> > + * If enet_ref from ANATOP/CCM is the PTP clock source, we need to
> > + * set bit IOMUXC_GPR1[21]. Or the PTP clock must be from pad
> > + * (external OSC), and we need to clear the bit.
> > + */
> > + clksel = ptp_clk == enet_ref ? IMX6Q_GPR1_ENET_CLK_SEL_ANATOP :
> > + IMX6Q_GPR1_ENET_CLK_SEL_PAD;
> > gpr = syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr");
> > if (!IS_ERR(gpr))
> > regmap_update_bits(gpr, IOMUXC_GPR1,
> > IMX6Q_GPR1_ENET_CLK_SEL_MASK,
> > - IMX6Q_GPR1_ENET_CLK_SEL_ANATOP);
> > + clksel);
> > else
> > pr_err("failed to find fsl,imx6q-iomux-gpr regmap\n");
> >
> > + clk_put(enet_ref);
> > +put_ptp_clk:
> > + clk_put(ptp_clk);
> > +put_node:
> > + of_node_put(np);
> > }
> >
> > static void __init imx6q_init_machine(void)
> > --
> > 1.7.9.5
> >
>
> --
> Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles
More information about the linux-arm-kernel
mailing list