Message ID | 20230119124848.26364-13-Garmin.Chang@mediatek.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | MediaTek MT8188 clock support | expand |
On Thu, Jan 19, 2023 at 8:54 PM Garmin.Chang <Garmin.Chang@mediatek.com> wrote: > > Add MT8188 vdosys0 clock controller which provides clock gate > control in video system. This is integrated with mtk-mmsys > driver which will populate device by platform_device_register_data > to start vdosys clock driver. > > Signed-off-by: Garmin.Chang <Garmin.Chang@mediatek.com> > --- > drivers/clk/mediatek/Makefile | 3 +- > drivers/clk/mediatek/clk-mt8188-vdo0.c | 134 +++++++++++++++++++++++++ > 2 files changed, 136 insertions(+), 1 deletion(-) > create mode 100644 drivers/clk/mediatek/clk-mt8188-vdo0.c > > diff --git a/drivers/clk/mediatek/Makefile b/drivers/clk/mediatek/Makefile > index 7d09e9fc6538..df78c0777fef 100644 > --- a/drivers/clk/mediatek/Makefile > +++ b/drivers/clk/mediatek/Makefile > @@ -86,7 +86,8 @@ obj-$(CONFIG_COMMON_CLK_MT8186) += clk-mt8186-mcu.o clk-mt8186-topckgen.o clk-mt > obj-$(CONFIG_COMMON_CLK_MT8188) += clk-mt8188-apmixedsys.o clk-mt8188-topckgen.o \ > clk-mt8188-peri_ao.o clk-mt8188-infra_ao.o \ > clk-mt8188-cam.o clk-mt8188-ccu.o clk-mt8188-img.o \ > - clk-mt8188-ipe.o clk-mt8188-mfg.o clk-mt8188-vdec.o > + clk-mt8188-ipe.o clk-mt8188-mfg.o clk-mt8188-vdec.o \ > + clk-mt8188-vdo0.o > obj-$(CONFIG_COMMON_CLK_MT8192) += 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-mt8188-vdo0.c b/drivers/clk/mediatek/clk-mt8188-vdo0.c > new file mode 100644 > index 000000000000..30dd64374ace > --- /dev/null > +++ b/drivers/clk/mediatek/clk-mt8188-vdo0.c > @@ -0,0 +1,134 @@ > +// SPDX-License-Identifier: GPL-2.0-only > +// > +// Copyright (c) 2022 MediaTek Inc. > +// Author: Garmin Chang <garmin.chang@mediatek.com> > + > +#include <linux/clk-provider.h> > +#include <linux/platform_device.h> > +#include <dt-bindings/clock/mediatek,mt8188-clk.h> > + > +#include "clk-gate.h" > +#include "clk-mtk.h" > + > +static const struct mtk_gate_regs vdo0_0_cg_regs = { > + .set_ofs = 0x104, > + .clr_ofs = 0x108, > + .sta_ofs = 0x100, > +}; > + > +static const struct mtk_gate_regs vdo0_1_cg_regs = { > + .set_ofs = 0x114, > + .clr_ofs = 0x118, > + .sta_ofs = 0x110, > +}; > + > +static const struct mtk_gate_regs vdo0_2_cg_regs = { > + .set_ofs = 0x124, > + .clr_ofs = 0x128, > + .sta_ofs = 0x120, > +}; > + > +#define GATE_VDO0_0(_id, _name, _parent, _shift) \ > + GATE_MTK(_id, _name, _parent, &vdo0_0_cg_regs, _shift, &mtk_clk_gate_ops_setclr) > + > +#define GATE_VDO0_1(_id, _name, _parent, _shift) \ > + GATE_MTK(_id, _name, _parent, &vdo0_1_cg_regs, _shift, &mtk_clk_gate_ops_setclr) > + > +#define GATE_VDO0_2(_id, _name, _parent, _shift) \ > + GATE_MTK(_id, _name, _parent, &vdo0_2_cg_regs, _shift, &mtk_clk_gate_ops_setclr) > + > +#define GATE_VDO0_2_FLAGS(_id, _name, _parent, _shift, _flags) \ > + GATE_MTK_FLAGS(_id, _name, _parent, &vdo0_2_cg_regs, _shift, \ > + &mtk_clk_gate_ops_setclr, _flags) > + > +static const struct mtk_gate vdo0_clks[] = { > + /* VDO0_0 */ > + GATE_VDO0_0(CLK_VDO0_DISP_OVL0, "vdo0_disp_ovl0", "top_vpp", 0), > + GATE_VDO0_0(CLK_VDO0_FAKE_ENG0, "vdo0_fake_eng0", "top_vpp", 2), > + GATE_VDO0_0(CLK_VDO0_DISP_CCORR0, "vdo0_disp_ccorr0", "top_vpp", 4), > + GATE_VDO0_0(CLK_VDO0_DISP_MUTEX0, "vdo0_disp_mutex0", "top_vpp", 6), > + GATE_VDO0_0(CLK_VDO0_DISP_GAMMA0, "vdo0_disp_gamma0", "top_vpp", 8), > + GATE_VDO0_0(CLK_VDO0_DISP_DITHER0, "vdo0_disp_dither0", "top_vpp", 10), > + GATE_VDO0_0(CLK_VDO0_DISP_WDMA0, "vdo0_disp_wdma0", "top_vpp", 17), > + GATE_VDO0_0(CLK_VDO0_DISP_RDMA0, "vdo0_disp_rdma0", "top_vpp", 19), > + GATE_VDO0_0(CLK_VDO0_DSI0, "vdo0_dsi0", "top_vpp", 21), > + GATE_VDO0_0(CLK_VDO0_DSI1, "vdo0_dsi1", "top_vpp", 22), > + GATE_VDO0_0(CLK_VDO0_DSC_WRAP0, "vdo0_dsc_wrap0", "top_vpp", 23), > + GATE_VDO0_0(CLK_VDO0_VPP_MERGE0, "vdo0_vpp_merge0", "top_vpp", 24), > + GATE_VDO0_0(CLK_VDO0_DP_INTF0, "vdo0_dp_intf0", "top_vpp", 25), > + GATE_VDO0_0(CLK_VDO0_DISP_AAL0, "vdo0_disp_aal0", "top_vpp", 26), > + GATE_VDO0_0(CLK_VDO0_INLINEROT0, "vdo0_inlinerot0", "top_vpp", 27), > + GATE_VDO0_0(CLK_VDO0_APB_BUS, "vdo0_apb_bus", "top_vpp", 28), > + GATE_VDO0_0(CLK_VDO0_DISP_COLOR0, "vdo0_disp_color0", "top_vpp", 29), > + GATE_VDO0_0(CLK_VDO0_MDP_WROT0, "vdo0_mdp_wrot0", "top_vpp", 30), > + GATE_VDO0_0(CLK_VDO0_DISP_RSZ0, "vdo0_disp_rsz0", "top_vpp", 31), > + /* VDO0_1 */ > + GATE_VDO0_1(CLK_VDO0_DISP_POSTMASK0, "vdo0_disp_postmask0", "top_vpp", 0), > + GATE_VDO0_1(CLK_VDO0_FAKE_ENG1, "vdo0_fake_eng1", "top_vpp", 1), > + GATE_VDO0_1(CLK_VDO0_DL_ASYNC2, "vdo0_dl_async2", "top_vpp", 5), > + GATE_VDO0_1(CLK_VDO0_DL_RELAY3, "vdo0_dl_relay3", "top_vpp", 6), > + GATE_VDO0_1(CLK_VDO0_DL_RELAY4, "vdo0_dl_relay4", "top_vpp", 7), > + GATE_VDO0_1(CLK_VDO0_SMI_GALS, "vdo0_smi_gals", "top_vpp", 10), > + GATE_VDO0_1(CLK_VDO0_SMI_COMMON, "vdo0_smi_common", "top_vpp", 11), > + GATE_VDO0_1(CLK_VDO0_SMI_EMI, "vdo0_smi_emi", "top_vpp", 12), > + GATE_VDO0_1(CLK_VDO0_SMI_IOMMU, "vdo0_smi_iommu", "top_vpp", 13), > + GATE_VDO0_1(CLK_VDO0_SMI_LARB, "vdo0_smi_larb", "top_vpp", 14), > + GATE_VDO0_1(CLK_VDO0_SMI_RSI, "vdo0_smi_rsi", "top_vpp", 15), > + /* VDO0_2 */ > + GATE_VDO0_2(CLK_VDO0_DSI0_DSI, "vdo0_dsi0_dsi", "top_dsi_occ", 0), > + GATE_VDO0_2(CLK_VDO0_DSI1_DSI, "vdo0_dsi1_dsi", "top_dsi_occ", 8), > + GATE_VDO0_2_FLAGS(CLK_VDO0_DP_INTF0_DP_INTF, "vdo0_dp_intf0_dp_intf", > + "top_edp", 16, CLK_SET_RATE_PARENT), > +}; > + > +static int clk_mt8188_vdo0_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct device_node *node = dev->parent->of_node; > + struct clk_hw_onecell_data *clk_data; > + int r; > + > + clk_data = mtk_alloc_clk_data(CLK_VDO0_NR_CLK); > + if (!clk_data) > + return -ENOMEM; > + > + r = mtk_clk_register_gates(node, vdo0_clks, ARRAY_SIZE(vdo0_clks), clk_data); This API was changed. Please rebase onto the latest -next and update. Angelo (CC-ed) also mentioned a new simple probe variant for non-DT clock drivers is being developed. He didn't mention a timeline though.
Il 03/02/23 08:19, Chen-Yu Tsai ha scritto: > On Thu, Jan 19, 2023 at 8:54 PM Garmin.Chang <Garmin.Chang@mediatek.com> wrote: >> >> Add MT8188 vdosys0 clock controller which provides clock gate >> control in video system. This is integrated with mtk-mmsys >> driver which will populate device by platform_device_register_data >> to start vdosys clock driver. >> >> Signed-off-by: Garmin.Chang <Garmin.Chang@mediatek.com> >> --- >> drivers/clk/mediatek/Makefile | 3 +- >> drivers/clk/mediatek/clk-mt8188-vdo0.c | 134 +++++++++++++++++++++++++ >> 2 files changed, 136 insertions(+), 1 deletion(-) >> create mode 100644 drivers/clk/mediatek/clk-mt8188-vdo0.c >> >> diff --git a/drivers/clk/mediatek/Makefile b/drivers/clk/mediatek/Makefile >> index 7d09e9fc6538..df78c0777fef 100644 >> --- a/drivers/clk/mediatek/Makefile >> +++ b/drivers/clk/mediatek/Makefile >> @@ -86,7 +86,8 @@ obj-$(CONFIG_COMMON_CLK_MT8186) += clk-mt8186-mcu.o clk-mt8186-topckgen.o clk-mt >> obj-$(CONFIG_COMMON_CLK_MT8188) += clk-mt8188-apmixedsys.o clk-mt8188-topckgen.o \ >> clk-mt8188-peri_ao.o clk-mt8188-infra_ao.o \ >> clk-mt8188-cam.o clk-mt8188-ccu.o clk-mt8188-img.o \ >> - clk-mt8188-ipe.o clk-mt8188-mfg.o clk-mt8188-vdec.o >> + clk-mt8188-ipe.o clk-mt8188-mfg.o clk-mt8188-vdec.o \ >> + clk-mt8188-vdo0.o >> obj-$(CONFIG_COMMON_CLK_MT8192) += 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-mt8188-vdo0.c b/drivers/clk/mediatek/clk-mt8188-vdo0.c >> new file mode 100644 >> index 000000000000..30dd64374ace >> --- /dev/null >> +++ b/drivers/clk/mediatek/clk-mt8188-vdo0.c >> @@ -0,0 +1,134 @@ >> +// SPDX-License-Identifier: GPL-2.0-only >> +// >> +// Copyright (c) 2022 MediaTek Inc. >> +// Author: Garmin Chang <garmin.chang@mediatek.com> >> + >> +#include <linux/clk-provider.h> >> +#include <linux/platform_device.h> >> +#include <dt-bindings/clock/mediatek,mt8188-clk.h> >> + >> +#include "clk-gate.h" >> +#include "clk-mtk.h" >> + >> +static const struct mtk_gate_regs vdo0_0_cg_regs = { >> + .set_ofs = 0x104, >> + .clr_ofs = 0x108, >> + .sta_ofs = 0x100, >> +}; >> + >> +static const struct mtk_gate_regs vdo0_1_cg_regs = { >> + .set_ofs = 0x114, >> + .clr_ofs = 0x118, >> + .sta_ofs = 0x110, >> +}; >> + >> +static const struct mtk_gate_regs vdo0_2_cg_regs = { >> + .set_ofs = 0x124, >> + .clr_ofs = 0x128, >> + .sta_ofs = 0x120, >> +}; >> + >> +#define GATE_VDO0_0(_id, _name, _parent, _shift) \ >> + GATE_MTK(_id, _name, _parent, &vdo0_0_cg_regs, _shift, &mtk_clk_gate_ops_setclr) >> + >> +#define GATE_VDO0_1(_id, _name, _parent, _shift) \ >> + GATE_MTK(_id, _name, _parent, &vdo0_1_cg_regs, _shift, &mtk_clk_gate_ops_setclr) >> + >> +#define GATE_VDO0_2(_id, _name, _parent, _shift) \ >> + GATE_MTK(_id, _name, _parent, &vdo0_2_cg_regs, _shift, &mtk_clk_gate_ops_setclr) >> + >> +#define GATE_VDO0_2_FLAGS(_id, _name, _parent, _shift, _flags) \ >> + GATE_MTK_FLAGS(_id, _name, _parent, &vdo0_2_cg_regs, _shift, \ >> + &mtk_clk_gate_ops_setclr, _flags) >> + >> +static const struct mtk_gate vdo0_clks[] = { >> + /* VDO0_0 */ >> + GATE_VDO0_0(CLK_VDO0_DISP_OVL0, "vdo0_disp_ovl0", "top_vpp", 0), >> + GATE_VDO0_0(CLK_VDO0_FAKE_ENG0, "vdo0_fake_eng0", "top_vpp", 2), >> + GATE_VDO0_0(CLK_VDO0_DISP_CCORR0, "vdo0_disp_ccorr0", "top_vpp", 4), >> + GATE_VDO0_0(CLK_VDO0_DISP_MUTEX0, "vdo0_disp_mutex0", "top_vpp", 6), >> + GATE_VDO0_0(CLK_VDO0_DISP_GAMMA0, "vdo0_disp_gamma0", "top_vpp", 8), >> + GATE_VDO0_0(CLK_VDO0_DISP_DITHER0, "vdo0_disp_dither0", "top_vpp", 10), >> + GATE_VDO0_0(CLK_VDO0_DISP_WDMA0, "vdo0_disp_wdma0", "top_vpp", 17), >> + GATE_VDO0_0(CLK_VDO0_DISP_RDMA0, "vdo0_disp_rdma0", "top_vpp", 19), >> + GATE_VDO0_0(CLK_VDO0_DSI0, "vdo0_dsi0", "top_vpp", 21), >> + GATE_VDO0_0(CLK_VDO0_DSI1, "vdo0_dsi1", "top_vpp", 22), >> + GATE_VDO0_0(CLK_VDO0_DSC_WRAP0, "vdo0_dsc_wrap0", "top_vpp", 23), >> + GATE_VDO0_0(CLK_VDO0_VPP_MERGE0, "vdo0_vpp_merge0", "top_vpp", 24), >> + GATE_VDO0_0(CLK_VDO0_DP_INTF0, "vdo0_dp_intf0", "top_vpp", 25), >> + GATE_VDO0_0(CLK_VDO0_DISP_AAL0, "vdo0_disp_aal0", "top_vpp", 26), >> + GATE_VDO0_0(CLK_VDO0_INLINEROT0, "vdo0_inlinerot0", "top_vpp", 27), >> + GATE_VDO0_0(CLK_VDO0_APB_BUS, "vdo0_apb_bus", "top_vpp", 28), >> + GATE_VDO0_0(CLK_VDO0_DISP_COLOR0, "vdo0_disp_color0", "top_vpp", 29), >> + GATE_VDO0_0(CLK_VDO0_MDP_WROT0, "vdo0_mdp_wrot0", "top_vpp", 30), >> + GATE_VDO0_0(CLK_VDO0_DISP_RSZ0, "vdo0_disp_rsz0", "top_vpp", 31), >> + /* VDO0_1 */ >> + GATE_VDO0_1(CLK_VDO0_DISP_POSTMASK0, "vdo0_disp_postmask0", "top_vpp", 0), >> + GATE_VDO0_1(CLK_VDO0_FAKE_ENG1, "vdo0_fake_eng1", "top_vpp", 1), >> + GATE_VDO0_1(CLK_VDO0_DL_ASYNC2, "vdo0_dl_async2", "top_vpp", 5), >> + GATE_VDO0_1(CLK_VDO0_DL_RELAY3, "vdo0_dl_relay3", "top_vpp", 6), >> + GATE_VDO0_1(CLK_VDO0_DL_RELAY4, "vdo0_dl_relay4", "top_vpp", 7), >> + GATE_VDO0_1(CLK_VDO0_SMI_GALS, "vdo0_smi_gals", "top_vpp", 10), >> + GATE_VDO0_1(CLK_VDO0_SMI_COMMON, "vdo0_smi_common", "top_vpp", 11), >> + GATE_VDO0_1(CLK_VDO0_SMI_EMI, "vdo0_smi_emi", "top_vpp", 12), >> + GATE_VDO0_1(CLK_VDO0_SMI_IOMMU, "vdo0_smi_iommu", "top_vpp", 13), >> + GATE_VDO0_1(CLK_VDO0_SMI_LARB, "vdo0_smi_larb", "top_vpp", 14), >> + GATE_VDO0_1(CLK_VDO0_SMI_RSI, "vdo0_smi_rsi", "top_vpp", 15), >> + /* VDO0_2 */ >> + GATE_VDO0_2(CLK_VDO0_DSI0_DSI, "vdo0_dsi0_dsi", "top_dsi_occ", 0), >> + GATE_VDO0_2(CLK_VDO0_DSI1_DSI, "vdo0_dsi1_dsi", "top_dsi_occ", 8), >> + GATE_VDO0_2_FLAGS(CLK_VDO0_DP_INTF0_DP_INTF, "vdo0_dp_intf0_dp_intf", >> + "top_edp", 16, CLK_SET_RATE_PARENT), >> +}; >> + >> +static int clk_mt8188_vdo0_probe(struct platform_device *pdev) >> +{ >> + struct device *dev = &pdev->dev; >> + struct device_node *node = dev->parent->of_node; >> + struct clk_hw_onecell_data *clk_data; >> + int r; >> + >> + clk_data = mtk_alloc_clk_data(CLK_VDO0_NR_CLK); >> + if (!clk_data) >> + return -ENOMEM; >> + >> + r = mtk_clk_register_gates(node, vdo0_clks, ARRAY_SIZE(vdo0_clks), clk_data); > > This API was changed. Please rebase onto the latest -next and update. > > Angelo (CC-ed) also mentioned a new simple probe variant for non-DT > clock drivers is being developed. He didn't mention a timeline though. I've already tested the new simple probe variant for non-DT clock drivers and it works fine on MT8173, MT8192 and MT8195. Timeline - I should be able to push the part 2 series next week, which will include more conversion to simple probe and almost all clock drivers changed to allow building as modules. Cheers, Angelo
On Fri, 2023-02-03 at 15:19 +0800, Chen-Yu Tsai wrote: > On Thu, Jan 19, 2023 at 8:54 PM Garmin.Chang < > Garmin.Chang@mediatek.com> wrote: > > > > Add MT8188 vdosys0 clock controller which provides clock gate > > control in video system. This is integrated with mtk-mmsys > > driver which will populate device by platform_device_register_data > > to start vdosys clock driver. > > > > Signed-off-by: Garmin.Chang <Garmin.Chang@mediatek.com> > > --- > > drivers/clk/mediatek/Makefile | 3 +- > > drivers/clk/mediatek/clk-mt8188-vdo0.c | 134 > > +++++++++++++++++++++++++ > > 2 files changed, 136 insertions(+), 1 deletion(-) > > create mode 100644 drivers/clk/mediatek/clk-mt8188-vdo0.c > > > > diff --git a/drivers/clk/mediatek/Makefile > > b/drivers/clk/mediatek/Makefile > > index 7d09e9fc6538..df78c0777fef 100644 > > --- a/drivers/clk/mediatek/Makefile > > +++ b/drivers/clk/mediatek/Makefile > > @@ -86,7 +86,8 @@ obj-$(CONFIG_COMMON_CLK_MT8186) += clk-mt8186- > > mcu.o clk-mt8186-topckgen.o clk-mt > > obj-$(CONFIG_COMMON_CLK_MT8188) += clk-mt8188-apmixedsys.o clk- > > mt8188-topckgen.o \ > > clk-mt8188-peri_ao.o clk-mt8188- > > infra_ao.o \ > > clk-mt8188-cam.o clk-mt8188- > > ccu.o clk-mt8188-img.o \ > > - clk-mt8188-ipe.o clk-mt8188- > > mfg.o clk-mt8188-vdec.o > > + clk-mt8188-ipe.o clk-mt8188- > > mfg.o clk-mt8188-vdec.o \ > > + clk-mt8188-vdo0.o > > obj-$(CONFIG_COMMON_CLK_MT8192) += 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-mt8188-vdo0.c > > b/drivers/clk/mediatek/clk-mt8188-vdo0.c > > new file mode 100644 > > index 000000000000..30dd64374ace > > --- /dev/null > > +++ b/drivers/clk/mediatek/clk-mt8188-vdo0.c > > @@ -0,0 +1,134 @@ > > +// SPDX-License-Identifier: GPL-2.0-only > > +// > > +// Copyright (c) 2022 MediaTek Inc. > > +// Author: Garmin Chang <garmin.chang@mediatek.com> > > + > > +#include <linux/clk-provider.h> > > +#include <linux/platform_device.h> > > +#include <dt-bindings/clock/mediatek,mt8188-clk.h> > > + > > +#include "clk-gate.h" > > +#include "clk-mtk.h" > > + > > +static const struct mtk_gate_regs vdo0_0_cg_regs = { > > + .set_ofs = 0x104, > > + .clr_ofs = 0x108, > > + .sta_ofs = 0x100, > > +}; > > + > > +static const struct mtk_gate_regs vdo0_1_cg_regs = { > > + .set_ofs = 0x114, > > + .clr_ofs = 0x118, > > + .sta_ofs = 0x110, > > +}; > > + > > +static const struct mtk_gate_regs vdo0_2_cg_regs = { > > + .set_ofs = 0x124, > > + .clr_ofs = 0x128, > > + .sta_ofs = 0x120, > > +}; > > + > > +#define GATE_VDO0_0(_id, _name, _parent, > > _shift) \ > > + GATE_MTK(_id, _name, _parent, &vdo0_0_cg_regs, _shift, > > &mtk_clk_gate_ops_setclr) > > + > > +#define GATE_VDO0_1(_id, _name, _parent, > > _shift) \ > > + GATE_MTK(_id, _name, _parent, &vdo0_1_cg_regs, _shift, > > &mtk_clk_gate_ops_setclr) > > + > > +#define GATE_VDO0_2(_id, _name, _parent, > > _shift) \ > > + GATE_MTK(_id, _name, _parent, &vdo0_2_cg_regs, _shift, > > &mtk_clk_gate_ops_setclr) > > + > > +#define GATE_VDO0_2_FLAGS(_id, _name, _parent, _shift, > > _flags) \ > > + GATE_MTK_FLAGS(_id, _name, _parent, &vdo0_2_cg_regs, > > _shift, \ > > + &mtk_clk_gate_ops_setclr, _flags) > > + > > +static const struct mtk_gate vdo0_clks[] = { > > + /* VDO0_0 */ > > + GATE_VDO0_0(CLK_VDO0_DISP_OVL0, "vdo0_disp_ovl0", > > "top_vpp", 0), > > + GATE_VDO0_0(CLK_VDO0_FAKE_ENG0, "vdo0_fake_eng0", > > "top_vpp", 2), > > + GATE_VDO0_0(CLK_VDO0_DISP_CCORR0, "vdo0_disp_ccorr0", > > "top_vpp", 4), > > + GATE_VDO0_0(CLK_VDO0_DISP_MUTEX0, "vdo0_disp_mutex0", > > "top_vpp", 6), > > + GATE_VDO0_0(CLK_VDO0_DISP_GAMMA0, "vdo0_disp_gamma0", > > "top_vpp", 8), > > + GATE_VDO0_0(CLK_VDO0_DISP_DITHER0, "vdo0_disp_dither0", > > "top_vpp", 10), > > + GATE_VDO0_0(CLK_VDO0_DISP_WDMA0, "vdo0_disp_wdma0", > > "top_vpp", 17), > > + GATE_VDO0_0(CLK_VDO0_DISP_RDMA0, "vdo0_disp_rdma0", > > "top_vpp", 19), > > + GATE_VDO0_0(CLK_VDO0_DSI0, "vdo0_dsi0", "top_vpp", 21), > > + GATE_VDO0_0(CLK_VDO0_DSI1, "vdo0_dsi1", "top_vpp", 22), > > + GATE_VDO0_0(CLK_VDO0_DSC_WRAP0, "vdo0_dsc_wrap0", > > "top_vpp", 23), > > + GATE_VDO0_0(CLK_VDO0_VPP_MERGE0, "vdo0_vpp_merge0", > > "top_vpp", 24), > > + GATE_VDO0_0(CLK_VDO0_DP_INTF0, "vdo0_dp_intf0", "top_vpp", > > 25), > > + GATE_VDO0_0(CLK_VDO0_DISP_AAL0, "vdo0_disp_aal0", > > "top_vpp", 26), > > + GATE_VDO0_0(CLK_VDO0_INLINEROT0, "vdo0_inlinerot0", > > "top_vpp", 27), > > + GATE_VDO0_0(CLK_VDO0_APB_BUS, "vdo0_apb_bus", "top_vpp", > > 28), > > + GATE_VDO0_0(CLK_VDO0_DISP_COLOR0, "vdo0_disp_color0", > > "top_vpp", 29), > > + GATE_VDO0_0(CLK_VDO0_MDP_WROT0, "vdo0_mdp_wrot0", > > "top_vpp", 30), > > + GATE_VDO0_0(CLK_VDO0_DISP_RSZ0, "vdo0_disp_rsz0", > > "top_vpp", 31), > > + /* VDO0_1 */ > > + GATE_VDO0_1(CLK_VDO0_DISP_POSTMASK0, "vdo0_disp_postmask0", > > "top_vpp", 0), > > + GATE_VDO0_1(CLK_VDO0_FAKE_ENG1, "vdo0_fake_eng1", > > "top_vpp", 1), > > + GATE_VDO0_1(CLK_VDO0_DL_ASYNC2, "vdo0_dl_async2", > > "top_vpp", 5), > > + GATE_VDO0_1(CLK_VDO0_DL_RELAY3, "vdo0_dl_relay3", > > "top_vpp", 6), > > + GATE_VDO0_1(CLK_VDO0_DL_RELAY4, "vdo0_dl_relay4", > > "top_vpp", 7), > > + GATE_VDO0_1(CLK_VDO0_SMI_GALS, "vdo0_smi_gals", "top_vpp", > > 10), > > + GATE_VDO0_1(CLK_VDO0_SMI_COMMON, "vdo0_smi_common", > > "top_vpp", 11), > > + GATE_VDO0_1(CLK_VDO0_SMI_EMI, "vdo0_smi_emi", "top_vpp", > > 12), > > + GATE_VDO0_1(CLK_VDO0_SMI_IOMMU, "vdo0_smi_iommu", > > "top_vpp", 13), > > + GATE_VDO0_1(CLK_VDO0_SMI_LARB, "vdo0_smi_larb", "top_vpp", > > 14), > > + GATE_VDO0_1(CLK_VDO0_SMI_RSI, "vdo0_smi_rsi", "top_vpp", > > 15), > > + /* VDO0_2 */ > > + GATE_VDO0_2(CLK_VDO0_DSI0_DSI, "vdo0_dsi0_dsi", > > "top_dsi_occ", 0), > > + GATE_VDO0_2(CLK_VDO0_DSI1_DSI, "vdo0_dsi1_dsi", > > "top_dsi_occ", 8), > > + GATE_VDO0_2_FLAGS(CLK_VDO0_DP_INTF0_DP_INTF, > > "vdo0_dp_intf0_dp_intf", > > + "top_edp", 16, CLK_SET_RATE_PARENT), > > +}; > > + > > +static int clk_mt8188_vdo0_probe(struct platform_device *pdev) > > +{ > > + struct device *dev = &pdev->dev; > > + struct device_node *node = dev->parent->of_node; > > + struct clk_hw_onecell_data *clk_data; > > + int r; > > + > > + clk_data = mtk_alloc_clk_data(CLK_VDO0_NR_CLK); > > + if (!clk_data) > > + return -ENOMEM; > > + > > + r = mtk_clk_register_gates(node, vdo0_clks, > > ARRAY_SIZE(vdo0_clks), clk_data); > > This API was changed. Please rebase onto the latest -next and update. Thank you for your suggestions. OK I will modify it in v6. > > Angelo (CC-ed) also mentioned a new simple probe variant for non-DT > clock drivers is being developed. He didn't mention a timeline > though. When the new simple probe variant is ready, I will rebase latest-next and update it.
On Fri, 2023-02-03 at 11:49 +0100, AngeloGioacchino Del Regno wrote: > Il 03/02/23 08:19, Chen-Yu Tsai ha scritto: > > On Thu, Jan 19, 2023 at 8:54 PM Garmin.Chang < > > Garmin.Chang@mediatek.com> wrote: > > > > > > Add MT8188 vdosys0 clock controller which provides clock gate > > > control in video system. This is integrated with mtk-mmsys > > > driver which will populate device by > > > platform_device_register_data > > > to start vdosys clock driver. > > > > > > Signed-off-by: Garmin.Chang <Garmin.Chang@mediatek.com> > > > --- > > > drivers/clk/mediatek/Makefile | 3 +- > > > drivers/clk/mediatek/clk-mt8188-vdo0.c | 134 > > > +++++++++++++++++++++++++ > > > 2 files changed, 136 insertions(+), 1 deletion(-) > > > create mode 100644 drivers/clk/mediatek/clk-mt8188-vdo0.c > > > > > > diff --git a/drivers/clk/mediatek/Makefile > > > b/drivers/clk/mediatek/Makefile > > > index 7d09e9fc6538..df78c0777fef 100644 > > > --- a/drivers/clk/mediatek/Makefile > > > +++ b/drivers/clk/mediatek/Makefile > > > @@ -86,7 +86,8 @@ obj-$(CONFIG_COMMON_CLK_MT8186) += clk-mt8186- > > > mcu.o clk-mt8186-topckgen.o clk-mt > > > obj-$(CONFIG_COMMON_CLK_MT8188) += clk-mt8188-apmixedsys.o clk- > > > mt8188-topckgen.o \ > > > clk-mt8188-peri_ao.o clk- > > > mt8188-infra_ao.o \ > > > clk-mt8188-cam.o clk-mt8188- > > > ccu.o clk-mt8188-img.o \ > > > - clk-mt8188-ipe.o clk-mt8188- > > > mfg.o clk-mt8188-vdec.o > > > + clk-mt8188-ipe.o clk-mt8188- > > > mfg.o clk-mt8188-vdec.o \ > > > + clk-mt8188-vdo0.o > > > obj-$(CONFIG_COMMON_CLK_MT8192) += 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-mt8188-vdo0.c > > > b/drivers/clk/mediatek/clk-mt8188-vdo0.c > > > new file mode 100644 > > > index 000000000000..30dd64374ace > > > --- /dev/null > > > +++ b/drivers/clk/mediatek/clk-mt8188-vdo0.c > > > @@ -0,0 +1,134 @@ > > > +// SPDX-License-Identifier: GPL-2.0-only > > > +// > > > +// Copyright (c) 2022 MediaTek Inc. > > > +// Author: Garmin Chang <garmin.chang@mediatek.com> > > > + > > > +#include <linux/clk-provider.h> > > > +#include <linux/platform_device.h> > > > +#include <dt-bindings/clock/mediatek,mt8188-clk.h> > > > + > > > +#include "clk-gate.h" > > > +#include "clk-mtk.h" > > > + > > > +static const struct mtk_gate_regs vdo0_0_cg_regs = { > > > + .set_ofs = 0x104, > > > + .clr_ofs = 0x108, > > > + .sta_ofs = 0x100, > > > +}; > > > + > > > +static const struct mtk_gate_regs vdo0_1_cg_regs = { > > > + .set_ofs = 0x114, > > > + .clr_ofs = 0x118, > > > + .sta_ofs = 0x110, > > > +}; > > > + > > > +static const struct mtk_gate_regs vdo0_2_cg_regs = { > > > + .set_ofs = 0x124, > > > + .clr_ofs = 0x128, > > > + .sta_ofs = 0x120, > > > +}; > > > + > > > +#define GATE_VDO0_0(_id, _name, _parent, > > > _shift) \ > > > + GATE_MTK(_id, _name, _parent, &vdo0_0_cg_regs, _shift, > > > &mtk_clk_gate_ops_setclr) > > > + > > > +#define GATE_VDO0_1(_id, _name, _parent, > > > _shift) \ > > > + GATE_MTK(_id, _name, _parent, &vdo0_1_cg_regs, _shift, > > > &mtk_clk_gate_ops_setclr) > > > + > > > +#define GATE_VDO0_2(_id, _name, _parent, > > > _shift) \ > > > + GATE_MTK(_id, _name, _parent, &vdo0_2_cg_regs, _shift, > > > &mtk_clk_gate_ops_setclr) > > > + > > > +#define GATE_VDO0_2_FLAGS(_id, _name, _parent, _shift, > > > _flags) \ > > > + GATE_MTK_FLAGS(_id, _name, _parent, &vdo0_2_cg_regs, > > > _shift, \ > > > + &mtk_clk_gate_ops_setclr, _flags) > > > + > > > +static const struct mtk_gate vdo0_clks[] = { > > > + /* VDO0_0 */ > > > + GATE_VDO0_0(CLK_VDO0_DISP_OVL0, "vdo0_disp_ovl0", > > > "top_vpp", 0), > > > + GATE_VDO0_0(CLK_VDO0_FAKE_ENG0, "vdo0_fake_eng0", > > > "top_vpp", 2), > > > + GATE_VDO0_0(CLK_VDO0_DISP_CCORR0, "vdo0_disp_ccorr0", > > > "top_vpp", 4), > > > + GATE_VDO0_0(CLK_VDO0_DISP_MUTEX0, "vdo0_disp_mutex0", > > > "top_vpp", 6), > > > + GATE_VDO0_0(CLK_VDO0_DISP_GAMMA0, "vdo0_disp_gamma0", > > > "top_vpp", 8), > > > + GATE_VDO0_0(CLK_VDO0_DISP_DITHER0, "vdo0_disp_dither0", > > > "top_vpp", 10), > > > + GATE_VDO0_0(CLK_VDO0_DISP_WDMA0, "vdo0_disp_wdma0", > > > "top_vpp", 17), > > > + GATE_VDO0_0(CLK_VDO0_DISP_RDMA0, "vdo0_disp_rdma0", > > > "top_vpp", 19), > > > + GATE_VDO0_0(CLK_VDO0_DSI0, "vdo0_dsi0", "top_vpp", 21), > > > + GATE_VDO0_0(CLK_VDO0_DSI1, "vdo0_dsi1", "top_vpp", 22), > > > + GATE_VDO0_0(CLK_VDO0_DSC_WRAP0, "vdo0_dsc_wrap0", > > > "top_vpp", 23), > > > + GATE_VDO0_0(CLK_VDO0_VPP_MERGE0, "vdo0_vpp_merge0", > > > "top_vpp", 24), > > > + GATE_VDO0_0(CLK_VDO0_DP_INTF0, "vdo0_dp_intf0", > > > "top_vpp", 25), > > > + GATE_VDO0_0(CLK_VDO0_DISP_AAL0, "vdo0_disp_aal0", > > > "top_vpp", 26), > > > + GATE_VDO0_0(CLK_VDO0_INLINEROT0, "vdo0_inlinerot0", > > > "top_vpp", 27), > > > + GATE_VDO0_0(CLK_VDO0_APB_BUS, "vdo0_apb_bus", "top_vpp", > > > 28), > > > + GATE_VDO0_0(CLK_VDO0_DISP_COLOR0, "vdo0_disp_color0", > > > "top_vpp", 29), > > > + GATE_VDO0_0(CLK_VDO0_MDP_WROT0, "vdo0_mdp_wrot0", > > > "top_vpp", 30), > > > + GATE_VDO0_0(CLK_VDO0_DISP_RSZ0, "vdo0_disp_rsz0", > > > "top_vpp", 31), > > > + /* VDO0_1 */ > > > + GATE_VDO0_1(CLK_VDO0_DISP_POSTMASK0, > > > "vdo0_disp_postmask0", "top_vpp", 0), > > > + GATE_VDO0_1(CLK_VDO0_FAKE_ENG1, "vdo0_fake_eng1", > > > "top_vpp", 1), > > > + GATE_VDO0_1(CLK_VDO0_DL_ASYNC2, "vdo0_dl_async2", > > > "top_vpp", 5), > > > + GATE_VDO0_1(CLK_VDO0_DL_RELAY3, "vdo0_dl_relay3", > > > "top_vpp", 6), > > > + GATE_VDO0_1(CLK_VDO0_DL_RELAY4, "vdo0_dl_relay4", > > > "top_vpp", 7), > > > + GATE_VDO0_1(CLK_VDO0_SMI_GALS, "vdo0_smi_gals", > > > "top_vpp", 10), > > > + GATE_VDO0_1(CLK_VDO0_SMI_COMMON, "vdo0_smi_common", > > > "top_vpp", 11), > > > + GATE_VDO0_1(CLK_VDO0_SMI_EMI, "vdo0_smi_emi", "top_vpp", > > > 12), > > > + GATE_VDO0_1(CLK_VDO0_SMI_IOMMU, "vdo0_smi_iommu", > > > "top_vpp", 13), > > > + GATE_VDO0_1(CLK_VDO0_SMI_LARB, "vdo0_smi_larb", > > > "top_vpp", 14), > > > + GATE_VDO0_1(CLK_VDO0_SMI_RSI, "vdo0_smi_rsi", "top_vpp", > > > 15), > > > + /* VDO0_2 */ > > > + GATE_VDO0_2(CLK_VDO0_DSI0_DSI, "vdo0_dsi0_dsi", > > > "top_dsi_occ", 0), > > > + GATE_VDO0_2(CLK_VDO0_DSI1_DSI, "vdo0_dsi1_dsi", > > > "top_dsi_occ", 8), > > > + GATE_VDO0_2_FLAGS(CLK_VDO0_DP_INTF0_DP_INTF, > > > "vdo0_dp_intf0_dp_intf", > > > + "top_edp", 16, CLK_SET_RATE_PARENT), > > > +}; > > > + > > > +static int clk_mt8188_vdo0_probe(struct platform_device *pdev) > > > +{ > > > + struct device *dev = &pdev->dev; > > > + struct device_node *node = dev->parent->of_node; > > > + struct clk_hw_onecell_data *clk_data; > > > + int r; > > > + > > > + clk_data = mtk_alloc_clk_data(CLK_VDO0_NR_CLK); > > > + if (!clk_data) > > > + return -ENOMEM; > > > + > > > + r = mtk_clk_register_gates(node, vdo0_clks, > > > ARRAY_SIZE(vdo0_clks), clk_data); > > > > This API was changed. Please rebase onto the latest -next and > > update. > > > > Angelo (CC-ed) also mentioned a new simple probe variant for non-DT > > clock drivers is being developed. He didn't mention a timeline > > though. > > I've already tested the new simple probe variant for non-DT clock > drivers and > it works fine on MT8173, MT8192 and MT8195. > > Timeline - I should be able to push the part 2 series next week, > which will include > more conversion to simple probe and almost all clock drivers changed > to allow > building as modules. > > Cheers, > Angelo > Thank you for your suggestions. When the new simple probe variant and building as modules are ready, I will rebase latest-next and update it.
Il 09/03/23 06:49, Garmin Chang (張家銘) ha scritto: > On Fri, 2023-02-03 at 11:49 +0100, AngeloGioacchino Del Regno wrote: >> Il 03/02/23 08:19, Chen-Yu Tsai ha scritto: >>> On Thu, Jan 19, 2023 at 8:54 PM Garmin.Chang < >>> Garmin.Chang@mediatek.com> wrote: >>>> >>>> Add MT8188 vdosys0 clock controller which provides clock gate >>>> control in video system. This is integrated with mtk-mmsys >>>> driver which will populate device by >>>> platform_device_register_data >>>> to start vdosys clock driver. >>>> >>>> Signed-off-by: Garmin.Chang <Garmin.Chang@mediatek.com> >>>> --- >>>> drivers/clk/mediatek/Makefile | 3 +- >>>> drivers/clk/mediatek/clk-mt8188-vdo0.c | 134 >>>> +++++++++++++++++++++++++ >>>> 2 files changed, 136 insertions(+), 1 deletion(-) >>>> create mode 100644 drivers/clk/mediatek/clk-mt8188-vdo0.c >>>> >>>> diff --git a/drivers/clk/mediatek/Makefile >>>> b/drivers/clk/mediatek/Makefile >>>> index 7d09e9fc6538..df78c0777fef 100644 >>>> --- a/drivers/clk/mediatek/Makefile >>>> +++ b/drivers/clk/mediatek/Makefile >>>> @@ -86,7 +86,8 @@ obj-$(CONFIG_COMMON_CLK_MT8186) += clk-mt8186- >>>> mcu.o clk-mt8186-topckgen.o clk-mt >>>> obj-$(CONFIG_COMMON_CLK_MT8188) += clk-mt8188-apmixedsys.o clk- >>>> mt8188-topckgen.o \ >>>> clk-mt8188-peri_ao.o clk- >>>> mt8188-infra_ao.o \ >>>> clk-mt8188-cam.o clk-mt8188- >>>> ccu.o clk-mt8188-img.o \ >>>> - clk-mt8188-ipe.o clk-mt8188- >>>> mfg.o clk-mt8188-vdec.o >>>> + clk-mt8188-ipe.o clk-mt8188- >>>> mfg.o clk-mt8188-vdec.o \ >>>> + clk-mt8188-vdo0.o >>>> obj-$(CONFIG_COMMON_CLK_MT8192) += 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-mt8188-vdo0.c >>>> b/drivers/clk/mediatek/clk-mt8188-vdo0.c >>>> new file mode 100644 >>>> index 000000000000..30dd64374ace >>>> --- /dev/null >>>> +++ b/drivers/clk/mediatek/clk-mt8188-vdo0.c >>>> @@ -0,0 +1,134 @@ >>>> +// SPDX-License-Identifier: GPL-2.0-only >>>> +// >>>> +// Copyright (c) 2022 MediaTek Inc. >>>> +// Author: Garmin Chang <garmin.chang@mediatek.com> >>>> + >>>> +#include <linux/clk-provider.h> >>>> +#include <linux/platform_device.h> >>>> +#include <dt-bindings/clock/mediatek,mt8188-clk.h> >>>> + >>>> +#include "clk-gate.h" >>>> +#include "clk-mtk.h" >>>> + >>>> +static const struct mtk_gate_regs vdo0_0_cg_regs = { >>>> + .set_ofs = 0x104, >>>> + .clr_ofs = 0x108, >>>> + .sta_ofs = 0x100, >>>> +}; >>>> + >>>> +static const struct mtk_gate_regs vdo0_1_cg_regs = { >>>> + .set_ofs = 0x114, >>>> + .clr_ofs = 0x118, >>>> + .sta_ofs = 0x110, >>>> +}; >>>> + >>>> +static const struct mtk_gate_regs vdo0_2_cg_regs = { >>>> + .set_ofs = 0x124, >>>> + .clr_ofs = 0x128, >>>> + .sta_ofs = 0x120, >>>> +}; >>>> + >>>> +#define GATE_VDO0_0(_id, _name, _parent, >>>> _shift) \ >>>> + GATE_MTK(_id, _name, _parent, &vdo0_0_cg_regs, _shift, >>>> &mtk_clk_gate_ops_setclr) >>>> + >>>> +#define GATE_VDO0_1(_id, _name, _parent, >>>> _shift) \ >>>> + GATE_MTK(_id, _name, _parent, &vdo0_1_cg_regs, _shift, >>>> &mtk_clk_gate_ops_setclr) >>>> + >>>> +#define GATE_VDO0_2(_id, _name, _parent, >>>> _shift) \ >>>> + GATE_MTK(_id, _name, _parent, &vdo0_2_cg_regs, _shift, >>>> &mtk_clk_gate_ops_setclr) >>>> + >>>> +#define GATE_VDO0_2_FLAGS(_id, _name, _parent, _shift, >>>> _flags) \ >>>> + GATE_MTK_FLAGS(_id, _name, _parent, &vdo0_2_cg_regs, >>>> _shift, \ >>>> + &mtk_clk_gate_ops_setclr, _flags) >>>> + >>>> +static const struct mtk_gate vdo0_clks[] = { >>>> + /* VDO0_0 */ >>>> + GATE_VDO0_0(CLK_VDO0_DISP_OVL0, "vdo0_disp_ovl0", >>>> "top_vpp", 0), >>>> + GATE_VDO0_0(CLK_VDO0_FAKE_ENG0, "vdo0_fake_eng0", >>>> "top_vpp", 2), >>>> + GATE_VDO0_0(CLK_VDO0_DISP_CCORR0, "vdo0_disp_ccorr0", >>>> "top_vpp", 4), >>>> + GATE_VDO0_0(CLK_VDO0_DISP_MUTEX0, "vdo0_disp_mutex0", >>>> "top_vpp", 6), >>>> + GATE_VDO0_0(CLK_VDO0_DISP_GAMMA0, "vdo0_disp_gamma0", >>>> "top_vpp", 8), >>>> + GATE_VDO0_0(CLK_VDO0_DISP_DITHER0, "vdo0_disp_dither0", >>>> "top_vpp", 10), >>>> + GATE_VDO0_0(CLK_VDO0_DISP_WDMA0, "vdo0_disp_wdma0", >>>> "top_vpp", 17), >>>> + GATE_VDO0_0(CLK_VDO0_DISP_RDMA0, "vdo0_disp_rdma0", >>>> "top_vpp", 19), >>>> + GATE_VDO0_0(CLK_VDO0_DSI0, "vdo0_dsi0", "top_vpp", 21), >>>> + GATE_VDO0_0(CLK_VDO0_DSI1, "vdo0_dsi1", "top_vpp", 22), >>>> + GATE_VDO0_0(CLK_VDO0_DSC_WRAP0, "vdo0_dsc_wrap0", >>>> "top_vpp", 23), >>>> + GATE_VDO0_0(CLK_VDO0_VPP_MERGE0, "vdo0_vpp_merge0", >>>> "top_vpp", 24), >>>> + GATE_VDO0_0(CLK_VDO0_DP_INTF0, "vdo0_dp_intf0", >>>> "top_vpp", 25), >>>> + GATE_VDO0_0(CLK_VDO0_DISP_AAL0, "vdo0_disp_aal0", >>>> "top_vpp", 26), >>>> + GATE_VDO0_0(CLK_VDO0_INLINEROT0, "vdo0_inlinerot0", >>>> "top_vpp", 27), >>>> + GATE_VDO0_0(CLK_VDO0_APB_BUS, "vdo0_apb_bus", "top_vpp", >>>> 28), >>>> + GATE_VDO0_0(CLK_VDO0_DISP_COLOR0, "vdo0_disp_color0", >>>> "top_vpp", 29), >>>> + GATE_VDO0_0(CLK_VDO0_MDP_WROT0, "vdo0_mdp_wrot0", >>>> "top_vpp", 30), >>>> + GATE_VDO0_0(CLK_VDO0_DISP_RSZ0, "vdo0_disp_rsz0", >>>> "top_vpp", 31), >>>> + /* VDO0_1 */ >>>> + GATE_VDO0_1(CLK_VDO0_DISP_POSTMASK0, >>>> "vdo0_disp_postmask0", "top_vpp", 0), >>>> + GATE_VDO0_1(CLK_VDO0_FAKE_ENG1, "vdo0_fake_eng1", >>>> "top_vpp", 1), >>>> + GATE_VDO0_1(CLK_VDO0_DL_ASYNC2, "vdo0_dl_async2", >>>> "top_vpp", 5), >>>> + GATE_VDO0_1(CLK_VDO0_DL_RELAY3, "vdo0_dl_relay3", >>>> "top_vpp", 6), >>>> + GATE_VDO0_1(CLK_VDO0_DL_RELAY4, "vdo0_dl_relay4", >>>> "top_vpp", 7), >>>> + GATE_VDO0_1(CLK_VDO0_SMI_GALS, "vdo0_smi_gals", >>>> "top_vpp", 10), >>>> + GATE_VDO0_1(CLK_VDO0_SMI_COMMON, "vdo0_smi_common", >>>> "top_vpp", 11), >>>> + GATE_VDO0_1(CLK_VDO0_SMI_EMI, "vdo0_smi_emi", "top_vpp", >>>> 12), >>>> + GATE_VDO0_1(CLK_VDO0_SMI_IOMMU, "vdo0_smi_iommu", >>>> "top_vpp", 13), >>>> + GATE_VDO0_1(CLK_VDO0_SMI_LARB, "vdo0_smi_larb", >>>> "top_vpp", 14), >>>> + GATE_VDO0_1(CLK_VDO0_SMI_RSI, "vdo0_smi_rsi", "top_vpp", >>>> 15), >>>> + /* VDO0_2 */ >>>> + GATE_VDO0_2(CLK_VDO0_DSI0_DSI, "vdo0_dsi0_dsi", >>>> "top_dsi_occ", 0), >>>> + GATE_VDO0_2(CLK_VDO0_DSI1_DSI, "vdo0_dsi1_dsi", >>>> "top_dsi_occ", 8), >>>> + GATE_VDO0_2_FLAGS(CLK_VDO0_DP_INTF0_DP_INTF, >>>> "vdo0_dp_intf0_dp_intf", >>>> + "top_edp", 16, CLK_SET_RATE_PARENT), >>>> +}; >>>> + >>>> +static int clk_mt8188_vdo0_probe(struct platform_device *pdev) >>>> +{ >>>> + struct device *dev = &pdev->dev; >>>> + struct device_node *node = dev->parent->of_node; >>>> + struct clk_hw_onecell_data *clk_data; >>>> + int r; >>>> + >>>> + clk_data = mtk_alloc_clk_data(CLK_VDO0_NR_CLK); >>>> + if (!clk_data) >>>> + return -ENOMEM; >>>> + >>>> + r = mtk_clk_register_gates(node, vdo0_clks, >>>> ARRAY_SIZE(vdo0_clks), clk_data); >>> >>> This API was changed. Please rebase onto the latest -next and >>> update. >>> >>> Angelo (CC-ed) also mentioned a new simple probe variant for non-DT >>> clock drivers is being developed. He didn't mention a timeline >>> though. >> >> I've already tested the new simple probe variant for non-DT clock >> drivers and >> it works fine on MT8173, MT8192 and MT8195. >> >> Timeline - I should be able to push the part 2 series next week, >> which will include >> more conversion to simple probe and almost all clock drivers changed >> to allow >> building as modules. >> >> Cheers, >> Angelo >> > Thank you for your suggestions. > When the new simple probe variant and building as modules are ready, I > will rebase latest-next and update it. It's *finally* ready as it's fully tested and reviewed. You can go on with a rebase on top of it: https://patchwork.kernel.org/project/linux-mediatek/list/?series=726914 Regards, Angelo
diff --git a/drivers/clk/mediatek/Makefile b/drivers/clk/mediatek/Makefile index 7d09e9fc6538..df78c0777fef 100644 --- a/drivers/clk/mediatek/Makefile +++ b/drivers/clk/mediatek/Makefile @@ -86,7 +86,8 @@ obj-$(CONFIG_COMMON_CLK_MT8186) += clk-mt8186-mcu.o clk-mt8186-topckgen.o clk-mt obj-$(CONFIG_COMMON_CLK_MT8188) += clk-mt8188-apmixedsys.o clk-mt8188-topckgen.o \ clk-mt8188-peri_ao.o clk-mt8188-infra_ao.o \ clk-mt8188-cam.o clk-mt8188-ccu.o clk-mt8188-img.o \ - clk-mt8188-ipe.o clk-mt8188-mfg.o clk-mt8188-vdec.o + clk-mt8188-ipe.o clk-mt8188-mfg.o clk-mt8188-vdec.o \ + clk-mt8188-vdo0.o obj-$(CONFIG_COMMON_CLK_MT8192) += 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-mt8188-vdo0.c b/drivers/clk/mediatek/clk-mt8188-vdo0.c new file mode 100644 index 000000000000..30dd64374ace --- /dev/null +++ b/drivers/clk/mediatek/clk-mt8188-vdo0.c @@ -0,0 +1,134 @@ +// SPDX-License-Identifier: GPL-2.0-only +// +// Copyright (c) 2022 MediaTek Inc. +// Author: Garmin Chang <garmin.chang@mediatek.com> + +#include <linux/clk-provider.h> +#include <linux/platform_device.h> +#include <dt-bindings/clock/mediatek,mt8188-clk.h> + +#include "clk-gate.h" +#include "clk-mtk.h" + +static const struct mtk_gate_regs vdo0_0_cg_regs = { + .set_ofs = 0x104, + .clr_ofs = 0x108, + .sta_ofs = 0x100, +}; + +static const struct mtk_gate_regs vdo0_1_cg_regs = { + .set_ofs = 0x114, + .clr_ofs = 0x118, + .sta_ofs = 0x110, +}; + +static const struct mtk_gate_regs vdo0_2_cg_regs = { + .set_ofs = 0x124, + .clr_ofs = 0x128, + .sta_ofs = 0x120, +}; + +#define GATE_VDO0_0(_id, _name, _parent, _shift) \ + GATE_MTK(_id, _name, _parent, &vdo0_0_cg_regs, _shift, &mtk_clk_gate_ops_setclr) + +#define GATE_VDO0_1(_id, _name, _parent, _shift) \ + GATE_MTK(_id, _name, _parent, &vdo0_1_cg_regs, _shift, &mtk_clk_gate_ops_setclr) + +#define GATE_VDO0_2(_id, _name, _parent, _shift) \ + GATE_MTK(_id, _name, _parent, &vdo0_2_cg_regs, _shift, &mtk_clk_gate_ops_setclr) + +#define GATE_VDO0_2_FLAGS(_id, _name, _parent, _shift, _flags) \ + GATE_MTK_FLAGS(_id, _name, _parent, &vdo0_2_cg_regs, _shift, \ + &mtk_clk_gate_ops_setclr, _flags) + +static const struct mtk_gate vdo0_clks[] = { + /* VDO0_0 */ + GATE_VDO0_0(CLK_VDO0_DISP_OVL0, "vdo0_disp_ovl0", "top_vpp", 0), + GATE_VDO0_0(CLK_VDO0_FAKE_ENG0, "vdo0_fake_eng0", "top_vpp", 2), + GATE_VDO0_0(CLK_VDO0_DISP_CCORR0, "vdo0_disp_ccorr0", "top_vpp", 4), + GATE_VDO0_0(CLK_VDO0_DISP_MUTEX0, "vdo0_disp_mutex0", "top_vpp", 6), + GATE_VDO0_0(CLK_VDO0_DISP_GAMMA0, "vdo0_disp_gamma0", "top_vpp", 8), + GATE_VDO0_0(CLK_VDO0_DISP_DITHER0, "vdo0_disp_dither0", "top_vpp", 10), + GATE_VDO0_0(CLK_VDO0_DISP_WDMA0, "vdo0_disp_wdma0", "top_vpp", 17), + GATE_VDO0_0(CLK_VDO0_DISP_RDMA0, "vdo0_disp_rdma0", "top_vpp", 19), + GATE_VDO0_0(CLK_VDO0_DSI0, "vdo0_dsi0", "top_vpp", 21), + GATE_VDO0_0(CLK_VDO0_DSI1, "vdo0_dsi1", "top_vpp", 22), + GATE_VDO0_0(CLK_VDO0_DSC_WRAP0, "vdo0_dsc_wrap0", "top_vpp", 23), + GATE_VDO0_0(CLK_VDO0_VPP_MERGE0, "vdo0_vpp_merge0", "top_vpp", 24), + GATE_VDO0_0(CLK_VDO0_DP_INTF0, "vdo0_dp_intf0", "top_vpp", 25), + GATE_VDO0_0(CLK_VDO0_DISP_AAL0, "vdo0_disp_aal0", "top_vpp", 26), + GATE_VDO0_0(CLK_VDO0_INLINEROT0, "vdo0_inlinerot0", "top_vpp", 27), + GATE_VDO0_0(CLK_VDO0_APB_BUS, "vdo0_apb_bus", "top_vpp", 28), + GATE_VDO0_0(CLK_VDO0_DISP_COLOR0, "vdo0_disp_color0", "top_vpp", 29), + GATE_VDO0_0(CLK_VDO0_MDP_WROT0, "vdo0_mdp_wrot0", "top_vpp", 30), + GATE_VDO0_0(CLK_VDO0_DISP_RSZ0, "vdo0_disp_rsz0", "top_vpp", 31), + /* VDO0_1 */ + GATE_VDO0_1(CLK_VDO0_DISP_POSTMASK0, "vdo0_disp_postmask0", "top_vpp", 0), + GATE_VDO0_1(CLK_VDO0_FAKE_ENG1, "vdo0_fake_eng1", "top_vpp", 1), + GATE_VDO0_1(CLK_VDO0_DL_ASYNC2, "vdo0_dl_async2", "top_vpp", 5), + GATE_VDO0_1(CLK_VDO0_DL_RELAY3, "vdo0_dl_relay3", "top_vpp", 6), + GATE_VDO0_1(CLK_VDO0_DL_RELAY4, "vdo0_dl_relay4", "top_vpp", 7), + GATE_VDO0_1(CLK_VDO0_SMI_GALS, "vdo0_smi_gals", "top_vpp", 10), + GATE_VDO0_1(CLK_VDO0_SMI_COMMON, "vdo0_smi_common", "top_vpp", 11), + GATE_VDO0_1(CLK_VDO0_SMI_EMI, "vdo0_smi_emi", "top_vpp", 12), + GATE_VDO0_1(CLK_VDO0_SMI_IOMMU, "vdo0_smi_iommu", "top_vpp", 13), + GATE_VDO0_1(CLK_VDO0_SMI_LARB, "vdo0_smi_larb", "top_vpp", 14), + GATE_VDO0_1(CLK_VDO0_SMI_RSI, "vdo0_smi_rsi", "top_vpp", 15), + /* VDO0_2 */ + GATE_VDO0_2(CLK_VDO0_DSI0_DSI, "vdo0_dsi0_dsi", "top_dsi_occ", 0), + GATE_VDO0_2(CLK_VDO0_DSI1_DSI, "vdo0_dsi1_dsi", "top_dsi_occ", 8), + GATE_VDO0_2_FLAGS(CLK_VDO0_DP_INTF0_DP_INTF, "vdo0_dp_intf0_dp_intf", + "top_edp", 16, CLK_SET_RATE_PARENT), +}; + +static int clk_mt8188_vdo0_probe(struct platform_device *pdev) +{ + struct device *dev = &pdev->dev; + struct device_node *node = dev->parent->of_node; + struct clk_hw_onecell_data *clk_data; + int r; + + clk_data = mtk_alloc_clk_data(CLK_VDO0_NR_CLK); + if (!clk_data) + return -ENOMEM; + + r = mtk_clk_register_gates(node, vdo0_clks, ARRAY_SIZE(vdo0_clks), clk_data); + if (r) + goto free_vdo0_data; + + r = of_clk_add_hw_provider(node, of_clk_hw_onecell_get, clk_data); + if (r) + goto unregister_gates; + + platform_set_drvdata(pdev, clk_data); + + return r; + +unregister_gates: + mtk_clk_unregister_gates(vdo0_clks, ARRAY_SIZE(vdo0_clks), clk_data); +free_vdo0_data: + mtk_free_clk_data(clk_data); + return r; +} + +static int clk_mt8188_vdo0_remove(struct platform_device *pdev) +{ + struct device *dev = &pdev->dev; + struct device_node *node = dev->parent->of_node; + struct clk_hw_onecell_data *clk_data = platform_get_drvdata(pdev); + + of_clk_del_provider(node); + mtk_clk_unregister_gates(vdo0_clks, ARRAY_SIZE(vdo0_clks), clk_data); + mtk_free_clk_data(clk_data); + + return 0; +} + +static struct platform_driver clk_mt8188_vdo0_drv = { + .probe = clk_mt8188_vdo0_probe, + .remove = clk_mt8188_vdo0_remove, + .driver = { + .name = "clk-mt8188-vdo0", + }, +}; +builtin_platform_driver(clk_mt8188_vdo0_drv);
Add MT8188 vdosys0 clock controller which provides clock gate control in video system. This is integrated with mtk-mmsys driver which will populate device by platform_device_register_data to start vdosys clock driver. Signed-off-by: Garmin.Chang <Garmin.Chang@mediatek.com> --- drivers/clk/mediatek/Makefile | 3 +- drivers/clk/mediatek/clk-mt8188-vdo0.c | 134 +++++++++++++++++++++++++ 2 files changed, 136 insertions(+), 1 deletion(-) create mode 100644 drivers/clk/mediatek/clk-mt8188-vdo0.c