Message ID | 20221001030403.27659-1-quic_molvera@quicinc.com |
---|---|
Headers | show |
Series | clk: qcom: Add clocks for the QDU1000 and QRU1000 SoCs | expand |
On Sat, 1 Oct 2022 at 06:05, Melody Olvera <quic_molvera@quicinc.com> wrote: > > Add support for RMPh clocks for QDU1000 and QRU1000 SoCs. > > Signed-off-by: Melody Olvera <quic_molvera@quicinc.com> > --- > drivers/clk/qcom/clk-rpmh.c | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/drivers/clk/qcom/clk-rpmh.c b/drivers/clk/qcom/clk-rpmh.c > index c07cab6905cb..27d11ffac71e 100644 > --- a/drivers/clk/qcom/clk-rpmh.c > +++ b/drivers/clk/qcom/clk-rpmh.c > @@ -628,6 +628,18 @@ static const struct clk_rpmh_desc clk_rpmh_sdx65 = { > .num_clks = ARRAY_SIZE(sdx65_rpmh_clocks), > }; > > +DEFINE_CLK_RPMH_ARC(qdru1000, bi_tcxo, bi_tcxo_ao, "xo.lvl", 0x3, 1); > + > +static struct clk_hw *qdru1000_rpmh_clocks[] = { > + [RPMH_CXO_CLK] = &qdru1000_bi_tcxo.hw, > + [RPMH_CXO_CLK_A] = &qdru1000_bi_tcxo_ao.hw, > +}; > + > +static const struct clk_rpmh_desc clk_rpmh_qdru1000 = { > + .clks = qdru1000_rpmh_clocks, > + .num_clks = ARRAY_SIZE(qdru1000_rpmh_clocks), > +}; > + > static struct clk_hw *of_clk_rpmh_hw_get(struct of_phandle_args *clkspec, > void *data) > { > @@ -723,6 +735,8 @@ static const struct of_device_id clk_rpmh_match_table[] = { > { .compatible = "qcom,sm8350-rpmh-clk", .data = &clk_rpmh_sm8350}, > { .compatible = "qcom,sm8450-rpmh-clk", .data = &clk_rpmh_sm8450}, > { .compatible = "qcom,sc7280-rpmh-clk", .data = &clk_rpmh_sc7280}, > + { .compatible = "qcom,qdu1000-rpmh-clk", .data = &clk_rpmh_qdru1000}, > + { .compatible = "qcom,qru1000-rpmh-clk", .data = &clk_rpmh_qdru1000}, Also the list isn't fully sorted, let's target that. Please move your compat strings into the beginning. > { } > }; > MODULE_DEVICE_TABLE(of, clk_rpmh_match_table); > -- > 2.37.3 >
On Sat, 1 Oct 2022 at 06:05, Melody Olvera <quic_molvera@quicinc.com> wrote: > > From: Taniya Das <quic_tdas@quicinc.com> > > Add Global Clock Controller (GCC) support for QDU1000 and QRU1000 SoCs. > > Signed-off-by: Taniya Das <quic_tdas@quicinc.com> > Signed-off-by: Melody Olvera <quic_molvera@quicinc.com> > --- > drivers/clk/qcom/Kconfig | 8 + > drivers/clk/qcom/Makefile | 1 + > drivers/clk/qcom/clk-branch.c | 5 + > drivers/clk/qcom/clk-branch.h | 2 + > drivers/clk/qcom/gcc-qdru1000.c | 2649 +++++++++++++++++++++++++++++++ > 5 files changed, 2665 insertions(+) > create mode 100644 drivers/clk/qcom/gcc-qdru1000.c > > diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig > index 1cf1ef70e347..195c018c6f87 100644 > --- a/drivers/clk/qcom/Kconfig > +++ b/drivers/clk/qcom/Kconfig > @@ -696,6 +696,14 @@ config SM_GCC_8450 > Say Y if you want to use peripheral devices such as UART, > SPI, I2C, USB, SD/UFS, PCIe etc. > > +config QDRU_GCC_1000 > + tristate "QDU1000/QRU1000 Global Clock Controller" > + select QCOM_GDSC > + help > + Support for the global clock controller on QDU1000 and > + QRU1000 devices. Say Y if you want to use peripheral > + devices such as UART, SPI, I2C, USB, SD, PCIe, etc. > + > config SM_GPUCC_6350 > tristate "SM6350 Graphics Clock Controller" > select SM_GCC_6350 > diff --git a/drivers/clk/qcom/Makefile b/drivers/clk/qcom/Makefile > index fbcf04073f07..080253a338e7 100644 > --- a/drivers/clk/qcom/Makefile > +++ b/drivers/clk/qcom/Makefile > @@ -100,6 +100,7 @@ obj-$(CONFIG_SM_GCC_8150) += gcc-sm8150.o > obj-$(CONFIG_SM_GCC_8250) += gcc-sm8250.o > obj-$(CONFIG_SM_GCC_8350) += gcc-sm8350.o > obj-$(CONFIG_SM_GCC_8450) += gcc-sm8450.o > +obj-$(CONFIG_QDRU_GCC_1000) += gcc-qdru1000.o > obj-$(CONFIG_SM_GPUCC_6350) += gpucc-sm6350.o > obj-$(CONFIG_SM_GPUCC_8150) += gpucc-sm8150.o > obj-$(CONFIG_SM_GPUCC_8250) += gpucc-sm8250.o > diff --git a/drivers/clk/qcom/clk-branch.c b/drivers/clk/qcom/clk-branch.c > index f869fc6aaed6..b5dc1f4ef277 100644 > --- a/drivers/clk/qcom/clk-branch.c > +++ b/drivers/clk/qcom/clk-branch.c > @@ -1,6 +1,7 @@ > // SPDX-License-Identifier: GPL-2.0 > /* > * Copyright (c) 2013, The Linux Foundation. All rights reserved. > + * Copyright (c) 2022, Qualcomm Innovation Center, Inc. All rights reserved. > */ > > #include <linux/kernel.h> > @@ -56,6 +57,10 @@ static bool clk_branch2_check_halt(const struct clk_branch *br, bool enabling) > > if (enabling) { > val &= mask; > + > + if (br->halt_check == BRANCH_HALT_INVERT) > + return (val & BRANCH_CLK_OFF) == BRANCH_CLK_OFF; > + > return (val & BRANCH_CLK_OFF) == 0 || > val == BRANCH_NOC_FSM_STATUS_ON; > } else { > diff --git a/drivers/clk/qcom/clk-branch.h b/drivers/clk/qcom/clk-branch.h > index 17a58119165e..4ac1debeb91e 100644 > --- a/drivers/clk/qcom/clk-branch.h > +++ b/drivers/clk/qcom/clk-branch.h > @@ -1,5 +1,6 @@ > /* SPDX-License-Identifier: GPL-2.0 */ > /* Copyright (c) 2013, The Linux Foundation. All rights reserved. */ > +/* Copyright (c) 2022, Qualcomm Innovation Center, Inc. All rights reserved. */ > > #ifndef __QCOM_CLK_BRANCH_H__ > #define __QCOM_CLK_BRANCH_H__ > @@ -33,6 +34,7 @@ struct clk_branch { > #define BRANCH_HALT_ENABLE_VOTED (BRANCH_HALT_ENABLE | BRANCH_VOTED) > #define BRANCH_HALT_DELAY 2 /* No bit to check; just delay */ > #define BRANCH_HALT_SKIP 3 /* Don't check halt bit */ > +#define BRANCH_HALT_INVERT 4 /* Invert logic for halt bit */ > > struct clk_regmap clkr; > }; I'm tempted to ask to move these two chunks to a separate commit. > diff --git a/drivers/clk/qcom/gcc-qdru1000.c b/drivers/clk/qcom/gcc-qdru1000.c > new file mode 100644 > index 000000000000..4bc54f7b43a0 > --- /dev/null > +++ b/drivers/clk/qcom/gcc-qdru1000.c > @@ -0,0 +1,2649 @@ > +// SPDX-License-Identifier: GPL-2.0-only > +/* > + * Copyright (c) 2022, Qualcomm Innovation Center, Inc. All rights reserved. > + */ > + > +#include <linux/clk-provider.h> > +#include <linux/module.h> > +#include <linux/of_device.h> > +#include <linux/regmap.h> > + > +#include <dt-bindings/clock/qcom,gcc-qdru1000.h> > + > +#include "clk-alpha-pll.h" > +#include "clk-branch.h" > +#include "clk-rcg.h" > +#include "clk-regmap.h" > +#include "clk-regmap-divider.h" > +#include "clk-regmap-mux.h" > +#include "clk-regmap-phy-mux.h" > +#include "reset.h" > + > +enum { > + P_BI_TCXO, > + P_GCC_GPLL0_OUT_EVEN, > + P_GCC_GPLL0_OUT_MAIN, > + P_GCC_GPLL1_OUT_MAIN, > + P_GCC_GPLL2_OUT_MAIN, > + P_GCC_GPLL3_OUT_MAIN, > + P_GCC_GPLL4_OUT_MAIN, > + P_GCC_GPLL5_OUT_MAIN, > + P_GCC_GPLL6_OUT_MAIN, > + P_GCC_GPLL7_OUT_MAIN, > + P_GCC_GPLL8_OUT_MAIN, > + P_PCIE_0_PHY_AUX_CLK, > + P_PCIE_0_PIPE_CLK, > + P_SLEEP_CLK, > + P_USB3_PHY_WRAPPER_GCC_USB30_PIPE_CLK, > +}; > + > +static struct clk_alpha_pll gcc_gpll0 = { > + .offset = 0x0, > + .regs = clk_alpha_pll_regs[CLK_ALPHA_PLL_TYPE_LUCID_EVO], > + .clkr = { > + .enable_reg = 0x62018, > + .enable_mask = BIT(0), > + .hw.init = &(const struct clk_init_data){ > + .name = "gcc_gpll0", > + .parent_data = &(const struct clk_parent_data){ > + .fw_name = "bi_tcxo", New drivers are requested to use .index rather than .fw_name. > + }, > + .num_parents = 1, > + .ops = &clk_alpha_pll_fixed_lucid_evo_ops, > + }, > + }, > +}; > + > +static const struct clk_div_table post_div_table_gcc_gpll0_out_even[] = { > + { 0x1, 2 }, > + { } No need for a sentinel here > +}; > + [skipped] > + > +static int gcc_qdru1000_probe(struct platform_device *pdev) > +{ > + struct regmap *regmap; > + int ret; > + > + regmap = qcom_cc_map(pdev, &gcc_qdru1000_desc); > + if (IS_ERR(regmap)) > + return PTR_ERR(regmap); > + > + /* Update FORCE_MEM_CORE_ON for gcc_pcie_0_mstr_axi_clk */ > + regmap_update_bits(regmap, 0x9d024, BIT(14), BIT(14)); > + > + ret = qcom_cc_register_rcg_dfs(regmap, gcc_dfs_clocks, > + ARRAY_SIZE(gcc_dfs_clocks)); > + if (ret) > + return ret; > + > + ret = qcom_cc_really_probe(pdev, &gcc_qdru1000_desc, regmap); > + if (ret) { > + dev_err(&pdev->dev, "Failed to register GCC clocks\n"); > + return ret; > + } if (ret) return dev_err_probe(....); > + > + dev_info(&pdev->dev, "Registered GCC clocks\n"); No need to spam here. > + > + return ret; > +} > + > +static struct platform_driver gcc_qdru1000_driver = { > + .probe = gcc_qdru1000_probe, > + .driver = { > + .name = "gcc-qdru1000", > + .of_match_table = gcc_qdru1000_match_table, > + }, > +}; > + > +static int __init gcc_qdru1000_init(void) > +{ > + return platform_driver_register(&gcc_qdru1000_driver); > +} > +subsys_initcall(gcc_qdru1000_init); > + > +static void __exit gcc_qdru1000_exit(void) > +{ > + platform_driver_unregister(&gcc_qdru1000_driver); > +} > +module_exit(gcc_qdru1000_exit); > + > +MODULE_DESCRIPTION("QTI GCC QDRU1000 Driver"); > +MODULE_LICENSE("GPL"); > -- > 2.37.3 >
On 10/1/2022 2:10 AM, Dmitry Baryshkov wrote: > On Sat, 1 Oct 2022 at 06:05, Melody Olvera <quic_molvera@quicinc.com> wrote: >> From: Taniya Das <quic_tdas@quicinc.com> >> >> Add Global Clock Controller (GCC) support for QDU1000 and QRU1000 SoCs. >> >> Signed-off-by: Taniya Das <quic_tdas@quicinc.com> >> Signed-off-by: Melody Olvera <quic_molvera@quicinc.com> >> --- >> drivers/clk/qcom/Kconfig | 8 + >> drivers/clk/qcom/Makefile | 1 + >> drivers/clk/qcom/clk-branch.c | 5 + >> drivers/clk/qcom/clk-branch.h | 2 + >> drivers/clk/qcom/gcc-qdru1000.c | 2649 +++++++++++++++++++++++++++++++ >> 5 files changed, 2665 insertions(+) >> create mode 100644 drivers/clk/qcom/gcc-qdru1000.c >> >> diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig >> index 1cf1ef70e347..195c018c6f87 100644 >> --- a/drivers/clk/qcom/Kconfig >> +++ b/drivers/clk/qcom/Kconfig >> @@ -696,6 +696,14 @@ config SM_GCC_8450 >> Say Y if you want to use peripheral devices such as UART, >> SPI, I2C, USB, SD/UFS, PCIe etc. >> >> +config QDRU_GCC_1000 >> + tristate "QDU1000/QRU1000 Global Clock Controller" >> + select QCOM_GDSC >> + help >> + Support for the global clock controller on QDU1000 and >> + QRU1000 devices. Say Y if you want to use peripheral >> + devices such as UART, SPI, I2C, USB, SD, PCIe, etc. >> + >> config SM_GPUCC_6350 >> tristate "SM6350 Graphics Clock Controller" >> select SM_GCC_6350 >> diff --git a/drivers/clk/qcom/Makefile b/drivers/clk/qcom/Makefile >> index fbcf04073f07..080253a338e7 100644 >> --- a/drivers/clk/qcom/Makefile >> +++ b/drivers/clk/qcom/Makefile >> @@ -100,6 +100,7 @@ obj-$(CONFIG_SM_GCC_8150) += gcc-sm8150.o >> obj-$(CONFIG_SM_GCC_8250) += gcc-sm8250.o >> obj-$(CONFIG_SM_GCC_8350) += gcc-sm8350.o >> obj-$(CONFIG_SM_GCC_8450) += gcc-sm8450.o >> +obj-$(CONFIG_QDRU_GCC_1000) += gcc-qdru1000.o >> obj-$(CONFIG_SM_GPUCC_6350) += gpucc-sm6350.o >> obj-$(CONFIG_SM_GPUCC_8150) += gpucc-sm8150.o >> obj-$(CONFIG_SM_GPUCC_8250) += gpucc-sm8250.o >> diff --git a/drivers/clk/qcom/clk-branch.c b/drivers/clk/qcom/clk-branch.c >> index f869fc6aaed6..b5dc1f4ef277 100644 >> --- a/drivers/clk/qcom/clk-branch.c >> +++ b/drivers/clk/qcom/clk-branch.c >> @@ -1,6 +1,7 @@ >> // SPDX-License-Identifier: GPL-2.0 >> /* >> * Copyright (c) 2013, The Linux Foundation. All rights reserved. >> + * Copyright (c) 2022, Qualcomm Innovation Center, Inc. All rights reserved. >> */ >> >> #include <linux/kernel.h> >> @@ -56,6 +57,10 @@ static bool clk_branch2_check_halt(const struct clk_branch *br, bool enabling) >> >> if (enabling) { >> val &= mask; >> + >> + if (br->halt_check == BRANCH_HALT_INVERT) >> + return (val & BRANCH_CLK_OFF) == BRANCH_CLK_OFF; >> + >> return (val & BRANCH_CLK_OFF) == 0 || >> val == BRANCH_NOC_FSM_STATUS_ON; >> } else { >> diff --git a/drivers/clk/qcom/clk-branch.h b/drivers/clk/qcom/clk-branch.h >> index 17a58119165e..4ac1debeb91e 100644 >> --- a/drivers/clk/qcom/clk-branch.h >> +++ b/drivers/clk/qcom/clk-branch.h >> @@ -1,5 +1,6 @@ >> /* SPDX-License-Identifier: GPL-2.0 */ >> /* Copyright (c) 2013, The Linux Foundation. All rights reserved. */ >> +/* Copyright (c) 2022, Qualcomm Innovation Center, Inc. All rights reserved. */ >> >> #ifndef __QCOM_CLK_BRANCH_H__ >> #define __QCOM_CLK_BRANCH_H__ >> @@ -33,6 +34,7 @@ struct clk_branch { >> #define BRANCH_HALT_ENABLE_VOTED (BRANCH_HALT_ENABLE | BRANCH_VOTED) >> #define BRANCH_HALT_DELAY 2 /* No bit to check; just delay */ >> #define BRANCH_HALT_SKIP 3 /* Don't check halt bit */ >> +#define BRANCH_HALT_INVERT 4 /* Invert logic for halt bit */ >> >> struct clk_regmap clkr; >> }; > I'm tempted to ask to move these two chunks to a separate commit. That makes sense to me; will separate these into their own commit. > >> diff --git a/drivers/clk/qcom/gcc-qdru1000.c b/drivers/clk/qcom/gcc-qdru1000.c >> new file mode 100644 >> index 000000000000..4bc54f7b43a0 >> --- /dev/null >> +++ b/drivers/clk/qcom/gcc-qdru1000.c >> @@ -0,0 +1,2649 @@ >> +// SPDX-License-Identifier: GPL-2.0-only >> +/* >> + * Copyright (c) 2022, Qualcomm Innovation Center, Inc. All rights reserved. >> + */ >> + >> +#include <linux/clk-provider.h> >> +#include <linux/module.h> >> +#include <linux/of_device.h> >> +#include <linux/regmap.h> >> + >> +#include <dt-bindings/clock/qcom,gcc-qdru1000.h> >> + >> +#include "clk-alpha-pll.h" >> +#include "clk-branch.h" >> +#include "clk-rcg.h" >> +#include "clk-regmap.h" >> +#include "clk-regmap-divider.h" >> +#include "clk-regmap-mux.h" >> +#include "clk-regmap-phy-mux.h" >> +#include "reset.h" >> + >> +enum { >> + P_BI_TCXO, >> + P_GCC_GPLL0_OUT_EVEN, >> + P_GCC_GPLL0_OUT_MAIN, >> + P_GCC_GPLL1_OUT_MAIN, >> + P_GCC_GPLL2_OUT_MAIN, >> + P_GCC_GPLL3_OUT_MAIN, >> + P_GCC_GPLL4_OUT_MAIN, >> + P_GCC_GPLL5_OUT_MAIN, >> + P_GCC_GPLL6_OUT_MAIN, >> + P_GCC_GPLL7_OUT_MAIN, >> + P_GCC_GPLL8_OUT_MAIN, >> + P_PCIE_0_PHY_AUX_CLK, >> + P_PCIE_0_PIPE_CLK, >> + P_SLEEP_CLK, >> + P_USB3_PHY_WRAPPER_GCC_USB30_PIPE_CLK, >> +}; >> + >> +static struct clk_alpha_pll gcc_gpll0 = { >> + .offset = 0x0, >> + .regs = clk_alpha_pll_regs[CLK_ALPHA_PLL_TYPE_LUCID_EVO], >> + .clkr = { >> + .enable_reg = 0x62018, >> + .enable_mask = BIT(0), >> + .hw.init = &(const struct clk_init_data){ >> + .name = "gcc_gpll0", >> + .parent_data = &(const struct clk_parent_data){ >> + .fw_name = "bi_tcxo", > New drivers are requested to use .index rather than .fw_name. Got it; will fix. > >> + }, >> + .num_parents = 1, >> + .ops = &clk_alpha_pll_fixed_lucid_evo_ops, >> + }, >> + }, >> +}; >> + >> +static const struct clk_div_table post_div_table_gcc_gpll0_out_even[] = { >> + { 0x1, 2 }, >> + { } > No need for a sentinel here Understood. > >> +}; >> + > [skipped] > > >> + >> +static int gcc_qdru1000_probe(struct platform_device *pdev) >> +{ >> + struct regmap *regmap; >> + int ret; >> + >> + regmap = qcom_cc_map(pdev, &gcc_qdru1000_desc); >> + if (IS_ERR(regmap)) >> + return PTR_ERR(regmap); >> + >> + /* Update FORCE_MEM_CORE_ON for gcc_pcie_0_mstr_axi_clk */ >> + regmap_update_bits(regmap, 0x9d024, BIT(14), BIT(14)); >> + >> + ret = qcom_cc_register_rcg_dfs(regmap, gcc_dfs_clocks, >> + ARRAY_SIZE(gcc_dfs_clocks)); >> + if (ret) >> + return ret; >> + >> + ret = qcom_cc_really_probe(pdev, &gcc_qdru1000_desc, regmap); >> + if (ret) { >> + dev_err(&pdev->dev, "Failed to register GCC clocks\n"); >> + return ret; >> + } > if (ret) > return dev_err_probe(....); Got it. > >> + >> + dev_info(&pdev->dev, "Registered GCC clocks\n"); > > No need to spam here. Will remove. > >> + >> + return ret; >> +} >> + >> +static struct platform_driver gcc_qdru1000_driver = { >> + .probe = gcc_qdru1000_probe, >> + .driver = { >> + .name = "gcc-qdru1000", >> + .of_match_table = gcc_qdru1000_match_table, >> + }, >> +}; >> + >> +static int __init gcc_qdru1000_init(void) >> +{ >> + return platform_driver_register(&gcc_qdru1000_driver); >> +} >> +subsys_initcall(gcc_qdru1000_init); >> + >> +static void __exit gcc_qdru1000_exit(void) >> +{ >> + platform_driver_unregister(&gcc_qdru1000_driver); >> +} >> +module_exit(gcc_qdru1000_exit); >> + >> +MODULE_DESCRIPTION("QTI GCC QDRU1000 Driver"); >> +MODULE_LICENSE("GPL"); >> -- >> 2.37.3 >> Thanks, Melody
On 10/1/2022 2:03 AM, Dmitry Baryshkov wrote: > On Sat, 1 Oct 2022 at 06:05, Melody Olvera <quic_molvera@quicinc.com> wrote: >> Add support for RMPh clocks for QDU1000 and QRU1000 SoCs. >> >> Signed-off-by: Melody Olvera <quic_molvera@quicinc.com> >> --- >> drivers/clk/qcom/clk-rpmh.c | 14 ++++++++++++++ >> 1 file changed, 14 insertions(+) >> >> diff --git a/drivers/clk/qcom/clk-rpmh.c b/drivers/clk/qcom/clk-rpmh.c >> index c07cab6905cb..27d11ffac71e 100644 >> --- a/drivers/clk/qcom/clk-rpmh.c >> +++ b/drivers/clk/qcom/clk-rpmh.c >> @@ -628,6 +628,18 @@ static const struct clk_rpmh_desc clk_rpmh_sdx65 = { >> .num_clks = ARRAY_SIZE(sdx65_rpmh_clocks), >> }; >> >> +DEFINE_CLK_RPMH_ARC(qdru1000, bi_tcxo, bi_tcxo_ao, "xo.lvl", 0x3, 1); >> + >> +static struct clk_hw *qdru1000_rpmh_clocks[] = { >> + [RPMH_CXO_CLK] = &qdru1000_bi_tcxo.hw, >> + [RPMH_CXO_CLK_A] = &qdru1000_bi_tcxo_ao.hw, >> +}; >> + >> +static const struct clk_rpmh_desc clk_rpmh_qdru1000 = { >> + .clks = qdru1000_rpmh_clocks, >> + .num_clks = ARRAY_SIZE(qdru1000_rpmh_clocks), >> +}; >> + >> static struct clk_hw *of_clk_rpmh_hw_get(struct of_phandle_args *clkspec, >> void *data) >> { >> @@ -723,6 +735,8 @@ static const struct of_device_id clk_rpmh_match_table[] = { >> { .compatible = "qcom,sm8350-rpmh-clk", .data = &clk_rpmh_sm8350}, >> { .compatible = "qcom,sm8450-rpmh-clk", .data = &clk_rpmh_sm8450}, >> { .compatible = "qcom,sc7280-rpmh-clk", .data = &clk_rpmh_sc7280}, >> + { .compatible = "qcom,qdu1000-rpmh-clk", .data = &clk_rpmh_qdru1000}, >> + { .compatible = "qcom,qru1000-rpmh-clk", .data = &clk_rpmh_qdru1000}, > Also the list isn't fully sorted, let's target that. Please move your > compat strings into the beginning. Sure thing. > >> { } >> }; >> MODULE_DEVICE_TABLE(of, clk_rpmh_match_table); >> -- >> 2.37.3 >> Thanks, Melody