[PATCH v4 2/4] clk: imx8mq: Add a clock driver for the imx8mq

Peng Fan (OSS) peng.fan at oss.nxp.com
Mon Mar 21 02:32:43 CET 2022



On 2022/3/15 23:35, Heiko Thiery wrote:
> Hi Angus and all,
> 
> 
> 
> 
> Am Di., 15. März 2022 um 14:09 Uhr schrieb Angus Ainslie <angus at akkea.ca 
> <mailto:angus at akkea.ca>>:
> 
>     This is a DM clock driver based off the imx8mm u-boot driver and the
>     linux
>     kernel driver.
> 
>     All of the PLLs and clocks are initialized so the subsystems below are
>     functional and tested.
> 
>     1) USB host and peripheral
>     2) ECSPI
>     3) UART
>     4) I2C all busses
>     5) USDHC for eMMC support
>     6) USB storage
>     7) GPIO
>     8) DRAM
> 
>     Signed-off-by: Angus Ainslie <angus at akkea.ca <mailto:angus at akkea.ca>>
>     ---
>       drivers/clk/imx/Kconfig      |  16 +
>       drivers/clk/imx/Makefile     |   2 +
>       drivers/clk/imx/clk-imx8mq.c | 575 +++++++++++++++++++++++++++++++++++
>       3 files changed, 593 insertions(+)
>       create mode 100644 drivers/clk/imx/clk-imx8mq.c
> 
>     diff --git a/drivers/clk/imx/Kconfig b/drivers/clk/imx/Kconfig
>     index cdd348020b0..06d8c1a5dd3 100644
>     --- a/drivers/clk/imx/Kconfig
>     +++ b/drivers/clk/imx/Kconfig
>     @@ -71,6 +71,22 @@ config CLK_IMX8MP
>              help
>                This enables support clock driver for i.MX8MP platforms.
> 
>     +config SPL_CLK_IMX8MQ
>     +       bool "SPL clock support for i.MX8MQ"
>     +       depends on ARCH_IMX8M && SPL
>     +       select SPL_CLK
>     +       select SPL_CLK_CCF
>     +       help
>     +         This enables SPL DM/DTS support for clock driver in i.MX8MQ
>     +
>     +config CLK_IMX8MQ
>     +       bool "Clock support for i.MX8MQ"
>     +       depends on ARCH_IMX8M
>     +       select CLK
>     +       select CLK_CCF
>     +       help
>     +         This enables support clock driver for i.MX8MQ platforms.
>     +
>       config SPL_CLK_IMXRT1020
>              bool "SPL clock support for i.MXRT1020"
>              depends on ARCH_IMXRT && SPL
>     diff --git a/drivers/clk/imx/Makefile b/drivers/clk/imx/Makefile
>     index 01bbbdf3aea..c5766901f2b 100644
>     --- a/drivers/clk/imx/Makefile
>     +++ b/drivers/clk/imx/Makefile
>     @@ -16,6 +16,8 @@ obj-$(CONFIG_$(SPL_TPL_)CLK_IMX8MN) +=
>     clk-imx8mn.o clk-pll14xx.o \
>                                      clk-composite-8m.o
>       obj-$(CONFIG_$(SPL_TPL_)CLK_IMX8MP) += clk-imx8mp.o clk-pll14xx.o \
>                                      clk-composite-8m.o
>     +obj-$(CONFIG_$(SPL_TPL_)CLK_IMX8MQ) += clk-imx8mq.o clk-pll14xx.o \
>     +                               clk-composite-8m.o
> 
>       obj-$(CONFIG_$(SPL_TPL_)CLK_IMXRT1020) += clk-imxrt1020.o
>       obj-$(CONFIG_$(SPL_TPL_)CLK_IMXRT1050) += clk-imxrt1050.o
>     diff --git a/drivers/clk/imx/clk-imx8mq.c b/drivers/clk/imx/clk-imx8mq.c
>     new file mode 100644
>     index 00000000000..0aea417a29b
>     --- /dev/null
>     +++ b/drivers/clk/imx/clk-imx8mq.c
>     @@ -0,0 +1,575 @@
>     +// SPDX-License-Identifier: GPL-2.0
>     +/*
>     + * Copyright 2019 NXP
>     + * Copyright 2022 Purism
>     + * Peng Fan <peng.fan at nxp.com <mailto:peng.fan at nxp.com>>
>     + */
>     +
>     +#include <common.h>
>     +#include <clk.h>
>     +#include <clk-uclass.h>
>     +#include <dm.h>
>     +#include <log.h>
>     +#include <asm/arch/clock.h>
>     +#include <asm/arch/imx-regs.h>
>     +#include <dt-bindings/clock/imx8mq-clock.h>
>     +
>     +#include "clk.h"
>     +
>     +#define PLL_1416X_RATE(_rate, _m, _p, _s)              \
>     +       {                                               \
>     +               .rate   =       (_rate),                \
>     +               .mdiv   =       (_m),                   \
>     +               .pdiv   =       (_p),                   \
>     +               .sdiv   =       (_s),                   \
>     +       }
>     +
>     +#define PLL_1443X_RATE(_rate, _m, _p, _s, _k)          \
>     +       {                                               \
>     +               .rate   =       (_rate),                \
>     +               .mdiv   =       (_m),                   \
>     +               .pdiv   =       (_p),                   \
>     +               .sdiv   =       (_s),                   \
>     +               .kdiv   =       (_k),                   \
>     +       }
>     +
>     +static const struct imx_pll14xx_rate_table imx8mq_pll1416x_tbl[] = {
>     +       PLL_1416X_RATE(1800000000U, 225, 3, 0),
>     +       PLL_1416X_RATE(1600000000U, 200, 3, 0),
>     +       PLL_1416X_RATE(1200000000U, 300, 3, 1),
>     +       PLL_1416X_RATE(1000000000U, 250, 3, 1),
>     +       PLL_1416X_RATE(800000000U,  200, 3, 1),
>     +       PLL_1416X_RATE(750000000U,  250, 2, 2),
>     +       PLL_1416X_RATE(700000000U,  350, 3, 2),
>     +       PLL_1416X_RATE(600000000U,  300, 3, 2),
>     +};
>     +
>     +const struct imx_pll14xx_rate_table imx8mq_pll1443x_tbl[] = {
>     +       PLL_1443X_RATE(650000000U, 325, 3, 2, 0),
>     +       PLL_1443X_RATE(594000000U, 198, 2, 2, 0),
>     +       PLL_1443X_RATE(393216000U, 262, 2, 3, 9437),
>     +       PLL_1443X_RATE(361267200U, 361, 3, 3, 17511),
>     +};
>     +
>     +static struct imx_pll14xx_clk imx8mq_1416x_pll __initdata = {
>     +               .type = PLL_1416X,
>     +               .rate_table = imx8mq_pll1416x_tbl,
>     +               .rate_count = ARRAY_SIZE(imx8mq_pll1416x_tbl),
>     +};
>     +
>     +static struct imx_pll14xx_clk imx8mq_1443x_pll __initdata = {
>     +               .type = PLL_1443X,
>     +               .rate_table = imx8mq_pll1443x_tbl,
>     +               .rate_count = ARRAY_SIZE(imx8mq_pll1443x_tbl),
>     +};
>     +
>     +static const char *pll_ref_sels[] = { "clock-osc-25m",
>     "clock-osc-27m", "clock-phy-27m", "dummy", };
>     +static const char *arm_pll_bypass_sels[] = {"arm_pll",
>     "arm_pll_ref_sel", };
>     +static const char *gpu_pll_bypass_sels[] = {"gpu_pll",
>     "gpu_pll_ref_sel", };
>     +static const char *vpu_pll_bypass_sels[] = {"vpu_pll",
>     "vpu_pll_ref_sel", };
>     +static const char *audio_pll1_bypass_sels[] = {"audio_pll1",
>     "audio_pll1_ref_sel", };
>     +static const char *audio_pll2_bypass_sels[] = {"audio_pll2",
>     "audio_pll2_ref_sel", };
>     +static const char *video_pll1_bypass_sels[] = {"video_pll1",
>     "video_pll1_ref_sel", };
>     +
>     +static const char *imx8mq_a53_core_sels[] = {"arm_a53_div",
>     "arm_pll_out", };
>     +static const char *imx8mq_a53_sels[] = {"clock-osc-25m",
>     "arm_pll_out", "sys_pll2_500m",
>     +                                       "sys_pll2_1000m",
>     "sys_pll1_800m", "sys_pll1_400m",
>     +                                       "audio_pll1_out",
>     "sys_pll3_out", };
>     +
>     +static const char *imx8mq_ahb_sels[] = {"clock-osc-25m",
>     "sys_pll1_133m", "sys_pll1_800m",
>     +                                       "sys_pll1_400m",
>     "sys_pll2_125m", "sys_pll3_out",
>     +                                       "audio_pll1_out",
>     "video_pll1_out", };
>     +
>     +static const char *imx8mq_enet_axi_sels[] = {"clock-osc-25m",
>     "sys_pll1_266m", "sys_pll1_800m",
>     +                                            "sys_pll2_250m",
>     "sys_pll2_200m", "audio_pll1_out",
>     +                                            "video_pll1_out",
>     "sys_pll3_out", };
>     +
>     +static const char *imx8mq_enet_ref_sels[] = {"clock-osc-25m",
>     "sys_pll2_125m", "sys_pll2_50m",
>     +                                            "sys_pll2_100m",
>     "sys_pll1_160m", "audio_pll1_out",
>     +                                            "video_pll1_out",
>     "clk_ext4", };
>     +
>     +static const char *imx8mq_enet_timer_sels[] = {"clock-osc-25m",
>     "sys_pll2_100m", "audio_pll1_out",
>     +                                              "clk_ext1",
>     "clk_ext2", "clk_ext3", "clk_ext4",
>     +                                              "video_pll1_out", };
>     +
>     +static const char *imx8mq_enet_phy_sels[] = {"clock-osc-25m",
>     "sys_pll2_50m", "sys_pll2_125m",
>     +                                            "sys_pll2_200m",
>     "sys_pll2_500m", "video_pll1_out",
>     +                                            "audio_pll2_out", };
>     +
>     +static const char *imx8mq_nand_usdhc_sels[] = {"clock-osc-25m",
>     "sys_pll1_266m", "sys_pll1_800m",
>     +                                              "sys_pll2_200m",
>     "sys_pll1_133m", "sys_pll3_out",
>     +                                              "sys_pll2_250m",
>     "audio_pll1_out", };
>     +
>     +static const char *imx8mq_usb_bus_sels[] = {"clock-osc-25m",
>     "sys_pll2_500m", "sys_pll1_800m",
>     +                                           "sys_pll2_100m",
>     "sys_pll2_200m", "clk_ext2",
>     +                                           "clk_ext4",
>     "audio_pll2_out", };
>     +
>     +static const char *imx8mq_usdhc1_sels[] = {"clock-osc-25m",
>     "sys_pll1_400m", "sys_pll1_800m",
>     +                                          "sys_pll2_500m",
>     "sys_pll3_out", "sys_pll1_266m",
>     +                                          "audio_pll2_out",
>     "sys_pll1_100m", };
>     +
>     +static const char *imx8mq_usdhc2_sels[] = {"clock-osc-25m",
>     "sys_pll1_400m", "sys_pll1_800m",
>     +                                          "sys_pll2_500m",
>     "sys_pll3_out", "sys_pll1_266m",
>     +                                          "audio_pll2_out",
>     "sys_pll1_100m", };
>     +
>     +static const char *imx8mq_i2c1_sels[] = {"clock-osc-25m",
>     "sys_pll1_160m", "sys_pll2_50m",
>     +                                        "sys_pll3_out",
>     "audio_pll1_out", "video_pll1_out",
>     +                                        "audio_pll2_out",
>     "sys_pll1_133m", };
>     +
>     +static const char *imx8mq_i2c2_sels[] = {"clock-osc-25m",
>     "sys_pll1_160m", "sys_pll2_50m",
>     +                                        "sys_pll3_out",
>     "audio_pll1_out", "video_pll1_out",
>     +                                        "audio_pll2_out",
>     "sys_pll1_133m", };
>     +
>     +static const char *imx8mq_i2c3_sels[] = {"clock-osc-25m",
>     "sys_pll1_160m", "sys_pll2_50m",
>     +                                        "sys_pll3_out",
>     "audio_pll1_out", "video_pll1_out",
>     +                                        "audio_pll2_out",
>     "sys_pll1_133m", };
>     +
>     +static const char *imx8mq_i2c4_sels[] = {"clock-osc-25m",
>     "sys_pll1_160m", "sys_pll2_50m",
>     +                                        "sys_pll3_out",
>     "audio_pll1_out", "video_pll1_out",
>     +                                        "audio_pll2_out",
>     "sys_pll1_133m", };
>     +
>     +static const char *imx8mq_wdog_sels[] = {"clock-osc-25m",
>     "sys_pll1_133m", "sys_pll1_160m",
>     +                                        "vpu_pll_out",
>     "sys_pll2_125m", "sys_pll3_out",
>     +                                        "sys_pll1_80m",
>     "sys_pll2_166m", };
>     +
>     +static const char *imx8mq_qspi_sels[] = {"clock-osc-25m",
>     "sys_pll1_400m", "sys_pll2_333m",
>     +                                        "sys_pll2_500m",
>     "audio_pll2_out", "sys_pll1_266m",
>     +                                        "sys_pll3_out",
>     "sys_pll1_100m", };
>     +
>     +static const char *imx8mq_usb_core_sels[] = {"clock-osc-25m",
>     "sys_pll1_100m", "sys_pll1_40m",
>     +                                            "sys_pll2_100m",
>     "sys_pll2_200m", "clk_ext2",
>     +                                            "clk_ext3",
>     "audio_pll2_out", };
>     +
>     +static const char *imx8mq_usb_phy_sels[] = {"clock-osc-25m",
>     "sys_pll1_100m", "sys_pll1_40m",
>     +                                           "sys_pll2_100m",
>     "sys_pll2_200m", "clk_ext2",
>     +                                           "clk_ext3",
>     "audio_pll2_out", };
>     +
>     +static const char *imx8mq_ecspi1_sels[] = {"clock-osc-25m",
>     "sys_pll2_200m", "sys_pll1_40m",
>     +                                          "sys_pll1_160m",
>     "sys_pll1_800m", "sys_pll3_out",
>     +                                          "sys_pll2_250m",
>     "audio_pll2_out", };
>     +
>     +static const char *imx8mq_ecspi2_sels[] = {"clock-osc-25m",
>     "sys_pll2_200m", "sys_pll1_40m",
>     +                                          "sys_pll1_160m",
>     "sys_pll1_800m", "sys_pll3_out",
>     +                                          "sys_pll2_250m",
>     "audio_pll2_out", };
>     +
>     +static const char *imx8mq_ecspi3_sels[] = {"clock-osc-25m",
>     "sys_pll2_200m", "sys_pll1_40m",
>     +                                          "sys_pll1_160m",
>     "sys_pll1_800m", "sys_pll3_out",
>     +                                          "sys_pll2_250m",
>     "audio_pll2_out", };
>     +
>     +static const char *pllout_monitor_sels[] = {"clock-osc-25m",
>     "clock-osc-27m", "clock-phy-27m",
>     +                                           "dummy", "clock-ckil",
>     "audio_pll1_out_monitor",
>     +                                         
>       "audio_pll2_out_monitor", "gpu_pll_out_monitor",
>     +                                           "vpu_pll_out_monitor",
>     "video_pll1_out_monitor",
>     +                                           "arm_pll_out_monitor",
>     "sys_pll1_out_monitor",
>     +                                           "sys_pll2_out_monitor",
>     "sys_pll3_out_monitor",
>     +                                           "dummy",
>     "dram_pll_out_monitor", };
>     +
>     +static ulong imx8mq_clk_get_rate(struct clk *clk)
>     +{
>     +       struct clk *c;
>     +       int ret;
>     +
>     +       debug("%s(#%lu)\n", __func__, clk->id);
>     +
>     +       ret = clk_get_by_id(clk->id, &c);
>     +       if (ret)
>     +               return ret;
>     +
>     +       return clk_get_rate(c);
>     +}
>     +
>     +static ulong imx8mq_clk_set_rate(struct clk *clk, unsigned long rate)
>     +{
>     +       struct clk *c;
>     +       int ret;
>     +
>     +       debug("%s(#%lu), rate: %lu\n", __func__, clk->id, rate);
>     +
>     +       ret = clk_get_by_id(clk->id, &c);
>     +       if (ret)
>     +               return ret;
>     +
>     +       return clk_set_rate(c, rate);
>     +}
>     +
>     +static int __imx8mq_clk_enable(struct clk *clk, bool enable)
>     +{
>     +       struct clk *c;
>     +       int ret;
>     +
>     +       debug("%s(#%lu) en: %d\n", __func__, clk->id, enable);
>     +
>     +       ret = clk_get_by_id(clk->id, &c);
>     +       if (ret) {
>     +               debug("%s: clk_get_by_id failed\n", __func__);
>     +               return ret;
>     +       }
>     +
>     +       if (enable)
>     +               ret = clk_enable(c);
>     +       else
>     +               ret = clk_disable(c);
>     +
>     +       return ret;
>     +}
>     +
>     +static int imx8mq_clk_disable(struct clk *clk)
>     +{
>     +       return __imx8mq_clk_enable(clk, 0);
>     +}
>     +
>     +static int imx8mq_clk_enable(struct clk *clk)
>     +{
>     +       return __imx8mq_clk_enable(clk, 1);
>     +}
>     +
>     +static int imx8mq_clk_set_parent(struct clk *clk, struct clk *parent)
>     +{
>     +       struct clk *c, *cp;
>     +       int ret;
>     +
>     +       debug("%s(#%lu), parent: %lu\n", __func__, clk->id, parent->id);
>     +
>     +       ret = clk_get_by_id(clk->id, &c);
>     +       if (ret)
>     +               return ret;
>     +
>     +       ret = clk_get_by_id(parent->id, &cp);
>     +       if (ret)
>     +               return ret;
>     +
>     +       ret = clk_set_parent(c, cp);
>     +       c->dev->parent = cp->dev;
>     +
>     +       return ret;
>     +}
>     +
>     +static struct clk_ops imx8mq_clk_ops = {
>     +       .set_rate = imx8mq_clk_set_rate,
>     +       .get_rate = imx8mq_clk_get_rate,
>     +       .enable = imx8mq_clk_enable,
>     +       .disable = imx8mq_clk_disable,
>     +       .set_parent = imx8mq_clk_set_parent,
>     +};
>     +
>     +static int imx8mq_clk_probe(struct udevice *dev)
>     +{
>     +       void __iomem *base;
>     +
>     +       base = (void *)ANATOP_BASE_ADDR;
>     +
>     +       clk_dm(IMX8MQ_CLK_32K, clk_register_fixed_rate(NULL, "ckil",
>     32768));
>     +
>     +       clk_dm(IMX8MQ_DRAM_PLL1_REF_SEL,
>     +              imx_clk_mux("dram_pll_ref_sel", base + 0x60, 0, 2,
>     +                          pll_ref_sels, ARRAY_SIZE(pll_ref_sels)));
>     +       clk_dm(IMX8MQ_ARM_PLL_REF_SEL,
>     +              imx_clk_mux("arm_pll_ref_sel", base + 0x28, 0, 2,
>     +                          pll_ref_sels, ARRAY_SIZE(pll_ref_sels)));
>     +       clk_dm(IMX8MQ_GPU_PLL_REF_SEL,
>     +              imx_clk_mux("gpu_pll_ref_sel", base + 0x18, 0, 2,
>     +                          pll_ref_sels, ARRAY_SIZE(pll_ref_sels)));
>     +       clk_dm(IMX8MQ_VPU_PLL_REF_SEL,
>     +              imx_clk_mux("vpu_pll_ref_sel", base + 0x20, 0, 2,
>     +                          pll_ref_sels, ARRAY_SIZE(pll_ref_sels)));
>     +       clk_dm(IMX8MQ_SYS3_PLL1_REF_SEL,
>     +              imx_clk_mux("sys3_pll_ref_sel", base + 0x48, 0, 2,
>     +                          pll_ref_sels, ARRAY_SIZE(pll_ref_sels)));
>     +       clk_dm(IMX8MQ_AUDIO_PLL1_REF_SEL,
>     +              imx_clk_mux("audio_pll1_ref_sel", base + 0x0, 0, 2,
>     +                          pll_ref_sels, ARRAY_SIZE(pll_ref_sels)));
>     +       clk_dm(IMX8MQ_AUDIO_PLL2_REF_SEL,
>     +              imx_clk_mux("audio_pll2_ref_sel", base + 0x8, 0, 2,
>     +                          pll_ref_sels, ARRAY_SIZE(pll_ref_sels)));
>     +       clk_dm(IMX8MQ_VIDEO_PLL1_REF_SEL,
>     +              imx_clk_mux("video_pll1_ref_sel", base + 0x10, 0, 2,
>     +                          pll_ref_sels, ARRAY_SIZE(pll_ref_sels)));
>     +
>     +       clk_dm(IMX8MQ_ARM_PLL,
>     +              imx_clk_pll14xx("arm_pll", "arm_pll_ref_sel",
>     +                              base + 0x28, &imx8mq_1416x_pll));
>     +       clk_dm(IMX8MQ_GPU_PLL,
>     +              imx_clk_pll14xx("gpu_pll", "gpu_pll_ref_sel",
>     +                              base + 0x18, &imx8mq_1416x_pll));
>     +       clk_dm(IMX8MQ_VPU_PLL,
>     +              imx_clk_pll14xx("vpu_pll", "vpu_pll_ref_sel",
>     +                              base + 0x20, &imx8mq_1416x_pll));
>     +
>     +       clk_dm(IMX8MQ_SYS1_PLL1,
>     +              clk_register_fixed_rate(NULL, "sys1_pll", 800000000));
>     +       clk_dm(IMX8MQ_SYS2_PLL1,
>     +              clk_register_fixed_rate(NULL, "sys2_pll", 1000000000));
>     +       clk_dm(IMX8MQ_SYS2_PLL1,
>     +              clk_register_fixed_rate(NULL, "sys3_pll", 1000000000));
>     +       clk_dm(IMX8MQ_AUDIO_PLL1,
>     +              imx_clk_pll14xx("audio_pll1", "audio_pll1_ref_sel",
>     +                              base + 0x0, &imx8mq_1443x_pll));
>     +       clk_dm(IMX8MQ_AUDIO_PLL2,
>     +              imx_clk_pll14xx("audio_pll2", "audio_pll2_ref_sel",
>     +                              base + 0x8, &imx8mq_1443x_pll));
>     +       clk_dm(IMX8MQ_VIDEO_PLL1,
>     +              imx_clk_pll14xx("video_pll1", "video_pll1_ref_sel",
>     +                              base + 0x10, &imx8mq_1443x_pll));
>     +
>     +       /* PLL bypass out */
>     +       clk_dm(IMX8MQ_ARM_PLL_BYPASS,
>     +              imx_clk_mux_flags("arm_pll_bypass", base + 0x28, 4, 1,
>     +                                arm_pll_bypass_sels,
>     +                                ARRAY_SIZE(arm_pll_bypass_sels),
>     +                                CLK_SET_RATE_PARENT));
>     +       clk_dm(IMX8MQ_GPU_PLL_BYPASS,
>     +              imx_clk_mux_flags("gpu_pll_bypass", base + 0x18, 4, 1,
>     +                                gpu_pll_bypass_sels,
>     +                                ARRAY_SIZE(gpu_pll_bypass_sels),
>     +                                CLK_SET_RATE_PARENT));
>     +       clk_dm(IMX8MQ_VPU_PLL_BYPASS,
>     +              imx_clk_mux_flags("vpu_pll_bypass", base + 0x20, 4, 1,
>     +                                vpu_pll_bypass_sels,
>     +                                ARRAY_SIZE(vpu_pll_bypass_sels),
>     +                                CLK_SET_RATE_PARENT));
>     +       clk_dm(IMX8MQ_AUDIO_PLL1_BYPASS,
>     +              imx_clk_mux_flags("audio_pll1_bypass", base + 0x0, 4, 1,
>     +                                audio_pll1_bypass_sels,
>     +                                ARRAY_SIZE(audio_pll1_bypass_sels),
>     +                                CLK_SET_RATE_PARENT));
>     +       clk_dm(IMX8MQ_AUDIO_PLL2_BYPASS,
>     +              imx_clk_mux_flags("audio_pll2_bypass", base + 0x8, 4, 1,
>     +                                audio_pll2_bypass_sels,
>     +                                ARRAY_SIZE(audio_pll2_bypass_sels),
>     +                                CLK_SET_RATE_PARENT));
>     +       clk_dm(IMX8MQ_VIDEO_PLL1_BYPASS,
>     +              imx_clk_mux_flags("video_pll1_bypass", base + 0x10, 4, 1,
>     +                                video_pll1_bypass_sels,
>     +                                ARRAY_SIZE(video_pll1_bypass_sels),
>     +                                CLK_SET_RATE_PARENT));
>     +
>     +       /* PLL out gate */
>     +       clk_dm(IMX8MQ_DRAM_PLL_OUT,
>     +              imx_clk_gate("dram_pll_out", "dram_pll_ref_sel",
>     +                           base + 0x60, 13));
>     +       clk_dm(IMX8MQ_ARM_PLL_OUT,
>     +              imx_clk_gate("arm_pll_out", "arm_pll_bypass",
>     +                           base + 0x28, 11));
>     +       clk_dm(IMX8MQ_GPU_PLL_OUT,
>     +              imx_clk_gate("gpu_pll_out", "gpu_pll_bypass",
>     +                           base + 0x18, 11));
>     +       clk_dm(IMX8MQ_VPU_PLL_OUT,
>     +              imx_clk_gate("vpu_pll_out", "vpu_pll_bypass",
>     +                           base + 0x20, 11));
>     +       clk_dm(IMX8MQ_AUDIO_PLL1_OUT,
>     +              imx_clk_gate("audio_pll1_out", "audio_pll1_bypass",
>     +                           base + 0x0, 11));
>     +       clk_dm(IMX8MQ_AUDIO_PLL2_OUT,
>     +              imx_clk_gate("audio_pll2_out", "audio_pll2_bypass",
>     +                           base + 0x8, 11));
>     +       clk_dm(IMX8MQ_VIDEO_PLL1_OUT,
>     +              imx_clk_gate("video_pll1_out", "video_pll1_bypass",
>     +                           base + 0x10, 11));
>     +
>     +       clk_dm(IMX8MQ_SYS1_PLL_OUT,
>     +              imx_clk_gate("sys_pll1_out", "sys1_pll",
>     +                           base + 0x30, 11));
>     +       clk_dm(IMX8MQ_SYS2_PLL_OUT,
>     +              imx_clk_gate("sys_pll2_out", "sys2_pll",
>     +                           base + 0x3c, 11));
>     +       clk_dm(IMX8MQ_SYS3_PLL_OUT,
>     +              imx_clk_gate("sys_pll3_out", "sys3_pll",
>     +                           base + 0x48, 11));
>     +
>     +       /* SYS PLL fixed output */
>     +       clk_dm(IMX8MQ_SYS1_PLL_40M,
>     +              imx_clk_fixed_factor("sys_pll1_40m", "sys_pll1_out",
>     1, 20));
>     +       clk_dm(IMX8MQ_SYS1_PLL_80M,
>     +              imx_clk_fixed_factor("sys_pll1_80m", "sys_pll1_out",
>     1, 10));
>     +       clk_dm(IMX8MQ_SYS1_PLL_100M,
>     +              imx_clk_fixed_factor("sys_pll1_100m", "sys_pll1_out",
>     1, 8));
>     +       clk_dm(IMX8MQ_SYS1_PLL_133M,
>     +              imx_clk_fixed_factor("sys_pll1_133m", "sys_pll1_out",
>     1, 6));
>     +       clk_dm(IMX8MQ_SYS1_PLL_160M,
>     +              imx_clk_fixed_factor("sys_pll1_160m", "sys_pll1_out",
>     1, 5));
>     +       clk_dm(IMX8MQ_SYS1_PLL_200M,
>     +              imx_clk_fixed_factor("sys_pll1_200m", "sys_pll1_out",
>     1, 4));
>     +       clk_dm(IMX8MQ_SYS1_PLL_266M,
>     +              imx_clk_fixed_factor("sys_pll1_266m", "sys_pll1_out",
>     1, 3));
>     +       clk_dm(IMX8MQ_SYS1_PLL_400M,
>     +              imx_clk_fixed_factor("sys_pll1_400m", "sys_pll1_out",
>     1, 2));
>     +       clk_dm(IMX8MQ_SYS1_PLL_800M,
>     +              imx_clk_fixed_factor("sys_pll1_800m", "sys_pll1_out",
>     1, 1));
>     +
>     +       clk_dm(IMX8MQ_SYS2_PLL_50M,
>     +              imx_clk_fixed_factor("sys_pll2_50m", "sys_pll2_out",
>     1, 20));
>     +       clk_dm(IMX8MQ_SYS2_PLL_100M,
>     +              imx_clk_fixed_factor("sys_pll2_100m", "sys_pll2_out",
>     1, 10));
>     +       clk_dm(IMX8MQ_SYS2_PLL_125M,
>     +              imx_clk_fixed_factor("sys_pll2_125m", "sys_pll2_out",
>     1, 8));
>     +       clk_dm(IMX8MQ_SYS2_PLL_166M,
>     +              imx_clk_fixed_factor("sys_pll2_166m", "sys_pll2_out",
>     1, 6));
>     +       clk_dm(IMX8MQ_SYS2_PLL_200M,
>     +              imx_clk_fixed_factor("sys_pll2_200m", "sys_pll2_out",
>     1, 5));
>     +       clk_dm(IMX8MQ_SYS2_PLL_250M,
>     +              imx_clk_fixed_factor("sys_pll2_250m", "sys_pll2_out",
>     1, 4));
>     +       clk_dm(IMX8MQ_SYS2_PLL_333M,
>     +              imx_clk_fixed_factor("sys_pll2_333m", "sys_pll2_out",
>     1, 3));
>     +       clk_dm(IMX8MQ_SYS2_PLL_500M,
>     +              imx_clk_fixed_factor("sys_pll2_500m", "sys_pll2_out",
>     1, 2));
>     +       clk_dm(IMX8MQ_SYS2_PLL_1000M,
>     +              imx_clk_fixed_factor("sys_pll2_1000m",
>     "sys_pll2_out", 1, 1));
>     +
>     +       clk_dm(IMX8MQ_CLK_MON_AUDIO_PLL1_DIV,
>     +              imx_clk_divider("audio_pll1_out_monitor",
>     "audio_pll1_bypass", base + 0x78, 0, 3));
>     +       clk_dm(IMX8MQ_CLK_MON_AUDIO_PLL2_DIV,
>     +              imx_clk_divider("audio_pll2_out_monitor",
>     "audio_pll2_bypass", base + 0x78, 4, 3));
>     +       clk_dm(IMX8MQ_CLK_MON_VIDEO_PLL1_DIV,
>     +              imx_clk_divider("video_pll1_out_monitor",
>     "video_pll1_bypass", base + 0x78, 8, 3));
>     +       clk_dm(IMX8MQ_CLK_MON_GPU_PLL_DIV,
>     +              imx_clk_divider("gpu_pll_out_monitor",
>     "gpu_pll_bypass", base + 0x78, 12, 3));
>     +       clk_dm(IMX8MQ_CLK_MON_VPU_PLL_DIV,
>     +              imx_clk_divider("vpu_pll_out_monitor",
>     "vpu_pll_bypass", base + 0x78, 16, 3));
>     +       clk_dm(IMX8MQ_CLK_MON_ARM_PLL_DIV,
>     +              imx_clk_divider("arm_pll_out_monitor",
>     "arm_pll_bypass", base + 0x78, 20, 3));
>     +       clk_dm(IMX8MQ_CLK_MON_SYS_PLL1_DIV,
>     +              imx_clk_divider("sys_pll1_out_monitor",
>     "sys_pll1_out", base + 0x7c, 0, 3));
>     +       clk_dm(IMX8MQ_CLK_MON_SYS_PLL2_DIV,
>     +              imx_clk_divider("sys_pll2_out_monitor",
>     "sys_pll2_out", base + 0x7c, 4, 3));
>     +       clk_dm(IMX8MQ_CLK_MON_SYS_PLL3_DIV,
>     +              imx_clk_divider("sys_pll3_out_monitor",
>     "sys_pll3_out", base + 0x7c, 8, 3));
>     +       clk_dm(IMX8MQ_CLK_MON_DRAM_PLL_DIV,
>     +              imx_clk_divider("dram_pll_out_monitor",
>     "dram_pll_out", base + 0x7c, 12, 3));
>     +       clk_dm(IMX8MQ_CLK_MON_SEL,
>     +              imx_clk_mux_flags("pllout_monitor_sel", base + 0x74,
>     0, 4,
>     +                                pllout_monitor_sels,
>     +                                ARRAY_SIZE(pllout_monitor_sels),
>     +                                CLK_SET_RATE_PARENT));
>     +       clk_dm(IMX8MQ_CLK_MON_CLK2_OUT,
>     +              imx_clk_gate4("pllout_monitor_clk2",
>     "pllout_monitor_sel", base + 0x74, 4));
>     +
>     +       base = dev_read_addr_ptr(dev);
>     +       if (!base) {
>     +               printf("%s : base failed\n", __func__);
>     +               return -EINVAL;
>     +       }
>     +
>     +       clk_dm(IMX8MQ_CLK_A53_SRC,
>     +              imx_clk_mux2("arm_a53_src", base + 0x8000, 24, 3,
>     +                           imx8mq_a53_sels,
>     ARRAY_SIZE(imx8mq_a53_sels)));
>     +       clk_dm(IMX8MQ_CLK_A53_CG,
>     +              imx_clk_gate3("arm_a53_cg", "arm_a53_src", base +
>     0x8000, 28));
>     +       clk_dm(IMX8MQ_CLK_A53_DIV,
>     +              imx_clk_divider2("arm_a53_div", "arm_a53_cg",
>     +                               base + 0x8000, 0, 3));
>     +       clk_dm(IMX8MQ_CLK_A53_CORE,
>     +              imx_clk_mux2("arm_a53_src", base + 0x9880, 24, 1,
>     +                           imx8mq_a53_core_sels,
>     ARRAY_SIZE(imx8mq_a53_core_sels)));
>     +
>     +       clk_dm(IMX8MQ_CLK_AHB,
>     +              imx8m_clk_composite_critical("ahb", imx8mq_ahb_sels,
>     +                                           base + 0x9000));
>     +       clk_dm(IMX8MQ_CLK_IPG_ROOT,
>     +              imx_clk_divider2("ipg_root", "ahb", base + 0x9080, 0,
>     1));
>     +
>     +       clk_dm(IMX8MQ_CLK_ENET_AXI,
>     +              imx8m_clk_composite("enet_axi", imx8mq_enet_axi_sels,
>     +                                  base + 0x8880));
>     +       clk_dm(IMX8MQ_CLK_NAND_USDHC_BUS,
>     +              imx8m_clk_composite_critical("nand_usdhc_bus",
>     +                                           imx8mq_nand_usdhc_sels,
>     +                                           base + 0x8900));
>     +       clk_dm(IMX8MQ_CLK_USB_BUS,
>     +              imx8m_clk_composite("usb_bus", imx8mq_usb_bus_sels,
>     base + 0x8b80));
>     +
>     +       /* IP */
>     +       clk_dm(IMX8MQ_CLK_USDHC1,
>     +              imx8m_clk_composite("usdhc1", imx8mq_usdhc1_sels,
>     +                                  base + 0xac00));
>     +       clk_dm(IMX8MQ_CLK_USDHC2,
>     +              imx8m_clk_composite("usdhc2", imx8mq_usdhc2_sels,
>     +                                  base + 0xac80));
>     +       clk_dm(IMX8MQ_CLK_I2C1,
>     +              imx8m_clk_composite("i2c1", imx8mq_i2c1_sels, base +
>     0xad00));
>     +       clk_dm(IMX8MQ_CLK_I2C2,
>     +              imx8m_clk_composite("i2c2", imx8mq_i2c2_sels, base +
>     0xad80));
>     +       clk_dm(IMX8MQ_CLK_I2C3,
>     +              imx8m_clk_composite("i2c3", imx8mq_i2c3_sels, base +
>     0xae00));
>     +       clk_dm(IMX8MQ_CLK_I2C4,
>     +              imx8m_clk_composite("i2c4", imx8mq_i2c4_sels, base +
>     0xae80));
>     +       clk_dm(IMX8MQ_CLK_WDOG,
>     +              imx8m_clk_composite("wdog", imx8mq_wdog_sels, base +
>     0xb900));
>     +       clk_dm(IMX8MQ_CLK_QSPI,
>     +              imx8m_clk_composite("qspi", imx8mq_qspi_sels, base +
>     0xab80));
>     +       clk_dm(IMX8MQ_CLK_USB_CORE_REF,
>     +              imx8m_clk_composite("usb_core_ref",
>     imx8mq_usb_core_sels, base + 0xb100));
>     +       clk_dm(IMX8MQ_CLK_USB_PHY_REF,
>     +              imx8m_clk_composite("usb_phy_ref",
>     imx8mq_usb_phy_sels, base + 0xb180));
>     +       clk_dm(IMX8MQ_CLK_ECSPI1,
>     +              imx8m_clk_composite("ecspi1", imx8mq_ecspi1_sels,
>     base + 0xb280));
>     +       clk_dm(IMX8MQ_CLK_ECSPI2,
>     +              imx8m_clk_composite("ecspi2", imx8mq_ecspi2_sels,
>     base + 0xb300));
>     +       clk_dm(IMX8MQ_CLK_ECSPI3,
>     +              imx8m_clk_composite("ecspi3", imx8mq_ecspi3_sels,
>     base + 0xc180));
>     +
>     +       clk_dm(IMX8MQ_CLK_ECSPI1_ROOT,
>     +              imx_clk_gate4("ecspi1_root_clk", "ecspi1", base +
>     0x4070, 0));
>     +       clk_dm(IMX8MQ_CLK_ECSPI2_ROOT,
>     +              imx_clk_gate4("ecspi2_root_clk", "ecspi2", base +
>     0x4080, 0));
>     +       clk_dm(IMX8MQ_CLK_ECSPI3_ROOT,
>     +              imx_clk_gate4("ecspi3_root_clk", "ecspi3", base +
>     0x4090, 0));
>     +       clk_dm(IMX8MQ_CLK_I2C1_ROOT,
>     +              imx_clk_gate4("i2c1_root_clk", "i2c1", base + 0x4170,
>     0));
>     +       clk_dm(IMX8MQ_CLK_I2C2_ROOT,
>     +              imx_clk_gate4("i2c2_root_clk", "i2c2", base + 0x4180,
>     0));
>     +       clk_dm(IMX8MQ_CLK_I2C3_ROOT,
>     +              imx_clk_gate4("i2c3_root_clk", "i2c3", base + 0x4190,
>     0));
>     +       clk_dm(IMX8MQ_CLK_I2C4_ROOT,
>     +              imx_clk_gate4("i2c4_root_clk", "i2c4", base + 0x41a0,
>     0));
>     +       clk_dm(IMX8MQ_CLK_OCOTP_ROOT,
>     +              imx_clk_gate4("ocotp_root_clk", "ipg_root", base +
>     0x4220, 0));
>     +       clk_dm(IMX8MQ_CLK_USDHC1_ROOT,
>     +              imx_clk_gate4("usdhc1_root_clk", "usdhc1", base +
>     0x4510, 0));
>     +       clk_dm(IMX8MQ_CLK_USDHC2_ROOT,
>     +              imx_clk_gate4("usdhc2_root_clk", "usdhc2", base +
>     0x4520, 0));
>     +       clk_dm(IMX8MQ_CLK_WDOG1_ROOT,
>     +              imx_clk_gate4("wdog1_root_clk", "wdog", base +
>     0x4530, 0));
>     +       clk_dm(IMX8MQ_CLK_WDOG2_ROOT,
>     +              imx_clk_gate4("wdog2_root_clk", "wdog", base +
>     0x4540, 0));
>     +       clk_dm(IMX8MQ_CLK_WDOG3_ROOT,
>     +              imx_clk_gate4("wdog3_root_clk", "wdog", base +
>     0x4550, 0));
>     +       clk_dm(IMX8MQ_CLK_QSPI_ROOT,
>     +              imx_clk_gate4("qspi_root_clk", "qspi", base + 0x42f0,
>     0));
>     +       clk_dm(IMX8MQ_CLK_USB1_CTRL_ROOT,
>     +              imx_clk_gate4("usb1_ctrl_root_clk", "usb_bus", base +
>     0x44d0, 0));
>     +       clk_dm(IMX8MQ_CLK_USB2_CTRL_ROOT,
>     +              imx_clk_gate4("usb2_ctrl_root_clk", "usb_bus", base +
>     0x44e0, 0));
>     +       clk_dm(IMX8MQ_CLK_USB1_PHY_ROOT,
>     +              imx_clk_gate4("usb1_phy_root_clk", "usb_phy_ref",
>     base + 0x44f0, 0));
>     +       clk_dm(IMX8MQ_CLK_USB2_PHY_ROOT,
>     +              imx_clk_gate4("usb2_phy_root_clk", "usb_phy_ref",
>     base + 0x4500, 0));
>     +
>     +       clk_dm(IMX8MQ_CLK_ENET_REF,
>     +              imx8m_clk_composite("enet_ref", imx8mq_enet_ref_sels,
>     +                                  base + 0xa980));
>     +       clk_dm(IMX8MQ_CLK_ENET_TIMER,
>     +              imx8m_clk_composite("enet_timer", imx8mq_enet_timer_sels,
>     +                                  base + 0xaa00));
>     +       clk_dm(IMX8MQ_CLK_ENET_PHY_REF,
>     +              imx8m_clk_composite("enet_phy", imx8mq_enet_phy_sels,
>     +                                  base + 0xaa80));
>     +       clk_dm(IMX8MQ_CLK_ENET1_ROOT,
>     +              imx_clk_gate4("enet1_root_clk", "enet_axi",
>     +                            base + 0x40a0, 0));
>     +
>     +       return 0;
>     +}
>     +
>     +static const struct udevice_id imx8mq_clk_ids[] = {
>     +       { .compatible = "fsl,imx8mq-ccm" },
>     +       { },
>     +};
>     +
>     +U_BOOT_DRIVER(imx8mq_clk) = {
>     +       .name = "clk_imx8mq",
>     +       .id = UCLASS_CLK,
>     +       .of_match = imx8mq_clk_ids,
>     +       .ops = &imx8mq_clk_ops,
>     +       .probe = imx8mq_clk_probe,
>     +       .flags = DM_FLAG_PRE_RELOC,
>     +};
>     -- 
>     2.25.1
> 
> 
> when adding this patch and enabling CLK_IMX8MQ I see the following on my 
> board .. Any idea what I missed here?
> 
> --- >8 ---
> U-Boot SPL 2022.04-rc4-00008-g390d9bf9a1 (Mar 15 2022 - 16:26:59 +0100)
> Trying to boot from SD card
> 
> 
> U-Boot 2022.04-rc4-00008-g390d9bf9a1 (Mar 15 2022 - 16:26:59 +0100)
> 
> CPU:   Freescale i.MX8MQ rev2.1 at 800 MHz
> Reset cause: POR
> Model: Kontron pITX-imx8m
> DRAM:  alloc space exhausted
> alloc space exhausted
> alloc space exhausted
> alloc space exhausted
> alloc space exhausted
> alloc space exhausted
> alloc space exhausted
> alloc space exhausted
> alloc space exhausted
> alloc space exhausted
> alloc space exhausted
> alloc space exhausted

There maybe not enough space with MALLOC_F_LEN

Regards,
Peng.

> 4 GiB
> clk_register: failed to get <NULL> device (parent of ckil)
> clk_register: failed to get <NULL> device (parent of sys1_pll)
> clk_register: failed to get <NULL> device (parent of sys2_pll)
> clk_register: failed to get <NULL> device (parent of sys3_pll)
> No serial driver found
> resetting ...
> 
> --- >8 ---
> 
> -- 
> Heiko


More information about the U-Boot mailing list