[PATCH v2 08/14] soc: mediatek: add mtk-mmsys config API for mt8195 vdosys1
Nancy.Lin
nancy.lin at mediatek.com
Mon Aug 16 02:59:35 UTC 2021
Hi Matthias,
Thanks for the review.
On Fri, 2021-08-06 at 17:30 +0200, Matthias Brugger wrote:
>
> On 22/07/2021 11:45, Nancy.Lin wrote:
> > Add mmsys config API.
>
> This patch is doing a lot of things, it adds a "config" and it adds
> cmdq
> support. Please explain better in the commit message what the config
> is for.
> Please add comments to the different values of struct
> mtk_mmsys_config.
>
OK, I will explain in more detail in the commit message.
> I understand that cmdq is optional, so please make addition to cmdq a
> separate
> patch.
OK, I will add cmdq support in another patch.
> I'm a bit puzzled about that fact, can you please explain who you get
> the HW to
> behave the same way when you write the same value and offset to
> mmsys-regs and
> via cmdq.
>
> Thanks,
> Matthias
The mmsys config register settings need to take effect with the other
HW settings(like OVL_ADAPTOR...) at the same vblanking time.
If we use CPU to write the mmsys reg, we can't guarantee all the
settings can be written in the same vblanking time.
Cmdq is used for this purpose. We prepare all the related HW settings
in one cmdq packet.
The first command in the packet is "wait stream done", and then
following with all the HW settings. After the cmdq packet is flush to
GCE HW.
The GCE waits for the "stream done event" to coming and then starts
flushing all the HW settings. This can guarantee all the settings flush
in the same vblanking.
Regards,
Nancy
> >
> > Signed-off-by: Nancy.Lin <nancy.lin at mediatek.com>
> > ---
> > drivers/soc/mediatek/mt8195-mmsys.h | 38 ++++++++++++++++++++
> > drivers/soc/mediatek/mtk-mmsys.c | 50
> > ++++++++++++++++++++++++++
> > drivers/soc/mediatek/mtk-mmsys.h | 10 ++++++
> > include/linux/soc/mediatek/mtk-mmsys.h | 18 ++++++++++
> > 4 files changed, 116 insertions(+)
> >
> > diff --git a/drivers/soc/mediatek/mt8195-mmsys.h
> > b/drivers/soc/mediatek/mt8195-mmsys.h
> > index 104ba575f765..4bdb2087250c 100644
> > --- a/drivers/soc/mediatek/mt8195-mmsys.h
> > +++ b/drivers/soc/mediatek/mt8195-mmsys.h
> > @@ -154,6 +154,18 @@
> > #define DISP_DP_INTF0_SEL_IN_FROM_VDO0_MERGE_DL_ASYNC_MOUT (1 <<
> > 0)
> > #define DISP_DP_INTF0_SEL_IN_FROM_VDO0_DSC_DL_ASYNC_MOUT (2 <<
> > 0)
> >
> > +#define MT8195_VDO1_MERGE0_ASYNC_CFG_WD 0xe30
> > +#define MT8195_VDO1_MERGE1_ASYNC_CFG_WD 0xe40
> > +#define MT8195_VDO1_MERGE2_ASYNC_CFG_WD 0xe50
> > +#define MT8195_VDO1_MERGE3_ASYNC_CFG_WD 0xe60
> > +#define MT8195_VDO1_HDRBE_ASYNC_CFG_WD 0xe70
> > +#define MT8195_VDO1_HDR_TOP_CFG 0xd00
> > +#define MT8195_VDO1_MIXER_IN1_ALPHA 0xd30
> > +#define MT8195_VDO1_MIXER_IN2_ALPHA 0xd34
> > +#define MT8195_VDO1_MIXER_IN3_ALPHA 0xd38
> > +#define MT8195_VDO1_MIXER_IN4_ALPHA 0xd3c
> > +#define MT8195_VDO1_MIXER_IN4_PAD 0xd4c
> > +
> > static const struct mtk_mmsys_routes mmsys_mt8195_routing_table[]
> > = {
> > {
> > DDP_COMPONENT_OVL0, DDP_COMPONENT_RDMA0,
> > @@ -261,4 +273,30 @@ static const struct mtk_mmsys_routes
> > mmsys_mt8195_routing_table[] = {
> > }
> > };
> >
> > +static const struct mtk_mmsys_config mmsys_mt8195_config_table[] =
> > {
> > + { MMSYS_CONFIG_MERGE_ASYNC_WIDTH, 0,
> > MT8195_VDO1_MERGE0_ASYNC_CFG_WD, GENMASK(13, 0), 0},
> > + { MMSYS_CONFIG_MERGE_ASYNC_HEIGHT, 0,
> > MT8195_VDO1_MERGE0_ASYNC_CFG_WD, GENMASK(29, 16), 16},
> > + { MMSYS_CONFIG_MERGE_ASYNC_WIDTH, 1,
> > MT8195_VDO1_MERGE1_ASYNC_CFG_WD, GENMASK(13, 0), 0},
> > + { MMSYS_CONFIG_MERGE_ASYNC_HEIGHT, 1,
> > MT8195_VDO1_MERGE1_ASYNC_CFG_WD, GENMASK(29, 16), 16},
> > + { MMSYS_CONFIG_MERGE_ASYNC_WIDTH, 2,
> > MT8195_VDO1_MERGE2_ASYNC_CFG_WD, GENMASK(13, 0), 0},
> > + { MMSYS_CONFIG_MERGE_ASYNC_HEIGHT, 2,
> > MT8195_VDO1_MERGE2_ASYNC_CFG_WD, GENMASK(29, 16), 16},
> > + { MMSYS_CONFIG_MERGE_ASYNC_WIDTH, 3,
> > MT8195_VDO1_MERGE3_ASYNC_CFG_WD, GENMASK(13, 0), 0},
> > + { MMSYS_CONFIG_MERGE_ASYNC_HEIGHT, 3,
> > MT8195_VDO1_MERGE3_ASYNC_CFG_WD, GENMASK(29, 16), 16},
> > + { MMSYS_CONFIG_HDR_BE_ASYNC_WIDTH, 0,
> > MT8195_VDO1_HDRBE_ASYNC_CFG_WD, GENMASK(13, 0), 0},
> > + { MMSYS_CONFIG_HDR_BE_ASYNC_HEIGHT, 0,
> > MT8195_VDO1_HDRBE_ASYNC_CFG_WD, GENMASK(29, 16), 16},
> > + { MMSYS_CONFIG_MIXER_IN_ALPHA_ODD, 1,
> > MT8195_VDO1_MIXER_IN1_ALPHA, GENMASK(8, 0), 0},
> > + { MMSYS_CONFIG_MIXER_IN_ALPHA_EVEN, 1,
> > MT8195_VDO1_MIXER_IN1_ALPHA, GENMASK(24, 16), 16},
> > + { MMSYS_CONFIG_MIXER_IN_ALPHA_ODD, 2,
> > MT8195_VDO1_MIXER_IN2_ALPHA, GENMASK(8, 0), 0},
> > + { MMSYS_CONFIG_MIXER_IN_ALPHA_EVEN, 2,
> > MT8195_VDO1_MIXER_IN2_ALPHA, GENMASK(24, 16), 16},
> > + { MMSYS_CONFIG_MIXER_IN_ALPHA_ODD, 3,
> > MT8195_VDO1_MIXER_IN3_ALPHA, GENMASK(8, 0), 0},
> > + { MMSYS_CONFIG_MIXER_IN_ALPHA_EVEN, 3,
> > MT8195_VDO1_MIXER_IN3_ALPHA, GENMASK(24, 16), 16},
> > + { MMSYS_CONFIG_MIXER_IN_ALPHA_ODD, 4,
> > MT8195_VDO1_MIXER_IN4_ALPHA, GENMASK(8, 0), 0},
> > + { MMSYS_CONFIG_MIXER_IN_ALPHA_EVEN, 4,
> > MT8195_VDO1_MIXER_IN4_ALPHA, GENMASK(24, 16), 16},
> > + { MMSYS_CONFIG_MIXER_IN_CH_SWAP, 4, MT8195_VDO1_MIXER_IN4_PAD,
> > GENMASK(4, 4), 4},
> > + { MMSYS_CONFIG_HDR_ALPHA_SEL, 1, MT8195_VDO1_HDR_TOP_CFG,
> > GENMASK(20, 20), 20},
> > + { MMSYS_CONFIG_HDR_ALPHA_SEL, 2, MT8195_VDO1_HDR_TOP_CFG,
> > GENMASK(21, 21), 21},
> > + { MMSYS_CONFIG_HDR_ALPHA_SEL, 3, MT8195_VDO1_HDR_TOP_CFG,
> > GENMASK(22, 22), 22},
> > + { MMSYS_CONFIG_HDR_ALPHA_SEL, 4, MT8195_VDO1_HDR_TOP_CFG,
> > GENMASK(23, 23), 23},
> > +};
> > +
> > #endif /* __SOC_MEDIATEK_MT8195_MMSYS_H */
> > diff --git a/drivers/soc/mediatek/mtk-mmsys.c
> > b/drivers/soc/mediatek/mtk-mmsys.c
> > index 9e31aad6c5c8..d0f4a407f8f8 100644
> > --- a/drivers/soc/mediatek/mtk-mmsys.c
> > +++ b/drivers/soc/mediatek/mtk-mmsys.c
> > @@ -63,10 +63,13 @@ static const struct mtk_mmsys_driver_data
> > mt8195_vdosys1_driver_data = {
> > .clk_driver = "clk-mt8195-vdo1",
> > .routes = mmsys_mt8195_routing_table,
> > .num_routes = ARRAY_SIZE(mmsys_mt8195_routing_table),
> > + .config = mmsys_mt8195_config_table,
> > + .num_configs = ARRAY_SIZE(mmsys_mt8195_config_table),
> > };
> >
> > struct mtk_mmsys {
> > void __iomem *regs;
> > + struct cmdq_client_reg cmdq_base;
> > const struct mtk_mmsys_driver_data *data;
> > };
> >
> > @@ -104,6 +107,47 @@ void mtk_mmsys_ddp_disconnect(struct device
> > *dev,
> > }
> > EXPORT_SYMBOL_GPL(mtk_mmsys_ddp_disconnect);
> >
> > +void mtk_mmsys_ddp_config(struct device *dev, enum
> > mtk_mmsys_config_type config,
> > + u32 id, u32 val, struct cmdq_pkt *cmdq_pkt)
> > +{
> > + struct mtk_mmsys *mmsys = dev_get_drvdata(dev);
> > + const struct mtk_mmsys_config *mmsys_config = mmsys->data-
> > >config;
> > + u32 reg_val;
> > + u32 mask;
> > + u32 offset;
> > + int i;
> > +
> > + if (!mmsys->data->num_configs)
> > + return;
> > +
> > + for (i = 0; i < mmsys->data->num_configs; i++)
> > + if (config == mmsys_config[i].config && id ==
> > mmsys_config[i].id)
> > + break;
> > +
> > + if (i == mmsys->data->num_configs)
> > + return;
> > +
> > + offset = mmsys_config[i].addr;
> > + mask = mmsys_config[i].mask;
> > + reg_val = val << mmsys_config[i].shift;
> > +
> > +#if IS_REACHABLE(CONFIG_MTK_CMDQ)
> > + if (cmdq_pkt && mmsys->cmdq_base.size) {
> > + cmdq_pkt_write_mask(cmdq_pkt, mmsys->cmdq_base.subsys,
> > + mmsys->cmdq_base.offset + offset,
> > reg_val,
> > + mask);
> > + } else {
> > +#endif
> > + u32 tmp = readl(mmsys->regs + offset);
> > +
> > + tmp = (tmp & ~mask) | reg_val;
> > + writel(tmp, mmsys->regs + offset);
> > +#if IS_REACHABLE(CONFIG_MTK_CMDQ)
> > + }
> > +#endif
> > +}
> > +EXPORT_SYMBOL_GPL(mtk_mmsys_ddp_config);
> > +
> > static int mtk_mmsys_probe(struct platform_device *pdev)
> > {
> > struct device *dev = &pdev->dev;
> > @@ -124,6 +168,12 @@ static int mtk_mmsys_probe(struct
> > platform_device *pdev)
> > }
> >
> > mmsys->data = of_device_get_match_data(&pdev->dev);
> > +
> > +#if IS_REACHABLE(CONFIG_MTK_CMDQ)
> > + ret = cmdq_dev_get_client_reg(dev, &mmsys->cmdq_base, 0);
> > + if (ret)
> > + dev_dbg(dev, "No mediatek,gce-client-reg!\n");
> > +#endif
> > platform_set_drvdata(pdev, mmsys);
> >
> > clks = platform_device_register_data(&pdev->dev, mmsys->data-
> > >clk_driver,
> > diff --git a/drivers/soc/mediatek/mtk-mmsys.h
> > b/drivers/soc/mediatek/mtk-mmsys.h
> > index a760a34e6eca..084b1f5f3c88 100644
> > --- a/drivers/soc/mediatek/mtk-mmsys.h
> > +++ b/drivers/soc/mediatek/mtk-mmsys.h
> > @@ -73,10 +73,20 @@ struct mtk_mmsys_routes {
> > u32 val;
> > };
> >
> > +struct mtk_mmsys_config {
> > + enum mtk_mmsys_config_type config;
> > + u32 id;
> > + u32 addr;
> > + u32 mask;
> > + u32 shift;
> > +};
> > +
> > struct mtk_mmsys_driver_data {
> > const char *clk_driver;
> > const struct mtk_mmsys_routes *routes;
> > const unsigned int num_routes;
> > + const struct mtk_mmsys_config *config;
> > + const unsigned int num_configs;
> > };
> >
> > /*
> > diff --git a/include/linux/soc/mediatek/mtk-mmsys.h
> > b/include/linux/soc/mediatek/mtk-mmsys.h
> > index 338c71570aeb..ba3925661cc9 100644
> > --- a/include/linux/soc/mediatek/mtk-mmsys.h
> > +++ b/include/linux/soc/mediatek/mtk-mmsys.h
> > @@ -6,6 +6,10 @@
> > #ifndef __MTK_MMSYS_H
> > #define __MTK_MMSYS_H
> >
> > +#include <linux/mailbox_controller.h>
> > +#include <linux/mailbox/mtk-cmdq-mailbox.h>
> > +#include <linux/soc/mediatek/mtk-cmdq.h>
> > +
> > enum mtk_ddp_comp_id;
> > struct device;
> >
> > @@ -54,6 +58,17 @@ enum mtk_ddp_comp_id {
> > DDP_COMPONENT_ID_MAX,
> > };
> >
> > +enum mtk_mmsys_config_type {
> > + MMSYS_CONFIG_MERGE_ASYNC_WIDTH,
> > + MMSYS_CONFIG_MERGE_ASYNC_HEIGHT,
> > + MMSYS_CONFIG_HDR_BE_ASYNC_WIDTH,
> > + MMSYS_CONFIG_HDR_BE_ASYNC_HEIGHT,
> > + MMSYS_CONFIG_HDR_ALPHA_SEL,
> > + MMSYS_CONFIG_MIXER_IN_ALPHA_ODD,
> > + MMSYS_CONFIG_MIXER_IN_ALPHA_EVEN,
> > + MMSYS_CONFIG_MIXER_IN_CH_SWAP,
> > +};
> > +
> > void mtk_mmsys_ddp_connect(struct device *dev,
> > enum mtk_ddp_comp_id cur,
> > enum mtk_ddp_comp_id next);
> > @@ -62,4 +77,7 @@ void mtk_mmsys_ddp_disconnect(struct device *dev,
> > enum mtk_ddp_comp_id cur,
> > enum mtk_ddp_comp_id next);
> >
> > +void mtk_mmsys_ddp_config(struct device *dev, enum
> > mtk_mmsys_config_type config,
> > + u32 id, u32 val, struct cmdq_pkt *cmdq_pkt);
> > +
> > #endif /* __MTK_MMSYS_H */
> >
More information about the dri-devel
mailing list