Hi Irving-CH,
On Thu, 2025-11-06 at 20:41 +0800, irving.ch.lin wrote:
> From: Irving-CH Lin <irving-ch.lin@mediatek.com>
>
> Add support for the MT8189 vlpcfg clock controller,
> which provides clock gate control for vlp domain IPs.
>
> Signed-off-by: Irving-CH Lin <irving-ch.lin@mediatek.com>
> ---
> drivers/clk/mediatek/Makefile | 2 +-
> drivers/clk/mediatek/clk-mt8189-vlpcfg.c | 121
> +++++++++++++++++++++++
> 2 files changed, 122 insertions(+), 1 deletion(-)
> create mode 100644 drivers/clk/mediatek/clk-mt8189-vlpcfg.c
>
> diff --git a/drivers/clk/mediatek/Makefile
> b/drivers/clk/mediatek/Makefile
> index 3b25df9e7b50..d9279b237b7b 100644
> --- a/drivers/clk/mediatek/Makefile
> +++ b/drivers/clk/mediatek/Makefile
> @@ -124,7 +124,7 @@ obj-$(CONFIG_COMMON_CLK_MT8188_VENCSYS) += clk-
> mt8188-venc.o
> obj-$(CONFIG_COMMON_CLK_MT8188_VPPSYS) += clk-mt8188-vpp0.o clk-
> mt8188-vpp1.o
> obj-$(CONFIG_COMMON_CLK_MT8188_WPESYS) += clk-mt8188-wpe.o
> obj-$(CONFIG_COMMON_CLK_MT8189) += clk-mt8189-apmixedsys.o clk-
> mt8189-topckgen.o \
> - clk-mt8189-vlpckgen.o
> + clk-mt8189-vlpckgen.o clk-mt8189-
> vlpcfg.o
> obj-$(CONFIG_COMMON_CLK_MT8192) += clk-mt8192-apmixedsys.o clk-
> mt8192.o
> obj-$(CONFIG_COMMON_CLK_MT8192_AUDSYS) += clk-mt8192-aud.o
> obj-$(CONFIG_COMMON_CLK_MT8192_CAMSYS) += clk-mt8192-cam.o
> diff --git a/drivers/clk/mediatek/clk-mt8189-vlpcfg.c
> b/drivers/clk/mediatek/clk-mt8189-vlpcfg.c
> new file mode 100644
> index 000000000000..0508237a2b41
> --- /dev/null
> +++ b/drivers/clk/mediatek/clk-mt8189-vlpcfg.c
> @@ -0,0 +1,121 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Copyright (c) 2025 MediaTek Inc.
> + * Author: Qiqi Wang <qiqi.wang@mediatek.com>
> + */
> +
> +#include <linux/clk-provider.h>
> +#include <linux/module.h>
> +#include <linux/of_device.h>
> +#include <linux/platform_device.h>
> +
> +#include "clk-mtk.h"
> +#include "clk-gate.h"
> +
> +#include <dt-bindings/clock/mediatek,mt8189-clk.h>
> +
> +static const struct mtk_gate_regs vlpcfg_ao_reg_cg_regs = {
> + .set_ofs = 0x0,
> + .clr_ofs = 0x0,
> + .sta_ofs = 0x0,
> +};
> +
> +#define GATE_VLPCFG_AO_REG(_id, _name, _parent, _shift) { \
> + .id = _id, \
> + .name = _name, \
> + .parent_name = _parent, \
> + .regs = &vlpcfg_ao_reg_cg_regs, \
> + .shift = _shift, \
> + .ops = &mtk_clk_gate_ops_no_setclr, \
> + }
You can use the GATE_MTK() macro from clk-gate.h to simplify this macro
code. It would give something like:
```
#define GATE_VLPCFG_AO_REG(_id, _name, _parent, _shift) \
GATE_MTK(_id, _name, _parent, &vlpcfg_ao_reg_cg_regs, _shift,
&mtk_clk_gate_ops_no_setclr)
```
There are other macros in your patch series that can be simplified with
GATE_MTK use as well.
> +
> +static const struct mtk_gate vlpcfg_ao_reg_clks[] = {
> + GATE_VLPCFG_AO_REG(CLK_VLPCFG_AO_APEINT_RX,
> "vlpcfg_ao_apeint_rx", "clk26m", 8),
> +};
> +
> +static const struct mtk_clk_desc vlpcfg_ao_reg_mcd = {
> + .clks = vlpcfg_ao_reg_clks,
> + .num_clks = ARRAY_SIZE(vlpcfg_ao_reg_clks),
> +};
> +
> +static const struct mtk_gate_regs vlpcfg_reg_cg_regs = {
> + .set_ofs = 0x4,
> + .clr_ofs = 0x4,
> + .sta_ofs = 0x4,
> +};
> +
> +#define GATE_VLPCFG_REG_FLAGS(_id, _name, _parent, _shift, _flags)
> { \
> + .id = _id, \
> + .name = _name, \
> + .parent_name = _parent, \
> + .regs = &vlpcfg_reg_cg_regs, \
> + .shift = _shift, \
> + .flags = _flags, \
> + .ops = &mtk_clk_gate_ops_no_setclr_inv, \
> + }
Similarly, the GATE_MTK_FLAGS() macro can be used to simplify this one
and other macros from your patch series.
Regards,
Louis-Alexis
> +
> +#define GATE_VLPCFG_REG(_id, _name, _parent, _shift) \
> + GATE_VLPCFG_REG_FLAGS(_id, _name, _parent, _shift, 0)
> +
> +static const struct mtk_gate vlpcfg_reg_clks[] = {
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_SCP, "vlpcfg_scp",
> + "vlp_scp_sel", 28, CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_RG_R_APXGPT_26M,
> "vlpcfg_r_apxgpt_26m",
> + "clk26m", 24, CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_DPMSRCK_TEST,
> "vlpcfg_dpmsrck_test",
> + "clk26m", 23, CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_RG_DPMSRRTC_TEST,
> "vlpcfg_dpmsrrtc_test",
> + "clk32k", 22, CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_DPMSRULP_TEST,
> "vlpcfg_dpmsrulp_test",
> + "osc_d10", 21, CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_SPMI_P_MST,
> "vlpcfg_spmi_p",
> + "vlp_spmi_p_sel", 20,
> CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_SPMI_P_MST_32K,
> "vlpcfg_spmi_p_32k",
> + "clk32k", 18, CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_PMIF_SPMI_P_SYS,
> "vlpcfg_pmif_spmi_p_sys",
> + "vlp_pwrap_ulposc_sel", 13,
> CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_PMIF_SPMI_P_TMR,
> "vlpcfg_pmif_spmi_p_tmr",
> + "vlp_pwrap_ulposc_sel", 12,
> CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG(CLK_VLPCFG_REG_PMIF_SPMI_M_SYS,
> "vlpcfg_pmif_spmi_m_sys",
> + "vlp_pwrap_ulposc_sel", 11),
> + GATE_VLPCFG_REG(CLK_VLPCFG_REG_PMIF_SPMI_M_TMR,
> "vlpcfg_pmif_spmi_m_tmr",
> + "vlp_pwrap_ulposc_sel", 10),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_DVFSRC,
> "vlpcfg_dvfsrc",
> + "vlp_dvfsrc_sel", 9, CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_PWM_VLP,
> "vlpcfg_pwm_vlp",
> + "vlp_pwm_vlp_sel", 8,
> CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_SRCK, "vlpcfg_srck",
> + "vlp_srck_sel", 7, CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_SSPM_F26M,
> "vlpcfg_sspm_f26m",
> + "vlp_sspm_f26m_sel", 4,
> CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_SSPM_F32K,
> "vlpcfg_sspm_f32k",
> + "clk32k", 3, CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_SSPM_ULPOSC,
> "vlpcfg_sspm_ulposc",
> + "vlp_sspm_ulposc_sel", 2,
> CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_VLP_32K_COM,
> "vlpcfg_vlp_32k_com",
> + "clk32k", 1, CLK_IS_CRITICAL),
> + GATE_VLPCFG_REG_FLAGS(CLK_VLPCFG_REG_VLP_26M_COM,
> "vlpcfg_vlp_26m_com",
> + "clk26m", 0, CLK_IS_CRITICAL),
> +};
> +
> +static const struct mtk_clk_desc vlpcfg_reg_mcd = {
> + .clks = vlpcfg_reg_clks,
> + .num_clks = ARRAY_SIZE(vlpcfg_reg_clks),
> +};
> +
> +static const struct of_device_id of_match_clk_mt8189_vlpcfg[] = {
> + { .compatible = "mediatek,mt8189-vlp-ao", .data =
> &vlpcfg_ao_reg_mcd },
> + { .compatible = "mediatek,mt8189-vlpcfg-ao", .data =
> &vlpcfg_reg_mcd },
> + { /* sentinel */ }
> +};
> +
> +static struct platform_driver clk_mt8189_vlpcfg_drv = {
> + .probe = mtk_clk_simple_probe,
> + .driver = {
> + .name = "clk-mt8189-vlpcfg",
> + .of_match_table = of_match_clk_mt8189_vlpcfg,
> + },
> +};
> +
> +module_platform_driver(clk_mt8189_vlpcfg_drv);
> +MODULE_LICENSE("GPL");