diff mbox series

[V5,5/5] clk: imx: add i.MX93 clk

Message ID 20220225014234.1766305-6-peng.fan@oss.nxp.com (mailing list archive)
State Changes Requested, archived
Headers show
Series imx: add i.MX93 clk bindings and driver | expand

Commit Message

Peng Fan (OSS) Feb. 25, 2022, 1:42 a.m. UTC
From: Peng Fan <peng.fan@nxp.com>

Add i.MX93 clk driver. i.MX93 clk hardware design is different compared
with i.MX8M. It supports 4 sources for each clk root and the sources
are separated into a few groups, low speed/fast io/audio and etc.

Reviewed-by: Abel Vesa <abel.vesa@nxp.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 drivers/clk/imx/Kconfig     |   6 +
 drivers/clk/imx/Makefile    |   2 +
 drivers/clk/imx/clk-imx93.c | 338 ++++++++++++++++++++++++++++++++++++
 3 files changed, 346 insertions(+)
 create mode 100644 drivers/clk/imx/clk-imx93.c

Comments

Stephen Boyd Feb. 25, 2022, 1:58 a.m. UTC | #1
Quoting Peng Fan (OSS) (2022-02-24 17:42:34)
> diff --git a/drivers/clk/imx/clk-imx93.c b/drivers/clk/imx/clk-imx93.c
> new file mode 100644
> index 000000000000..7cd5e7fb0c8b
> --- /dev/null
> +++ b/drivers/clk/imx/clk-imx93.c
> @@ -0,0 +1,338 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Copyright 2021 NXP.
> + */
> +
> +#include <dt-bindings/clock/imx93-clock.h>

Include after linux headers please.

> +#include <linux/clk.h>

Is this include used?

> +#include <linux/clk-provider.h>
> +#include <linux/debugfs.h>

Is this used?

> +#include <linux/err.h>
> +#include <linux/io.h>
> +#include <linux/module.h>
> +#include <linux/of_address.h>
> +#include <linux/platform_device.h>
> +#include <linux/slab.h>
> +#include <linux/types.h>
> +
> +#include "clk.h"
> +
> +enum clk_sel {
> +       LOW_SPEED_IO_SEL,
> +       NON_IO_SEL,
> +       FAST_SEL,
> +       AUDIO_SEL,
> +       VIDEO_SEL,
> +       TPM_SEL,
> +       CKO1_SEL,
> +       CKO2_SEL,
> +       MISC_SEL,
> +       MAX_SEL
> +};
> +
> +static const char *parent_names[MAX_SEL][4] = {
> +       {"osc_24m", "sys_pll_pfd0_div2", "sys_pll_pfd1_div2", "video_pll"},
> +       {"osc_24m", "sys_pll_pfd0_div2", "sys_pll_pfd1_div2", "sys_pll_pfd2_div2"},
> +       {"osc_24m", "sys_pll_pfd0", "sys_pll_pfd1", "sys_pll_pfd2"},
> +       {"osc_24m", "audio_pll", "video_pll", "clk_ext1"},
> +       {"osc_24m", "audio_pll", "video_pll", "sys_pll_pfd0"},
> +       {"osc_24m", "sys_pll_pfd0", "audio_pll", "clk_ext1"},
> +       {"osc_24m", "sys_pll_pfd0", "sys_pll_pfd1", "audio_pll"},
> +       {"osc_24m", "sys_pll_pfd0", "sys_pll_pfd1", "video_pll"},
> +       {"osc_24m", "audio_pll", "video_pll", "sys_pll_pfd2"},
> +};
> +
> +struct imx93_clk_root {
> +       u32 clk;
> +       char *name;
> +       u32 off;
> +       enum clk_sel sel;
> +       unsigned long flags;
> +} root_array[] = {

Can this be const and static?

> +       { IMX93_CLK_A55_PERIPH,         "a55_periph_root",      0x0000, FAST_SEL, CLK_IS_CRITICAL },
> +       { IMX93_CLK_A55_MTR_BUS,        "a55_mtr_bus_root",     0x0080, LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
> +       { IMX93_CLK_A55,                "a55_root",             0x0100, FAST_SEL, CLK_IS_CRITICAL },
> +       { IMX93_CLK_M33,                "m33_root",             0x0180, LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
> +       { IMX93_CLK_BUS_WAKEUP,         "bus_wakeup_root",      0x0280, LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
> +       { IMX93_CLK_BUS_AON,            "bus_aon_root",         0x0300, LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
> +       { IMX93_CLK_WAKEUP_AXI,         "wakeup_axi_root",      0x0380, FAST_SEL, CLK_IS_CRITICAL },
> +       { IMX93_CLK_SWO_TRACE,          "swo_trace_root",       0x0400, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_M33_SYSTICK,        "m33_systick_root",     0x0480, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_FLEXIO1,            "flexio1_root",         0x0500, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_FLEXIO2,            "flexio2_root",         0x0580, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPIT1,              "lpit1_root",           0x0600, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPIT2,              "lpit2_root",           0x0680, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPTMR1,             "lptmr1_root",          0x0700, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPTMR2,             "lptmr2_root",          0x0780, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_TPM1,               "tpm1_root",            0x0800, TPM_SEL, },
> +       { IMX93_CLK_TPM2,               "tpm2_root",            0x0880, TPM_SEL, },
> +       { IMX93_CLK_TPM3,               "tpm3_root",            0x0900, TPM_SEL, },
> +       { IMX93_CLK_TPM4,               "tpm4_root",            0x0980, TPM_SEL, },
> +       { IMX93_CLK_TPM5,               "tpm5_root",            0x0a00, TPM_SEL, },
> +       { IMX93_CLK_TPM6,               "tpm6_root",            0x0a80, TPM_SEL, },
> +       { IMX93_CLK_FLEXSPI1,           "flexspi1_root",        0x0b00, FAST_SEL, },
> +       { IMX93_CLK_CAN1,               "can1_root",            0x0b80, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_CAN2,               "can2_root",            0x0c00, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPUART1,            "lpuart1_root",         0x0c80, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPUART2,            "lpuart2_root",         0x0d00, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPUART3,            "lpuart3_root",         0x0d80, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPUART4,            "lpuart4_root",         0x0e00, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPUART5,            "lpuart5_root",         0x0e80, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPUART6,            "lpuart6_root",         0x0f00, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPUART7,            "lpuart7_root",         0x0f80, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPUART8,            "lpuart8_root",         0x1000, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPI2C1,             "lpi2c1_root",          0x1080, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPI2C2,             "lpi2c2_root",          0x1100, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPI2C3,             "lpi2c3_root",          0x1180, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPI2C4,             "lpi2c4_root",          0x1200, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPI2C5,             "lpi2c5_root",          0x1280, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPI2C6,             "lpi2c6_root",          0x1300, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPI2C7,             "lpi2c7_root",          0x1380, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPI2C8,             "lpi2c8_root",          0x1400, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPSPI1,             "lpspi1_root",          0x1480, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPSPI2,             "lpspi2_root",          0x1500, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPSPI3,             "lpspi3_root",          0x1580, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPSPI4,             "lpspi4_root",          0x1600, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPSPI5,             "lpspi5_root",          0x1680, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPSPI6,             "lpspi6_root",          0x1700, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPSPI7,             "lpspi7_root",          0x1780, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_LPSPI8,             "lpspi8_root",          0x1800, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_I3C1,               "i3c1_root",            0x1880, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_I3C2,               "i3c2_root",            0x1900, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_USDHC1,             "usdhc1_root",          0x1980, FAST_SEL, },
> +       { IMX93_CLK_USDHC2,             "usdhc2_root",          0x1a00, FAST_SEL, },
> +       { IMX93_CLK_USDHC3,             "usdhc3_root",          0x1a80, FAST_SEL, },
> +       { IMX93_CLK_SAI1,               "sai1_root",            0x1b00, AUDIO_SEL, },
> +       { IMX93_CLK_SAI2,               "sai2_root",            0x1b80, AUDIO_SEL, },
> +       { IMX93_CLK_SAI3,               "sai3_root",            0x1c00, AUDIO_SEL, },
> +       { IMX93_CLK_CCM_CKO1,           "ccm_cko1_root",        0x1c80, CKO1_SEL, },
> +       { IMX93_CLK_CCM_CKO2,           "ccm_cko2_root",        0x1d00, CKO2_SEL, },
> +       { IMX93_CLK_CCM_CKO3,           "ccm_cko3_root",        0x1d80, CKO1_SEL, },
> +       { IMX93_CLK_CCM_CKO4,           "ccm_cko4_root",        0x1e00, CKO2_SEL, },
> +       { IMX93_CLK_HSIO,               "hsio_root",            0x1e80, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_HSIO_USB_TEST_60M,  "hsio_usb_test_60m_root", 0x1f00, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_HSIO_ACSCAN_80M,    "hsio_acscan_80m_root", 0x1f80, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_HSIO_ACSCAN_480M,   "hsio_acscan_480m_root", 0x2000, MISC_SEL, },
> +       { IMX93_CLK_ML_APB,             "ml_apb_root",          0x2180, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_ML,                 "ml_root",              0x2200, FAST_SEL, },
> +       { IMX93_CLK_MEDIA_AXI,          "media_axi_root",       0x2280, FAST_SEL, },
> +       { IMX93_CLK_MEDIA_APB,          "media_apb_root",       0x2300, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_MEDIA_LDB,          "media_ldb_root",       0x2380, VIDEO_SEL, },
> +       { IMX93_CLK_MEDIA_DISP_PIX,     "media_disp_pix_root",  0x2400, VIDEO_SEL, },
> +       { IMX93_CLK_CAM_PIX,            "cam_pix_root",         0x2480, VIDEO_SEL, },
> +       { IMX93_CLK_MIPI_TEST_BYTE,     "mipi_test_byte_root",  0x2500, VIDEO_SEL, },
> +       { IMX93_CLK_MIPI_PHY_CFG,       "mipi_phy_cfg_root",    0x2580, VIDEO_SEL, },
> +       { IMX93_CLK_ADC,                "adc_root",             0x2700, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_PDM,                "pdm_root",             0x2780, AUDIO_SEL, },
> +       { IMX93_CLK_TSTMR1,             "tstmr1_root",          0x2800, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_TSTMR2,             "tstmr2_root",          0x2880, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_MQS1,               "mqs1_root",            0x2900, AUDIO_SEL, },
> +       { IMX93_CLK_MQS2,               "mqs2_root",            0x2980, AUDIO_SEL, },
> +       { IMX93_CLK_AUDIO_XCVR,         "audio_xcvr_root",      0x2a00, NON_IO_SEL, },
> +       { IMX93_CLK_SPDIF,              "spdif_root",           0x2a80, AUDIO_SEL, },
> +       { IMX93_CLK_ENET,               "enet_root",            0x2b00, NON_IO_SEL, },
> +       { IMX93_CLK_ENET_TIMER1,        "enet_timer1_root",     0x2b80, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_ENET_TIMER2,        "enet_timer2_root",     0x2c00, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_ENET_REF,           "enet_ref_root",        0x2c80, NON_IO_SEL, },
> +       { IMX93_CLK_ENET_REF_PHY,       "enet_ref_phy_root",    0x2d00, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_I3C1_SLOW,          "i3c1_slow_root",       0x2d80, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_I3C2_SLOW,          "i3c2_slow_root",       0x2e00, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_USB_PHY_BURUNIN,    "usb_phy_root",         0x2e80, LOW_SPEED_IO_SEL, },
> +       { IMX93_CLK_PAL_CAME_SCAN,      "pal_came_scan_root",   0x2f00, MISC_SEL, }
> +};
> +
> +struct imx93_clk_ccgr {
> +       u32 clk;
> +       char *name;
> +       char *parent_name;
> +       u32 off;
> +       unsigned long flags;
> +} ccgr_array[] = {

static const?

> +       { IMX93_CLK_A55_GATE,           "a55",          "a55_root",             0x8000, CLK_IS_CRITICAL },

Please add comments on why clks are critical.

> +       { IMX93_CLK_CM33_GATE,          "cm33",         "m33_root",             0x8040, CLK_IS_CRITICAL },
> +       { IMX93_CLK_ADC1_GATE,          "adc1",         "osc_24m",              0x82c0, },
> +       { IMX93_CLK_WDOG1_GATE,         "wdog1",        "osc_24m",              0x8300, },
> +       { IMX93_CLK_WDOG2_GATE,         "wdog2",        "osc_24m",              0x8340, },
> +       { IMX93_CLK_WDOG3_GATE,         "wdog3",        "osc_24m",              0x8380, },
> +       { IMX93_CLK_WDOG4_GATE,         "wdog4",        "osc_24m",              0x83c0, },
> +       { IMX93_CLK_WDOG5_GATE,         "wdog5",        "osc_24m",              0x8400, },
> +       { IMX93_CLK_SEMA1_GATE,         "sema1",        "bus_aon_root",         0x8440, },
> +       { IMX93_CLK_SEMA2_GATE,         "sema2",        "bus_wakeup_root",      0x8480, },
> +       { IMX93_CLK_MU_A_GATE,          "mu_a",         "bus_aon_root",         0x84c0, },
> +       { IMX93_CLK_MU_B_GATE,          "mu_b",         "bus_aon_root",         0x8500, },
> +       { IMX93_CLK_EDMA1_GATE,         "edma1",        "wakeup_axi_root",      0x8540, },
> +       { IMX93_CLK_EDMA2_GATE,         "edma2",        "wakeup_axi_root",      0x8580, },
> +       { IMX93_CLK_FLEXSPI1_GATE,      "flexspi",      "flexspi_root",         0x8640, },
> +       { IMX93_CLK_GPIO1_GATE,         "gpio1",        "m33_root",             0x8880, },
> +       { IMX93_CLK_GPIO2_GATE,         "gpio2",        "bus_wakeup_root",      0x88c0, },
> +       { IMX93_CLK_GPIO3_GATE,         "gpio3",        "bus_wakeup_root",      0x8900, },
> +       { IMX93_CLK_GPIO4_GATE,         "gpio4",        "bus_wakeup_root",      0x8940, },
> +       { IMX93_CLK_FLEXIO1_GATE,       "flexio1",      "flexio1_root",         0x8980, },
> +       { IMX93_CLK_FLEXIO2_GATE,       "flexio2",      "flexio2_root",         0x89c0, },
> +       { IMX93_CLK_LPIT1_GATE,         "lpit1",        "lpit1_root",           0x8a00, },
> +       { IMX93_CLK_LPIT2_GATE,         "lpit2",        "lpit2_root",           0x8a40, },
> +       { IMX93_CLK_LPTMR1_GATE,        "lptmr1",       "lptmr1_root",          0x8a80, },
> +       { IMX93_CLK_LPTMR2_GATE,        "lptmr2",       "lptmr2_root",          0x8ac0, },
> +       { IMX93_CLK_TPM1_GATE,          "tpm1",         "tpm1_root",            0x8b00, },
> +       { IMX93_CLK_TPM2_GATE,          "tpm2",         "tpm2_root",            0x8b40, },
> +       { IMX93_CLK_TPM3_GATE,          "tpm3",         "tpm3_root",            0x8b80, },
> +       { IMX93_CLK_TPM4_GATE,          "tpm4",         "tpm4_root",            0x8bc0, },
> +       { IMX93_CLK_TPM5_GATE,          "tpm5",         "tpm5_root",            0x8c00, },
> +       { IMX93_CLK_TPM6_GATE,          "tpm6",         "tpm6_root",            0x8c40, },
> +       { IMX93_CLK_CAN1_GATE,          "can1",         "can1_root",            0x8c80, },
> +       { IMX93_CLK_CAN2_GATE,          "can2",         "can2_root",            0x8cc0, },
> +       { IMX93_CLK_LPUART1_GATE,       "lpuart1",      "lpuart1_root",         0x8d00, },
> +       { IMX93_CLK_LPUART2_GATE,       "lpuart2",      "lpuart2_root",         0x8d40, },
> +       { IMX93_CLK_LPUART3_GATE,       "lpuart3",      "lpuart3_root",         0x8d80, },
> +       { IMX93_CLK_LPUART4_GATE,       "lpuart4",      "lpuart4_root",         0x8dc0, },
> +       { IMX93_CLK_LPUART5_GATE,       "lpuart5",      "lpuart5_root",         0x8e00, },
> +       { IMX93_CLK_LPUART6_GATE,       "lpuart6",      "lpuart6_root",         0x8e40, },
> +       { IMX93_CLK_LPUART7_GATE,       "lpuart7",      "lpuart7_root",         0x8e80, },
> +       { IMX93_CLK_LPUART8_GATE,       "lpuart8",      "lpuart8_root",         0x8ec0, },
> +       { IMX93_CLK_LPI2C1_GATE,        "lpi2c1",       "lpi2c1_root",          0x8f00, },
> +       { IMX93_CLK_LPI2C2_GATE,        "lpi2c2",       "lpi2c2_root",          0x8f40, },
> +       { IMX93_CLK_LPI2C3_GATE,        "lpi2c3",       "lpi2c3_root",          0x8f80, },
> +       { IMX93_CLK_LPI2C4_GATE,        "lpi2c4",       "lpi2c4_root",          0x8fc0, },
> +       { IMX93_CLK_LPI2C5_GATE,        "lpi2c5",       "lpi2c5_root",          0x9000, },
> +       { IMX93_CLK_LPI2C6_GATE,        "lpi2c6",       "lpi2c6_root",          0x9040, },
> +       { IMX93_CLK_LPI2C7_GATE,        "lpi2c7",       "lpi2c7_root",          0x9080, },
> +       { IMX93_CLK_LPI2C8_GATE,        "lpi2c8",       "lpi2c8_root",          0x90c0, },
> +       { IMX93_CLK_LPSPI1_GATE,        "lpspi1",       "lpspi1_root",          0x9100, },
> +       { IMX93_CLK_LPSPI2_GATE,        "lpspi2",       "lpspi2_root",          0x9140, },
> +       { IMX93_CLK_LPSPI3_GATE,        "lpspi3",       "lpspi3_root",          0x9180, },
> +       { IMX93_CLK_LPSPI4_GATE,        "lpspi4",       "lpspi4_root",          0x91c0, },
> +       { IMX93_CLK_LPSPI5_GATE,        "lpspi5",       "lpspi5_root",          0x9200, },
> +       { IMX93_CLK_LPSPI6_GATE,        "lpspi6",       "lpspi6_root",          0x9240, },
> +       { IMX93_CLK_LPSPI7_GATE,        "lpspi7",       "lpspi7_root",          0x9280, },
> +       { IMX93_CLK_LPSPI8_GATE,        "lpspi8",       "lpspi8_root",          0x92c0, },
> +       { IMX93_CLK_I3C1_GATE,          "i3c1",         "i3c1_root",            0x9300, },
> +       { IMX93_CLK_I3C2_GATE,          "i3c2",         "i3c2_root",            0x9340, },
> +       { IMX93_CLK_USDHC1_GATE,        "usdhc1",       "usdhc1_root",          0x9380, },
> +       { IMX93_CLK_USDHC2_GATE,        "usdhc2",       "usdhc2_root",          0x93c0, },
> +       { IMX93_CLK_USDHC3_GATE,        "usdhc3",       "usdhc3_root",          0x9400, },
> +       { IMX93_CLK_SAI1_GATE,          "sai1",         "sai1_root",            0x9440, },
> +       { IMX93_CLK_SAI2_GATE,          "sai2",         "sai2_root",            0x9480, },
> +       { IMX93_CLK_SAI3_GATE,          "sai3",         "sai3_root",            0x94c0, },
> +       { IMX93_CLK_MIPI_CSI_GATE,      "mipi_csi",     "media_apb_root",       0x9580, },
> +       { IMX93_CLK_MIPI_DSI_GATE,      "mipi_dsi",     "media_apb_root",       0x95c0, },
> +       { IMX93_CLK_LVDS_GATE,          "lvds",         "media_ldb_root",       0x9600, },
> +       { IMX93_CLK_LCDIF_GATE,         "lcdif",        "media_apb_root",       0x9640, },
> +       { IMX93_CLK_PXP_GATE,           "pxp",          "media_apb_root",       0x9680, },
> +       { IMX93_CLK_ISI_GATE,           "isi",          "media_apb_root",       0x96c0, },
> +       { IMX93_CLK_NIC_MEDIA_GATE,     "nic_media",    "media_apb_root",       0x9700, },
> +       { IMX93_CLK_USB_CONTROLLER_GATE, "usb_controller", "hsio_root",         0x9a00, },
> +       { IMX93_CLK_USB_TEST_60M_GATE,  "usb_test_60m", "hsio_usb_test_60m_root", 0x9a40, },
> +       { IMX93_CLK_HSIO_TROUT_24M_GATE, "hsio_trout_24m", "osc_24m",           0x9a80, },
> +       { IMX93_CLK_PDM_GATE,           "pdm",          "pdm_root",             0x9ac0, },
> +       { IMX93_CLK_MQS1_GATE,          "mqs1",         "sai1_root",            0x9b00, },
> +       { IMX93_CLK_MQS2_GATE,          "mqs2",         "sai3_root",            0x9b40, },
> +       { IMX93_CLK_AUD_XCVR_GATE,      "aud_xcvr",     "audio_xcvr_root",      0x9b80, },
> +       { IMX93_CLK_SPDIF_GATE,         "spdif",        "spdif_root",           0x9c00, },
> +       { IMX93_CLK_HSIO_32K_GATE,      "hsio_32k",     "osc_32k",              0x9dc0, },
> +       { IMX93_CLK_ENET1_GATE,         "enet1",        "enet_root",            0x9e00, },
> +       { IMX93_CLK_ENET_QOS_GATE,      "enet_qos",     "wakeup_axi_root",      0x9e40, },
> +       { IMX93_CLK_SYS_CNT_GATE,       "sys_cnt",      "osc_24m",              0x9e80, },
> +       { IMX93_CLK_TSTMR1_GATE,        "tstmr1",       "bus_aon_root",         0x9ec0, },
> +       { IMX93_CLK_TSTMR2_GATE,        "tstmr2",       "bus_wakeup_root",      0x9f00, },
> +       { IMX93_CLK_TMC_GATE,           "tmc",          "osc_24m",              0x9f40, },
> +       { IMX93_CLK_PMRO_GATE,          "pmro",         "osc_24m",              0x9f80, }
> +};
> +
> +static struct clk_hw_onecell_data *clk_hw_data;
> +static struct clk_hw **clks;
> +
> +static int imx93_clocks_probe(struct platform_device *pdev)
> +{
> +       struct device *dev = &pdev->dev;
> +       struct device_node *np = dev->of_node;
> +       struct imx93_clk_root *root;
> +       struct imx93_clk_ccgr *ccgr;
> +       void __iomem *base = NULL;
> +       int i, ret;
> +
> +       clk_hw_data = kzalloc(struct_size(clk_hw_data, hws,
> +                                         IMX93_CLK_END), GFP_KERNEL);
> +       if (WARN_ON(!clk_hw_data))

Drop WARN_ON() as allocation already prints a stacktrace on failure.

> +               return -ENOMEM;
> +
> +       clk_hw_data->num = IMX93_CLK_END;
> +       clks = clk_hw_data->hws;
> +
> +       clks[IMX93_CLK_DUMMY] = imx_clk_hw_fixed("dummy", 0);
> +       clks[IMX93_CLK_24M] = imx_obtain_fixed_clk_hw(np, "osc_24m");
> +       clks[IMX93_CLK_32K] = imx_obtain_fixed_clk_hw(np, "osc_32k");
> +       clks[IMX93_CLK_EXT1] = imx_obtain_fixed_clk_hw(np, "clk_ext1");
> +
> +       clks[IMX93_CLK_SYS_PLL_PFD0] = imx_clk_hw_fixed("sys_pll_pfd0", 1000000000);
> +       clks[IMX93_CLK_SYS_PLL_PFD0_DIV2] = imx_clk_hw_fixed_factor("sys_pll_pfd0_div2",
> +                                                                   "sys_pll_pfd0", 1, 2);
> +       clks[IMX93_CLK_SYS_PLL_PFD1] = imx_clk_hw_fixed("sys_pll_pfd1", 800000000);
> +       clks[IMX93_CLK_SYS_PLL_PFD1_DIV2] = imx_clk_hw_fixed_factor("sys_pll_pfd1_div2",
Peng Fan Feb. 28, 2022, 1:49 a.m. UTC | #2
> Subject: Re: [PATCH V5 5/5] clk: imx: add i.MX93 clk
> 
> Quoting Peng Fan (OSS) (2022-02-24 17:42:34)
> > diff --git a/drivers/clk/imx/clk-imx93.c b/drivers/clk/imx/clk-imx93.c
> > new file mode 100644 index 000000000000..7cd5e7fb0c8b
> > --- /dev/null
> > +++ b/drivers/clk/imx/clk-imx93.c
> > @@ -0,0 +1,338 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Copyright 2021 NXP.
> > + */
> > +
> > +#include <dt-bindings/clock/imx93-clock.h>
> 
> Include after linux headers please.
> 
> > +#include <linux/clk.h>
> 
> Is this include used?
> 
> > +#include <linux/clk-provider.h>
> > +#include <linux/debugfs.h>
> 
> Is this used?

I'll cleanup the include in V6.

> 
> > +#include <linux/err.h>
> > +#include <linux/io.h>
> > +#include <linux/module.h>
> > +#include <linux/of_address.h>
> > +#include <linux/platform_device.h>
> > +#include <linux/slab.h>
> > +#include <linux/types.h>
> > +
> > +#include "clk.h"
> > +
> > +enum clk_sel {
> > +       LOW_SPEED_IO_SEL,
> > +       NON_IO_SEL,
> > +       FAST_SEL,
> > +       AUDIO_SEL,
> > +       VIDEO_SEL,
> > +       TPM_SEL,
> > +       CKO1_SEL,
> > +       CKO2_SEL,
> > +       MISC_SEL,
> > +       MAX_SEL
> > +};
> > +
> > +static const char *parent_names[MAX_SEL][4] = {
> > +       {"osc_24m", "sys_pll_pfd0_div2", "sys_pll_pfd1_div2",
> "video_pll"},
> > +       {"osc_24m", "sys_pll_pfd0_div2", "sys_pll_pfd1_div2",
> "sys_pll_pfd2_div2"},
> > +       {"osc_24m", "sys_pll_pfd0", "sys_pll_pfd1", "sys_pll_pfd2"},
> > +       {"osc_24m", "audio_pll", "video_pll", "clk_ext1"},
> > +       {"osc_24m", "audio_pll", "video_pll", "sys_pll_pfd0"},
> > +       {"osc_24m", "sys_pll_pfd0", "audio_pll", "clk_ext1"},
> > +       {"osc_24m", "sys_pll_pfd0", "sys_pll_pfd1", "audio_pll"},
> > +       {"osc_24m", "sys_pll_pfd0", "sys_pll_pfd1", "video_pll"},
> > +       {"osc_24m", "audio_pll", "video_pll", "sys_pll_pfd2"}, };
> > +
> > +struct imx93_clk_root {
> > +       u32 clk;
> > +       char *name;
> > +       u32 off;
> > +       enum clk_sel sel;
> > +       unsigned long flags;
> > +} root_array[] = {
> 
> Can this be const and static?
Yes. Fix in V6.

> 
> > +       { IMX93_CLK_A55_PERIPH,         "a55_periph_root",
> 0x0000, FAST_SEL, CLK_IS_CRITICAL },
> > +       { IMX93_CLK_A55_MTR_BUS,        "a55_mtr_bus_root",
> 0x0080, LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
> > +       { IMX93_CLK_A55,                "a55_root",
> 0x0100, FAST_SEL, CLK_IS_CRITICAL },
> > +       { IMX93_CLK_M33,                "m33_root",
> 0x0180, LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
> > +       { IMX93_CLK_BUS_WAKEUP,         "bus_wakeup_root",
> 0x0280, LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
> > +       { IMX93_CLK_BUS_AON,            "bus_aon_root",
> 0x0300, LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
> > +       { IMX93_CLK_WAKEUP_AXI,         "wakeup_axi_root",
> 0x0380, FAST_SEL, CLK_IS_CRITICAL },
> > +       { IMX93_CLK_SWO_TRACE,          "swo_trace_root",
> 0x0400, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_M33_SYSTICK,        "m33_systick_root",
> 0x0480, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_FLEXIO1,            "flexio1_root",
> 0x0500, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_FLEXIO2,            "flexio2_root",
> 0x0580, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPIT1,              "lpit1_root",
> 0x0600, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPIT2,              "lpit2_root",
> 0x0680, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPTMR1,             "lptmr1_root",
> 0x0700, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPTMR2,             "lptmr2_root",
> 0x0780, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_TPM1,               "tpm1_root",
> 0x0800, TPM_SEL, },
> > +       { IMX93_CLK_TPM2,               "tpm2_root",
> 0x0880, TPM_SEL, },
> > +       { IMX93_CLK_TPM3,               "tpm3_root",
> 0x0900, TPM_SEL, },
> > +       { IMX93_CLK_TPM4,               "tpm4_root",
> 0x0980, TPM_SEL, },
> > +       { IMX93_CLK_TPM5,               "tpm5_root",
> 0x0a00, TPM_SEL, },
> > +       { IMX93_CLK_TPM6,               "tpm6_root",
> 0x0a80, TPM_SEL, },
> > +       { IMX93_CLK_FLEXSPI1,           "flexspi1_root",
> 0x0b00, FAST_SEL, },
> > +       { IMX93_CLK_CAN1,               "can1_root",
> 0x0b80, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_CAN2,               "can2_root",
> 0x0c00, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPUART1,            "lpuart1_root",
> 0x0c80, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPUART2,            "lpuart2_root",
> 0x0d00, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPUART3,            "lpuart3_root",
> 0x0d80, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPUART4,            "lpuart4_root",
> 0x0e00, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPUART5,            "lpuart5_root",
> 0x0e80, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPUART6,            "lpuart6_root",
> 0x0f00, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPUART7,            "lpuart7_root",
> 0x0f80, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPUART8,            "lpuart8_root",
> 0x1000, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPI2C1,             "lpi2c1_root",
> 0x1080, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPI2C2,             "lpi2c2_root",
> 0x1100, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPI2C3,             "lpi2c3_root",
> 0x1180, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPI2C4,             "lpi2c4_root",
> 0x1200, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPI2C5,             "lpi2c5_root",
> 0x1280, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPI2C6,             "lpi2c6_root",
> 0x1300, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPI2C7,             "lpi2c7_root",
> 0x1380, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPI2C8,             "lpi2c8_root",
> 0x1400, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPSPI1,             "lpspi1_root",
> 0x1480, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPSPI2,             "lpspi2_root",
> 0x1500, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPSPI3,             "lpspi3_root",
> 0x1580, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPSPI4,             "lpspi4_root",
> 0x1600, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPSPI5,             "lpspi5_root",
> 0x1680, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPSPI6,             "lpspi6_root",
> 0x1700, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPSPI7,             "lpspi7_root",
> 0x1780, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_LPSPI8,             "lpspi8_root",
> 0x1800, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_I3C1,               "i3c1_root",
> 0x1880, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_I3C2,               "i3c2_root",
> 0x1900, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_USDHC1,             "usdhc1_root",
> 0x1980, FAST_SEL, },
> > +       { IMX93_CLK_USDHC2,             "usdhc2_root",
> 0x1a00, FAST_SEL, },
> > +       { IMX93_CLK_USDHC3,             "usdhc3_root",
> 0x1a80, FAST_SEL, },
> > +       { IMX93_CLK_SAI1,               "sai1_root",
> 0x1b00, AUDIO_SEL, },
> > +       { IMX93_CLK_SAI2,               "sai2_root",
> 0x1b80, AUDIO_SEL, },
> > +       { IMX93_CLK_SAI3,               "sai3_root",
> 0x1c00, AUDIO_SEL, },
> > +       { IMX93_CLK_CCM_CKO1,           "ccm_cko1_root",
> 0x1c80, CKO1_SEL, },
> > +       { IMX93_CLK_CCM_CKO2,           "ccm_cko2_root",
> 0x1d00, CKO2_SEL, },
> > +       { IMX93_CLK_CCM_CKO3,           "ccm_cko3_root",
> 0x1d80, CKO1_SEL, },
> > +       { IMX93_CLK_CCM_CKO4,           "ccm_cko4_root",
> 0x1e00, CKO2_SEL, },
> > +       { IMX93_CLK_HSIO,               "hsio_root",
> 0x1e80, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_HSIO_USB_TEST_60M,
> "hsio_usb_test_60m_root", 0x1f00, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_HSIO_ACSCAN_80M,    "hsio_acscan_80m_root",
> 0x1f80, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_HSIO_ACSCAN_480M,
> "hsio_acscan_480m_root", 0x2000, MISC_SEL, },
> > +       { IMX93_CLK_ML_APB,             "ml_apb_root",
> 0x2180, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_ML,                 "ml_root",
> 0x2200, FAST_SEL, },
> > +       { IMX93_CLK_MEDIA_AXI,          "media_axi_root",
> 0x2280, FAST_SEL, },
> > +       { IMX93_CLK_MEDIA_APB,          "media_apb_root",
> 0x2300, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_MEDIA_LDB,          "media_ldb_root",
> 0x2380, VIDEO_SEL, },
> > +       { IMX93_CLK_MEDIA_DISP_PIX,     "media_disp_pix_root",
> 0x2400, VIDEO_SEL, },
> > +       { IMX93_CLK_CAM_PIX,            "cam_pix_root",
> 0x2480, VIDEO_SEL, },
> > +       { IMX93_CLK_MIPI_TEST_BYTE,     "mipi_test_byte_root",
> 0x2500, VIDEO_SEL, },
> > +       { IMX93_CLK_MIPI_PHY_CFG,       "mipi_phy_cfg_root",
> 0x2580, VIDEO_SEL, },
> > +       { IMX93_CLK_ADC,                "adc_root",
> 0x2700, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_PDM,                "pdm_root",
> 0x2780, AUDIO_SEL, },
> > +       { IMX93_CLK_TSTMR1,             "tstmr1_root",
> 0x2800, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_TSTMR2,             "tstmr2_root",
> 0x2880, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_MQS1,               "mqs1_root",
> 0x2900, AUDIO_SEL, },
> > +       { IMX93_CLK_MQS2,               "mqs2_root",
> 0x2980, AUDIO_SEL, },
> > +       { IMX93_CLK_AUDIO_XCVR,         "audio_xcvr_root",
> 0x2a00, NON_IO_SEL, },
> > +       { IMX93_CLK_SPDIF,              "spdif_root",
> 0x2a80, AUDIO_SEL, },
> > +       { IMX93_CLK_ENET,               "enet_root",
> 0x2b00, NON_IO_SEL, },
> > +       { IMX93_CLK_ENET_TIMER1,        "enet_timer1_root",
> 0x2b80, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_ENET_TIMER2,        "enet_timer2_root",
> 0x2c00, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_ENET_REF,           "enet_ref_root",
> 0x2c80, NON_IO_SEL, },
> > +       { IMX93_CLK_ENET_REF_PHY,       "enet_ref_phy_root",
> 0x2d00, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_I3C1_SLOW,          "i3c1_slow_root",
> 0x2d80, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_I3C2_SLOW,          "i3c2_slow_root",
> 0x2e00, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_USB_PHY_BURUNIN,    "usb_phy_root",
> 0x2e80, LOW_SPEED_IO_SEL, },
> > +       { IMX93_CLK_PAL_CAME_SCAN,      "pal_came_scan_root",
> 0x2f00, MISC_SEL, }
> > +};
> > +
> > +struct imx93_clk_ccgr {
> > +       u32 clk;
> > +       char *name;
> > +       char *parent_name;
> > +       u32 off;
> > +       unsigned long flags;
> > +} ccgr_array[] = {
> 
> static const?

Yes.

> 
> > +       { IMX93_CLK_A55_GATE,           "a55",
> "a55_root",             0x8000, CLK_IS_CRITICAL },
> 
> Please add comments on why clks are critical.

Technically a55_root should not be ciritical and should be managed
by CPU dev, but since the early SW support will not have
that ready in upstream, so I add cirtical. I'll drop in V6.

For the following M33, disable it will cause system hang.
> 
> > +       { IMX93_CLK_CM33_GATE,          "cm33",
> "m33_root",             0x8040, CLK_IS_CRITICAL },
> > +       { IMX93_CLK_ADC1_GATE,          "adc1",
> "osc_24m",              0x82c0, },
> > +       { IMX93_CLK_WDOG1_GATE,         "wdog1",
> "osc_24m",              0x8300, },
> > +       { IMX93_CLK_WDOG2_GATE,         "wdog2",
> "osc_24m",              0x8340, },
> > +       { IMX93_CLK_WDOG3_GATE,         "wdog3",
> "osc_24m",              0x8380, },
> > +       { IMX93_CLK_WDOG4_GATE,         "wdog4",
> "osc_24m",              0x83c0, },
> > +       { IMX93_CLK_WDOG5_GATE,         "wdog5",
> "osc_24m",              0x8400, },
> > +       { IMX93_CLK_SEMA1_GATE,         "sema1",
> "bus_aon_root",         0x8440, },
> > +       { IMX93_CLK_SEMA2_GATE,         "sema2",
> "bus_wakeup_root",      0x8480, },
> > +       { IMX93_CLK_MU_A_GATE,          "mu_a",
> "bus_aon_root",         0x84c0, },
> > +       { IMX93_CLK_MU_B_GATE,          "mu_b",
> "bus_aon_root",         0x8500, },
> > +       { IMX93_CLK_EDMA1_GATE,         "edma1",
> "wakeup_axi_root",      0x8540, },
> > +       { IMX93_CLK_EDMA2_GATE,         "edma2",
> "wakeup_axi_root",      0x8580, },
> > +       { IMX93_CLK_FLEXSPI1_GATE,      "flexspi",
> "flexspi_root",         0x8640, },
> > +       { IMX93_CLK_GPIO1_GATE,         "gpio1",
> "m33_root",             0x8880, },
> > +       { IMX93_CLK_GPIO2_GATE,         "gpio2",
> "bus_wakeup_root",      0x88c0, },
> > +       { IMX93_CLK_GPIO3_GATE,         "gpio3",
> "bus_wakeup_root",      0x8900, },
> > +       { IMX93_CLK_GPIO4_GATE,         "gpio4",
> "bus_wakeup_root",      0x8940, },
> > +       { IMX93_CLK_FLEXIO1_GATE,       "flexio1",
> "flexio1_root",         0x8980, },
> > +       { IMX93_CLK_FLEXIO2_GATE,       "flexio2",
> "flexio2_root",         0x89c0, },
> > +       { IMX93_CLK_LPIT1_GATE,         "lpit1",
> "lpit1_root",           0x8a00, },
> > +       { IMX93_CLK_LPIT2_GATE,         "lpit2",
> "lpit2_root",           0x8a40, },
> > +       { IMX93_CLK_LPTMR1_GATE,        "lptmr1",
> "lptmr1_root",          0x8a80, },
> > +       { IMX93_CLK_LPTMR2_GATE,        "lptmr2",
> "lptmr2_root",          0x8ac0, },
> > +       { IMX93_CLK_TPM1_GATE,          "tpm1",
> "tpm1_root",            0x8b00, },
> > +       { IMX93_CLK_TPM2_GATE,          "tpm2",
> "tpm2_root",            0x8b40, },
> > +       { IMX93_CLK_TPM3_GATE,          "tpm3",
> "tpm3_root",            0x8b80, },
> > +       { IMX93_CLK_TPM4_GATE,          "tpm4",
> "tpm4_root",            0x8bc0, },
> > +       { IMX93_CLK_TPM5_GATE,          "tpm5",
> "tpm5_root",            0x8c00, },
> > +       { IMX93_CLK_TPM6_GATE,          "tpm6",
> "tpm6_root",            0x8c40, },
> > +       { IMX93_CLK_CAN1_GATE,          "can1",
> "can1_root",            0x8c80, },
> > +       { IMX93_CLK_CAN2_GATE,          "can2",
> "can2_root",            0x8cc0, },
> > +       { IMX93_CLK_LPUART1_GATE,       "lpuart1",
> "lpuart1_root",         0x8d00, },
> > +       { IMX93_CLK_LPUART2_GATE,       "lpuart2",
> "lpuart2_root",         0x8d40, },
> > +       { IMX93_CLK_LPUART3_GATE,       "lpuart3",
> "lpuart3_root",         0x8d80, },
> > +       { IMX93_CLK_LPUART4_GATE,       "lpuart4",
> "lpuart4_root",         0x8dc0, },
> > +       { IMX93_CLK_LPUART5_GATE,       "lpuart5",
> "lpuart5_root",         0x8e00, },
> > +       { IMX93_CLK_LPUART6_GATE,       "lpuart6",
> "lpuart6_root",         0x8e40, },
> > +       { IMX93_CLK_LPUART7_GATE,       "lpuart7",
> "lpuart7_root",         0x8e80, },
> > +       { IMX93_CLK_LPUART8_GATE,       "lpuart8",
> "lpuart8_root",         0x8ec0, },
> > +       { IMX93_CLK_LPI2C1_GATE,        "lpi2c1",
> "lpi2c1_root",          0x8f00, },
> > +       { IMX93_CLK_LPI2C2_GATE,        "lpi2c2",
> "lpi2c2_root",          0x8f40, },
> > +       { IMX93_CLK_LPI2C3_GATE,        "lpi2c3",
> "lpi2c3_root",          0x8f80, },
> > +       { IMX93_CLK_LPI2C4_GATE,        "lpi2c4",
> "lpi2c4_root",          0x8fc0, },
> > +       { IMX93_CLK_LPI2C5_GATE,        "lpi2c5",
> "lpi2c5_root",          0x9000, },
> > +       { IMX93_CLK_LPI2C6_GATE,        "lpi2c6",
> "lpi2c6_root",          0x9040, },
> > +       { IMX93_CLK_LPI2C7_GATE,        "lpi2c7",
> "lpi2c7_root",          0x9080, },
> > +       { IMX93_CLK_LPI2C8_GATE,        "lpi2c8",
> "lpi2c8_root",          0x90c0, },
> > +       { IMX93_CLK_LPSPI1_GATE,        "lpspi1",
> "lpspi1_root",          0x9100, },
> > +       { IMX93_CLK_LPSPI2_GATE,        "lpspi2",
> "lpspi2_root",          0x9140, },
> > +       { IMX93_CLK_LPSPI3_GATE,        "lpspi3",
> "lpspi3_root",          0x9180, },
> > +       { IMX93_CLK_LPSPI4_GATE,        "lpspi4",
> "lpspi4_root",          0x91c0, },
> > +       { IMX93_CLK_LPSPI5_GATE,        "lpspi5",
> "lpspi5_root",          0x9200, },
> > +       { IMX93_CLK_LPSPI6_GATE,        "lpspi6",
> "lpspi6_root",          0x9240, },
> > +       { IMX93_CLK_LPSPI7_GATE,        "lpspi7",
> "lpspi7_root",          0x9280, },
> > +       { IMX93_CLK_LPSPI8_GATE,        "lpspi8",
> "lpspi8_root",          0x92c0, },
> > +       { IMX93_CLK_I3C1_GATE,          "i3c1",
> "i3c1_root",            0x9300, },
> > +       { IMX93_CLK_I3C2_GATE,          "i3c2",
> "i3c2_root",            0x9340, },
> > +       { IMX93_CLK_USDHC1_GATE,        "usdhc1",
> "usdhc1_root",          0x9380, },
> > +       { IMX93_CLK_USDHC2_GATE,        "usdhc2",
> "usdhc2_root",          0x93c0, },
> > +       { IMX93_CLK_USDHC3_GATE,        "usdhc3",
> "usdhc3_root",          0x9400, },
> > +       { IMX93_CLK_SAI1_GATE,          "sai1",
> "sai1_root",            0x9440, },
> > +       { IMX93_CLK_SAI2_GATE,          "sai2",
> "sai2_root",            0x9480, },
> > +       { IMX93_CLK_SAI3_GATE,          "sai3",
> "sai3_root",            0x94c0, },
> > +       { IMX93_CLK_MIPI_CSI_GATE,      "mipi_csi",
> "media_apb_root",       0x9580, },
> > +       { IMX93_CLK_MIPI_DSI_GATE,      "mipi_dsi",
> "media_apb_root",       0x95c0, },
> > +       { IMX93_CLK_LVDS_GATE,          "lvds",
> "media_ldb_root",       0x9600, },
> > +       { IMX93_CLK_LCDIF_GATE,         "lcdif",
> "media_apb_root",       0x9640, },
> > +       { IMX93_CLK_PXP_GATE,           "pxp",
> "media_apb_root",       0x9680, },
> > +       { IMX93_CLK_ISI_GATE,           "isi",
> "media_apb_root",       0x96c0, },
> > +       { IMX93_CLK_NIC_MEDIA_GATE,     "nic_media",
> "media_apb_root",       0x9700, },
> > +       { IMX93_CLK_USB_CONTROLLER_GATE, "usb_controller",
> "hsio_root",         0x9a00, },
> > +       { IMX93_CLK_USB_TEST_60M_GATE,  "usb_test_60m",
> "hsio_usb_test_60m_root", 0x9a40, },
> > +       { IMX93_CLK_HSIO_TROUT_24M_GATE, "hsio_trout_24m",
> "osc_24m",           0x9a80, },
> > +       { IMX93_CLK_PDM_GATE,           "pdm",
> "pdm_root",             0x9ac0, },
> > +       { IMX93_CLK_MQS1_GATE,          "mqs1",
> "sai1_root",            0x9b00, },
> > +       { IMX93_CLK_MQS2_GATE,          "mqs2",
> "sai3_root",            0x9b40, },
> > +       { IMX93_CLK_AUD_XCVR_GATE,      "aud_xcvr",
> "audio_xcvr_root",      0x9b80, },
> > +       { IMX93_CLK_SPDIF_GATE,         "spdif",
> "spdif_root",           0x9c00, },
> > +       { IMX93_CLK_HSIO_32K_GATE,      "hsio_32k",
> "osc_32k",              0x9dc0, },
> > +       { IMX93_CLK_ENET1_GATE,         "enet1",
> "enet_root",            0x9e00, },
> > +       { IMX93_CLK_ENET_QOS_GATE,      "enet_qos",
> "wakeup_axi_root",      0x9e40, },
> > +       { IMX93_CLK_SYS_CNT_GATE,       "sys_cnt",
> "osc_24m",              0x9e80, },
> > +       { IMX93_CLK_TSTMR1_GATE,        "tstmr1",
> "bus_aon_root",         0x9ec0, },
> > +       { IMX93_CLK_TSTMR2_GATE,        "tstmr2",
> "bus_wakeup_root",      0x9f00, },
> > +       { IMX93_CLK_TMC_GATE,           "tmc",
> "osc_24m",              0x9f40, },
> > +       { IMX93_CLK_PMRO_GATE,          "pmro",
> "osc_24m",              0x9f80, }
> > +};
> > +
> > +static struct clk_hw_onecell_data *clk_hw_data; static struct clk_hw
> > +**clks;
> > +
> > +static int imx93_clocks_probe(struct platform_device *pdev) {
> > +       struct device *dev = &pdev->dev;
> > +       struct device_node *np = dev->of_node;
> > +       struct imx93_clk_root *root;
> > +       struct imx93_clk_ccgr *ccgr;
> > +       void __iomem *base = NULL;
> > +       int i, ret;
> > +
> > +       clk_hw_data = kzalloc(struct_size(clk_hw_data, hws,
> > +                                         IMX93_CLK_END),
> GFP_KERNEL);
> > +       if (WARN_ON(!clk_hw_data))
> 
> Drop WARN_ON() as allocation already prints a stacktrace on failure.

Fix in V6

Thanks,
Peng.

> 
> > +               return -ENOMEM;
> > +
> > +       clk_hw_data->num = IMX93_CLK_END;
> > +       clks = clk_hw_data->hws;
> > +
> > +       clks[IMX93_CLK_DUMMY] = imx_clk_hw_fixed("dummy", 0);
> > +       clks[IMX93_CLK_24M] = imx_obtain_fixed_clk_hw(np,
> "osc_24m");
> > +       clks[IMX93_CLK_32K] = imx_obtain_fixed_clk_hw(np, "osc_32k");
> > +       clks[IMX93_CLK_EXT1] = imx_obtain_fixed_clk_hw(np,
> > + "clk_ext1");
> > +
> > +       clks[IMX93_CLK_SYS_PLL_PFD0] =
> imx_clk_hw_fixed("sys_pll_pfd0", 1000000000);
> > +       clks[IMX93_CLK_SYS_PLL_PFD0_DIV2] =
> imx_clk_hw_fixed_factor("sys_pll_pfd0_div2",
> > +
> "sys_pll_pfd0", 1, 2);
> > +       clks[IMX93_CLK_SYS_PLL_PFD1] =
> imx_clk_hw_fixed("sys_pll_pfd1", 800000000);
> > +       clks[IMX93_CLK_SYS_PLL_PFD1_DIV2] =
> > + imx_clk_hw_fixed_factor("sys_pll_pfd1_div2",
diff mbox series

Patch

diff --git a/drivers/clk/imx/Kconfig b/drivers/clk/imx/Kconfig
index 45641b8bdc50..cc464a42d646 100644
--- a/drivers/clk/imx/Kconfig
+++ b/drivers/clk/imx/Kconfig
@@ -106,6 +106,12 @@  config CLK_IMX8ULP
 	help
 	    Build the driver for i.MX8ULP CCM Clock Driver
 
+config CLK_IMX93
+	tristate "IMX93 CCM Clock Driver"
+	depends on ARCH_MXC || COMPILE_TEST
+	help
+	    Build the driver for i.MX93 CCM Clock Driver
+
 config CLK_IMXRT1050
 	tristate "IMXRT1050 CCM Clock Driver"
 	depends on SOC_IMXRT
diff --git a/drivers/clk/imx/Makefile b/drivers/clk/imx/Makefile
index 60c8a4bb7574..88b9b9285d22 100644
--- a/drivers/clk/imx/Makefile
+++ b/drivers/clk/imx/Makefile
@@ -28,6 +28,8 @@  obj-$(CONFIG_CLK_IMX8MN) += clk-imx8mn.o
 obj-$(CONFIG_CLK_IMX8MP) += clk-imx8mp.o
 obj-$(CONFIG_CLK_IMX8MQ) += clk-imx8mq.o
 
+obj-$(CONFIG_CLK_IMX93) += clk-imx93.o
+
 obj-$(CONFIG_MXC_CLK_SCU) += clk-imx-scu.o clk-imx-lpcg-scu.o
 clk-imx-scu-$(CONFIG_CLK_IMX8QXP) += clk-scu.o clk-imx8qxp.o \
 				     clk-imx8qxp-rsrc.o clk-imx8qm-rsrc.o \
diff --git a/drivers/clk/imx/clk-imx93.c b/drivers/clk/imx/clk-imx93.c
new file mode 100644
index 000000000000..7cd5e7fb0c8b
--- /dev/null
+++ b/drivers/clk/imx/clk-imx93.c
@@ -0,0 +1,338 @@ 
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Copyright 2021 NXP.
+ */
+
+#include <dt-bindings/clock/imx93-clock.h>
+#include <linux/clk.h>
+#include <linux/clk-provider.h>
+#include <linux/debugfs.h>
+#include <linux/err.h>
+#include <linux/io.h>
+#include <linux/module.h>
+#include <linux/of_address.h>
+#include <linux/platform_device.h>
+#include <linux/slab.h>
+#include <linux/types.h>
+
+#include "clk.h"
+
+enum clk_sel {
+	LOW_SPEED_IO_SEL,
+	NON_IO_SEL,
+	FAST_SEL,
+	AUDIO_SEL,
+	VIDEO_SEL,
+	TPM_SEL,
+	CKO1_SEL,
+	CKO2_SEL,
+	MISC_SEL,
+	MAX_SEL
+};
+
+static const char *parent_names[MAX_SEL][4] = {
+	{"osc_24m", "sys_pll_pfd0_div2", "sys_pll_pfd1_div2", "video_pll"},
+	{"osc_24m", "sys_pll_pfd0_div2", "sys_pll_pfd1_div2", "sys_pll_pfd2_div2"},
+	{"osc_24m", "sys_pll_pfd0", "sys_pll_pfd1", "sys_pll_pfd2"},
+	{"osc_24m", "audio_pll", "video_pll", "clk_ext1"},
+	{"osc_24m", "audio_pll", "video_pll", "sys_pll_pfd0"},
+	{"osc_24m", "sys_pll_pfd0", "audio_pll", "clk_ext1"},
+	{"osc_24m", "sys_pll_pfd0", "sys_pll_pfd1", "audio_pll"},
+	{"osc_24m", "sys_pll_pfd0", "sys_pll_pfd1", "video_pll"},
+	{"osc_24m", "audio_pll", "video_pll", "sys_pll_pfd2"},
+};
+
+struct imx93_clk_root {
+	u32 clk;
+	char *name;
+	u32 off;
+	enum clk_sel sel;
+	unsigned long flags;
+} root_array[] = {
+	{ IMX93_CLK_A55_PERIPH,		"a55_periph_root",	0x0000,	FAST_SEL, CLK_IS_CRITICAL },
+	{ IMX93_CLK_A55_MTR_BUS,	"a55_mtr_bus_root",	0x0080,	LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
+	{ IMX93_CLK_A55,		"a55_root",		0x0100,	FAST_SEL, CLK_IS_CRITICAL },
+	{ IMX93_CLK_M33,		"m33_root",		0x0180,	LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
+	{ IMX93_CLK_BUS_WAKEUP,		"bus_wakeup_root",	0x0280,	LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
+	{ IMX93_CLK_BUS_AON,		"bus_aon_root",		0x0300,	LOW_SPEED_IO_SEL, CLK_IS_CRITICAL },
+	{ IMX93_CLK_WAKEUP_AXI,		"wakeup_axi_root",	0x0380,	FAST_SEL, CLK_IS_CRITICAL },
+	{ IMX93_CLK_SWO_TRACE,		"swo_trace_root",	0x0400,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_M33_SYSTICK,	"m33_systick_root",	0x0480,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_FLEXIO1,		"flexio1_root",		0x0500,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_FLEXIO2,		"flexio2_root",		0x0580,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPIT1,		"lpit1_root",		0x0600,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPIT2,		"lpit2_root",		0x0680,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPTMR1,		"lptmr1_root",		0x0700,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPTMR2,		"lptmr2_root",		0x0780,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_TPM1,		"tpm1_root",		0x0800,	TPM_SEL, },
+	{ IMX93_CLK_TPM2,		"tpm2_root",		0x0880,	TPM_SEL, },
+	{ IMX93_CLK_TPM3,		"tpm3_root",		0x0900,	TPM_SEL, },
+	{ IMX93_CLK_TPM4,		"tpm4_root",		0x0980,	TPM_SEL, },
+	{ IMX93_CLK_TPM5,		"tpm5_root",		0x0a00,	TPM_SEL, },
+	{ IMX93_CLK_TPM6,		"tpm6_root",		0x0a80,	TPM_SEL, },
+	{ IMX93_CLK_FLEXSPI1,		"flexspi1_root",	0x0b00,	FAST_SEL, },
+	{ IMX93_CLK_CAN1,		"can1_root",		0x0b80,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_CAN2,		"can2_root",		0x0c00,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPUART1,		"lpuart1_root",		0x0c80,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPUART2,		"lpuart2_root",		0x0d00,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPUART3,		"lpuart3_root",		0x0d80,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPUART4,		"lpuart4_root",		0x0e00,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPUART5,		"lpuart5_root",		0x0e80,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPUART6,		"lpuart6_root",		0x0f00,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPUART7,		"lpuart7_root",		0x0f80,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPUART8,		"lpuart8_root",		0x1000,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPI2C1,		"lpi2c1_root",		0x1080,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPI2C2,		"lpi2c2_root",		0x1100,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPI2C3,		"lpi2c3_root",		0x1180,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPI2C4,		"lpi2c4_root",		0x1200,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPI2C5,		"lpi2c5_root",		0x1280,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPI2C6,		"lpi2c6_root",		0x1300,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPI2C7,		"lpi2c7_root",		0x1380,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPI2C8,		"lpi2c8_root",		0x1400,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPSPI1,		"lpspi1_root",		0x1480,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPSPI2,		"lpspi2_root",		0x1500,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPSPI3,		"lpspi3_root",		0x1580,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPSPI4,		"lpspi4_root",		0x1600,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPSPI5,		"lpspi5_root",		0x1680,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPSPI6,		"lpspi6_root",		0x1700,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPSPI7,		"lpspi7_root",		0x1780,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_LPSPI8,		"lpspi8_root",		0x1800,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_I3C1,		"i3c1_root",		0x1880,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_I3C2,		"i3c2_root",		0x1900,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_USDHC1,		"usdhc1_root",		0x1980,	FAST_SEL, },
+	{ IMX93_CLK_USDHC2,		"usdhc2_root",		0x1a00,	FAST_SEL, },
+	{ IMX93_CLK_USDHC3,		"usdhc3_root",		0x1a80,	FAST_SEL, },
+	{ IMX93_CLK_SAI1,		"sai1_root",		0x1b00,	AUDIO_SEL, },
+	{ IMX93_CLK_SAI2,		"sai2_root",		0x1b80,	AUDIO_SEL, },
+	{ IMX93_CLK_SAI3,		"sai3_root",		0x1c00,	AUDIO_SEL, },
+	{ IMX93_CLK_CCM_CKO1,		"ccm_cko1_root",	0x1c80,	CKO1_SEL, },
+	{ IMX93_CLK_CCM_CKO2,		"ccm_cko2_root",	0x1d00,	CKO2_SEL, },
+	{ IMX93_CLK_CCM_CKO3,		"ccm_cko3_root",	0x1d80,	CKO1_SEL, },
+	{ IMX93_CLK_CCM_CKO4,		"ccm_cko4_root",	0x1e00,	CKO2_SEL, },
+	{ IMX93_CLK_HSIO,		"hsio_root",		0x1e80,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_HSIO_USB_TEST_60M,	"hsio_usb_test_60m_root", 0x1f00, LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_HSIO_ACSCAN_80M,	"hsio_acscan_80m_root",	0x1f80,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_HSIO_ACSCAN_480M,	"hsio_acscan_480m_root", 0x2000, MISC_SEL, },
+	{ IMX93_CLK_ML_APB,		"ml_apb_root",		0x2180,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_ML,			"ml_root",		0x2200,	FAST_SEL, },
+	{ IMX93_CLK_MEDIA_AXI,		"media_axi_root",	0x2280,	FAST_SEL, },
+	{ IMX93_CLK_MEDIA_APB,		"media_apb_root",	0x2300,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_MEDIA_LDB,		"media_ldb_root",	0x2380,	VIDEO_SEL, },
+	{ IMX93_CLK_MEDIA_DISP_PIX,	"media_disp_pix_root",	0x2400,	VIDEO_SEL, },
+	{ IMX93_CLK_CAM_PIX,		"cam_pix_root",		0x2480,	VIDEO_SEL, },
+	{ IMX93_CLK_MIPI_TEST_BYTE,	"mipi_test_byte_root",	0x2500,	VIDEO_SEL, },
+	{ IMX93_CLK_MIPI_PHY_CFG,	"mipi_phy_cfg_root",	0x2580,	VIDEO_SEL, },
+	{ IMX93_CLK_ADC,		"adc_root",		0x2700,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_PDM,		"pdm_root",		0x2780,	AUDIO_SEL, },
+	{ IMX93_CLK_TSTMR1,		"tstmr1_root",		0x2800,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_TSTMR2,		"tstmr2_root",		0x2880,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_MQS1,		"mqs1_root",		0x2900,	AUDIO_SEL, },
+	{ IMX93_CLK_MQS2,		"mqs2_root",		0x2980,	AUDIO_SEL, },
+	{ IMX93_CLK_AUDIO_XCVR,		"audio_xcvr_root",	0x2a00,	NON_IO_SEL, },
+	{ IMX93_CLK_SPDIF,		"spdif_root",		0x2a80,	AUDIO_SEL, },
+	{ IMX93_CLK_ENET,		"enet_root",		0x2b00,	NON_IO_SEL, },
+	{ IMX93_CLK_ENET_TIMER1,	"enet_timer1_root",	0x2b80,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_ENET_TIMER2,	"enet_timer2_root",	0x2c00,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_ENET_REF,		"enet_ref_root",	0x2c80,	NON_IO_SEL, },
+	{ IMX93_CLK_ENET_REF_PHY,	"enet_ref_phy_root",	0x2d00,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_I3C1_SLOW,		"i3c1_slow_root",	0x2d80,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_I3C2_SLOW,		"i3c2_slow_root",	0x2e00,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_USB_PHY_BURUNIN,	"usb_phy_root",		0x2e80,	LOW_SPEED_IO_SEL, },
+	{ IMX93_CLK_PAL_CAME_SCAN,	"pal_came_scan_root",	0x2f00,	MISC_SEL, }
+};
+
+struct imx93_clk_ccgr {
+	u32 clk;
+	char *name;
+	char *parent_name;
+	u32 off;
+	unsigned long flags;
+} ccgr_array[] = {
+	{ IMX93_CLK_A55_GATE,		"a55",		"a55_root",		0x8000, CLK_IS_CRITICAL },
+	{ IMX93_CLK_CM33_GATE,		"cm33",		"m33_root",		0x8040, CLK_IS_CRITICAL },
+	{ IMX93_CLK_ADC1_GATE,		"adc1",		"osc_24m",		0x82c0, },
+	{ IMX93_CLK_WDOG1_GATE,		"wdog1",	"osc_24m",		0x8300, },
+	{ IMX93_CLK_WDOG2_GATE,		"wdog2",	"osc_24m",		0x8340, },
+	{ IMX93_CLK_WDOG3_GATE,		"wdog3",	"osc_24m",		0x8380, },
+	{ IMX93_CLK_WDOG4_GATE,		"wdog4",	"osc_24m",		0x83c0, },
+	{ IMX93_CLK_WDOG5_GATE,		"wdog5",	"osc_24m",		0x8400, },
+	{ IMX93_CLK_SEMA1_GATE,		"sema1",	"bus_aon_root",		0x8440, },
+	{ IMX93_CLK_SEMA2_GATE,		"sema2",	"bus_wakeup_root",	0x8480, },
+	{ IMX93_CLK_MU_A_GATE,		"mu_a",		"bus_aon_root",		0x84c0, },
+	{ IMX93_CLK_MU_B_GATE,		"mu_b",		"bus_aon_root",		0x8500, },
+	{ IMX93_CLK_EDMA1_GATE,		"edma1",	"wakeup_axi_root",	0x8540, },
+	{ IMX93_CLK_EDMA2_GATE,		"edma2",	"wakeup_axi_root",	0x8580, },
+	{ IMX93_CLK_FLEXSPI1_GATE,	"flexspi",	"flexspi_root",		0x8640, },
+	{ IMX93_CLK_GPIO1_GATE,		"gpio1",	"m33_root",		0x8880, },
+	{ IMX93_CLK_GPIO2_GATE,		"gpio2",	"bus_wakeup_root",	0x88c0, },
+	{ IMX93_CLK_GPIO3_GATE,		"gpio3",	"bus_wakeup_root",	0x8900, },
+	{ IMX93_CLK_GPIO4_GATE,		"gpio4",	"bus_wakeup_root",	0x8940, },
+	{ IMX93_CLK_FLEXIO1_GATE,	"flexio1",	"flexio1_root",		0x8980, },
+	{ IMX93_CLK_FLEXIO2_GATE,	"flexio2",	"flexio2_root",		0x89c0, },
+	{ IMX93_CLK_LPIT1_GATE,		"lpit1",	"lpit1_root",		0x8a00, },
+	{ IMX93_CLK_LPIT2_GATE,		"lpit2",	"lpit2_root",		0x8a40, },
+	{ IMX93_CLK_LPTMR1_GATE,	"lptmr1",	"lptmr1_root",		0x8a80, },
+	{ IMX93_CLK_LPTMR2_GATE,	"lptmr2",	"lptmr2_root",		0x8ac0, },
+	{ IMX93_CLK_TPM1_GATE,		"tpm1",		"tpm1_root",		0x8b00, },
+	{ IMX93_CLK_TPM2_GATE,		"tpm2",		"tpm2_root",		0x8b40, },
+	{ IMX93_CLK_TPM3_GATE,		"tpm3",		"tpm3_root",		0x8b80, },
+	{ IMX93_CLK_TPM4_GATE,		"tpm4",		"tpm4_root",		0x8bc0, },
+	{ IMX93_CLK_TPM5_GATE,		"tpm5",		"tpm5_root",		0x8c00, },
+	{ IMX93_CLK_TPM6_GATE,		"tpm6",		"tpm6_root",		0x8c40, },
+	{ IMX93_CLK_CAN1_GATE,		"can1",		"can1_root",		0x8c80, },
+	{ IMX93_CLK_CAN2_GATE,		"can2",		"can2_root",		0x8cc0, },
+	{ IMX93_CLK_LPUART1_GATE,	"lpuart1",	"lpuart1_root",		0x8d00, },
+	{ IMX93_CLK_LPUART2_GATE,	"lpuart2",	"lpuart2_root",		0x8d40, },
+	{ IMX93_CLK_LPUART3_GATE,	"lpuart3",	"lpuart3_root",		0x8d80, },
+	{ IMX93_CLK_LPUART4_GATE,	"lpuart4",	"lpuart4_root",		0x8dc0, },
+	{ IMX93_CLK_LPUART5_GATE,	"lpuart5",	"lpuart5_root",		0x8e00, },
+	{ IMX93_CLK_LPUART6_GATE,	"lpuart6",	"lpuart6_root",		0x8e40, },
+	{ IMX93_CLK_LPUART7_GATE,	"lpuart7",	"lpuart7_root",		0x8e80, },
+	{ IMX93_CLK_LPUART8_GATE,	"lpuart8",	"lpuart8_root",		0x8ec0, },
+	{ IMX93_CLK_LPI2C1_GATE,	"lpi2c1",	"lpi2c1_root",		0x8f00, },
+	{ IMX93_CLK_LPI2C2_GATE,	"lpi2c2",	"lpi2c2_root",		0x8f40, },
+	{ IMX93_CLK_LPI2C3_GATE,	"lpi2c3",	"lpi2c3_root",		0x8f80, },
+	{ IMX93_CLK_LPI2C4_GATE,	"lpi2c4",	"lpi2c4_root",		0x8fc0, },
+	{ IMX93_CLK_LPI2C5_GATE,	"lpi2c5",	"lpi2c5_root",		0x9000, },
+	{ IMX93_CLK_LPI2C6_GATE,	"lpi2c6",	"lpi2c6_root",		0x9040, },
+	{ IMX93_CLK_LPI2C7_GATE,	"lpi2c7",	"lpi2c7_root",		0x9080, },
+	{ IMX93_CLK_LPI2C8_GATE,	"lpi2c8",	"lpi2c8_root",		0x90c0, },
+	{ IMX93_CLK_LPSPI1_GATE,	"lpspi1",	"lpspi1_root",		0x9100, },
+	{ IMX93_CLK_LPSPI2_GATE,	"lpspi2",	"lpspi2_root",		0x9140, },
+	{ IMX93_CLK_LPSPI3_GATE,	"lpspi3",	"lpspi3_root",		0x9180, },
+	{ IMX93_CLK_LPSPI4_GATE,	"lpspi4",	"lpspi4_root",		0x91c0, },
+	{ IMX93_CLK_LPSPI5_GATE,	"lpspi5",	"lpspi5_root",		0x9200, },
+	{ IMX93_CLK_LPSPI6_GATE,	"lpspi6",	"lpspi6_root",		0x9240, },
+	{ IMX93_CLK_LPSPI7_GATE,	"lpspi7",	"lpspi7_root",		0x9280, },
+	{ IMX93_CLK_LPSPI8_GATE,	"lpspi8",	"lpspi8_root",		0x92c0, },
+	{ IMX93_CLK_I3C1_GATE,		"i3c1",		"i3c1_root",		0x9300, },
+	{ IMX93_CLK_I3C2_GATE,		"i3c2",		"i3c2_root",		0x9340, },
+	{ IMX93_CLK_USDHC1_GATE,	"usdhc1",	"usdhc1_root",		0x9380, },
+	{ IMX93_CLK_USDHC2_GATE,	"usdhc2",	"usdhc2_root",		0x93c0, },
+	{ IMX93_CLK_USDHC3_GATE,	"usdhc3",	"usdhc3_root",		0x9400, },
+	{ IMX93_CLK_SAI1_GATE,		"sai1",		"sai1_root",		0x9440, },
+	{ IMX93_CLK_SAI2_GATE,		"sai2",		"sai2_root",		0x9480, },
+	{ IMX93_CLK_SAI3_GATE,		"sai3",		"sai3_root",		0x94c0, },
+	{ IMX93_CLK_MIPI_CSI_GATE,	"mipi_csi",	"media_apb_root",	0x9580, },
+	{ IMX93_CLK_MIPI_DSI_GATE,	"mipi_dsi",	"media_apb_root",	0x95c0, },
+	{ IMX93_CLK_LVDS_GATE,		"lvds",		"media_ldb_root",	0x9600, },
+	{ IMX93_CLK_LCDIF_GATE,		"lcdif",	"media_apb_root",	0x9640, },
+	{ IMX93_CLK_PXP_GATE,		"pxp",		"media_apb_root",	0x9680, },
+	{ IMX93_CLK_ISI_GATE,		"isi",		"media_apb_root",	0x96c0, },
+	{ IMX93_CLK_NIC_MEDIA_GATE,	"nic_media",	"media_apb_root",	0x9700, },
+	{ IMX93_CLK_USB_CONTROLLER_GATE, "usb_controller", "hsio_root",		0x9a00, },
+	{ IMX93_CLK_USB_TEST_60M_GATE,	"usb_test_60m",	"hsio_usb_test_60m_root", 0x9a40, },
+	{ IMX93_CLK_HSIO_TROUT_24M_GATE, "hsio_trout_24m", "osc_24m",		0x9a80, },
+	{ IMX93_CLK_PDM_GATE,		"pdm",		"pdm_root",		0x9ac0, },
+	{ IMX93_CLK_MQS1_GATE,		"mqs1",		"sai1_root",		0x9b00, },
+	{ IMX93_CLK_MQS2_GATE,		"mqs2",		"sai3_root",		0x9b40, },
+	{ IMX93_CLK_AUD_XCVR_GATE,	"aud_xcvr",	"audio_xcvr_root",	0x9b80, },
+	{ IMX93_CLK_SPDIF_GATE,		"spdif",	"spdif_root",		0x9c00, },
+	{ IMX93_CLK_HSIO_32K_GATE,	"hsio_32k",	"osc_32k",		0x9dc0, },
+	{ IMX93_CLK_ENET1_GATE,		"enet1",	"enet_root",		0x9e00, },
+	{ IMX93_CLK_ENET_QOS_GATE,	"enet_qos",	"wakeup_axi_root",	0x9e40, },
+	{ IMX93_CLK_SYS_CNT_GATE,	"sys_cnt",	"osc_24m",		0x9e80, },
+	{ IMX93_CLK_TSTMR1_GATE,	"tstmr1",	"bus_aon_root",		0x9ec0, },
+	{ IMX93_CLK_TSTMR2_GATE,	"tstmr2",	"bus_wakeup_root",	0x9f00, },
+	{ IMX93_CLK_TMC_GATE,		"tmc",		"osc_24m",		0x9f40, },
+	{ IMX93_CLK_PMRO_GATE,		"pmro",		"osc_24m",		0x9f80, }
+};
+
+static struct clk_hw_onecell_data *clk_hw_data;
+static struct clk_hw **clks;
+
+static int imx93_clocks_probe(struct platform_device *pdev)
+{
+	struct device *dev = &pdev->dev;
+	struct device_node *np = dev->of_node;
+	struct imx93_clk_root *root;
+	struct imx93_clk_ccgr *ccgr;
+	void __iomem *base = NULL;
+	int i, ret;
+
+	clk_hw_data = kzalloc(struct_size(clk_hw_data, hws,
+					  IMX93_CLK_END), GFP_KERNEL);
+	if (WARN_ON(!clk_hw_data))
+		return -ENOMEM;
+
+	clk_hw_data->num = IMX93_CLK_END;
+	clks = clk_hw_data->hws;
+
+	clks[IMX93_CLK_DUMMY] = imx_clk_hw_fixed("dummy", 0);
+	clks[IMX93_CLK_24M] = imx_obtain_fixed_clk_hw(np, "osc_24m");
+	clks[IMX93_CLK_32K] = imx_obtain_fixed_clk_hw(np, "osc_32k");
+	clks[IMX93_CLK_EXT1] = imx_obtain_fixed_clk_hw(np, "clk_ext1");
+
+	clks[IMX93_CLK_SYS_PLL_PFD0] = imx_clk_hw_fixed("sys_pll_pfd0", 1000000000);
+	clks[IMX93_CLK_SYS_PLL_PFD0_DIV2] = imx_clk_hw_fixed_factor("sys_pll_pfd0_div2",
+								    "sys_pll_pfd0", 1, 2);
+	clks[IMX93_CLK_SYS_PLL_PFD1] = imx_clk_hw_fixed("sys_pll_pfd1", 800000000);
+	clks[IMX93_CLK_SYS_PLL_PFD1_DIV2] = imx_clk_hw_fixed_factor("sys_pll_pfd1_div2",
+								    "sys_pll_pfd1", 1, 2);
+	clks[IMX93_CLK_SYS_PLL_PFD2] = imx_clk_hw_fixed("sys_pll_pfd2", 625000000);
+	clks[IMX93_CLK_SYS_PLL_PFD2_DIV2] = imx_clk_hw_fixed_factor("sys_pll_pfd2_div2",
+								    "sys_pll_pfd2", 1, 2);
+
+	np = of_find_compatible_node(NULL, NULL, "fsl,imx93-anatop");
+	base = of_iomap(np, 0);
+	of_node_put(np);
+	if (WARN_ON(!base))
+		return -ENOMEM;
+
+	clks[IMX93_CLK_AUDIO_PLL] = imx_clk_fracn_gppll("audio_pll", "osc_24m", base + 0x1200,
+							&imx_fracn_gppll);
+	clks[IMX93_CLK_VIDEO_PLL] = imx_clk_fracn_gppll("video_pll", "osc_24m", base + 0x1400,
+							&imx_fracn_gppll);
+
+	np = dev->of_node;
+	base = devm_platform_ioremap_resource(pdev, 0);
+	if (WARN_ON(IS_ERR(base)))
+		return PTR_ERR(base);
+
+	for (i = 0; i < ARRAY_SIZE(root_array); i++) {
+		root = &root_array[i];
+		clks[root->clk] = imx93_clk_composite_flags(root->name,
+							    parent_names[root->sel],
+							    4, base + root->off,
+							    root->flags);
+	}
+
+	for (i = 0; i < ARRAY_SIZE(ccgr_array); i++) {
+		ccgr = &ccgr_array[i];
+		clks[ccgr->clk] = imx_clk_hw_gate4_flags(ccgr->name,
+							 ccgr->parent_name,
+							 base + ccgr->off, 0,
+							 ccgr->flags);
+	}
+
+	imx_check_clk_hws(clks, IMX93_CLK_END);
+
+	ret = of_clk_add_hw_provider(np, of_clk_hw_onecell_get, clk_hw_data);
+	if (ret < 0) {
+		dev_err(dev, "failed to register clks for i.MX93\n");
+		goto unregister_hws;
+	}
+
+	return 0;
+
+unregister_hws:
+	imx_unregister_hw_clocks(clks, IMX93_CLK_END);
+
+	return ret;
+}
+
+static const struct of_device_id imx93_clk_of_match[] = {
+	{ .compatible = "fsl,imx93-ccm" },
+	{ /* Sentinel */ },
+};
+MODULE_DEVICE_TABLE(of, imx93_clk_of_match);
+
+static struct platform_driver imx93_clk_driver = {
+	.probe = imx93_clocks_probe,
+	.driver = {
+		.name = "imx93-ccm",
+		.suppress_bind_attrs = true,
+		.of_match_table = of_match_ptr(imx93_clk_of_match),
+	},
+};
+module_platform_driver(imx93_clk_driver);