We choose Mixer as CRC generator in OVL adaptor since
its frame done event will trigger vblanks, we can know
when is safe to retrieve CRC of the frame.

In OVL adaptor, there's no image procession after Mixer,
unlike the OVL in VDOSYS0, Mixer's CRC will include all
the effects that are applied to the frame.

Signed-off-by: Hsiao Chien Sung <shawn.s...@mediatek.com>
---
 drivers/gpu/drm/mediatek/mtk_disp_drv.h       |  3 +
 .../gpu/drm/mediatek/mtk_disp_ovl_adaptor.c   | 21 +++++++
 drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c   |  3 +
 drivers/gpu/drm/mediatek/mtk_ethdr.c          | 62 +++++++++++++++++++
 drivers/gpu/drm/mediatek/mtk_ethdr.h          |  5 ++
 5 files changed, 94 insertions(+)

diff --git a/drivers/gpu/drm/mediatek/mtk_disp_drv.h 
b/drivers/gpu/drm/mediatek/mtk_disp_drv.h
index 7ef8dbb07c2db..7ff4a5f8d7926 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_drv.h
+++ b/drivers/gpu/drm/mediatek/mtk_disp_drv.h
@@ -138,6 +138,9 @@ const u32 *mtk_ovl_adaptor_get_formats(struct device *dev);
 size_t mtk_ovl_adaptor_get_num_formats(struct device *dev);
 enum drm_mode_status mtk_ovl_adaptor_mode_valid(struct device *dev,
                                                const struct drm_display_mode 
*mode);
+size_t mtk_ovl_adaptor_crc_cnt(struct device *dev);
+u32 *mtk_ovl_adaptor_crc_entry(struct device *dev);
+void mtk_ovl_adaptor_crc_read(struct device *dev);
 
 void mtk_rdma_bypass_shadow(struct device *dev);
 int mtk_rdma_clk_enable(struct device *dev);
diff --git a/drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c 
b/drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c
index 68a20312ac6f1..61a8cf9b686ab 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c
+++ b/drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c
@@ -206,6 +206,27 @@ void mtk_ovl_adaptor_layer_config(struct device *dev, 
unsigned int idx,
        mtk_ethdr_layer_config(ethdr, idx, state, cmdq_pkt);
 }
 
+size_t mtk_ovl_adaptor_crc_cnt(struct device *dev)
+{
+       struct mtk_disp_ovl_adaptor *ovl_adaptor = dev_get_drvdata(dev);
+
+       return 
mtk_ethdr_crc_cnt(ovl_adaptor->ovl_adaptor_comp[OVL_ADAPTOR_ETHDR0]);
+}
+
+u32 *mtk_ovl_adaptor_crc_entry(struct device *dev)
+{
+       struct mtk_disp_ovl_adaptor *ovl_adaptor = dev_get_drvdata(dev);
+
+       return 
mtk_ethdr_crc_entry(ovl_adaptor->ovl_adaptor_comp[OVL_ADAPTOR_ETHDR0]);
+}
+
+void mtk_ovl_adaptor_crc_read(struct device *dev)
+{
+       struct mtk_disp_ovl_adaptor *ovl_adaptor = dev_get_drvdata(dev);
+
+       mtk_ethdr_crc_read(ovl_adaptor->ovl_adaptor_comp[OVL_ADAPTOR_ETHDR0]);
+}
+
 void mtk_ovl_adaptor_config(struct device *dev, unsigned int w,
                            unsigned int h, unsigned int vrefresh,
                            unsigned int bpc, struct cmdq_pkt *cmdq_pkt)
diff --git a/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c 
b/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c
index ea04b2769ae8b..43368eb342e2e 100644
--- a/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c
+++ b/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c
@@ -404,6 +404,9 @@ static const struct mtk_ddp_comp_funcs ddp_ovl_adaptor = {
        .clk_enable = mtk_ovl_adaptor_clk_enable,
        .clk_disable = mtk_ovl_adaptor_clk_disable,
        .config = mtk_ovl_adaptor_config,
+       .crc_cnt = mtk_ovl_adaptor_crc_cnt,
+       .crc_entry = mtk_ovl_adaptor_crc_entry,
+       .crc_read = mtk_ovl_adaptor_crc_read,
        .start = mtk_ovl_adaptor_start,
        .stop = mtk_ovl_adaptor_stop,
        .layer_nr = mtk_ovl_adaptor_layer_nr,
diff --git a/drivers/gpu/drm/mediatek/mtk_ethdr.c 
b/drivers/gpu/drm/mediatek/mtk_ethdr.c
index e95331c068151..30eb2c3d95c0b 100644
--- a/drivers/gpu/drm/mediatek/mtk_ethdr.c
+++ b/drivers/gpu/drm/mediatek/mtk_ethdr.c
@@ -24,6 +24,9 @@
 #define MIX_FME_CPL_INTEN                      BIT(1)
 #define MIX_INTSTA                     0x8
 #define MIX_EN                         0xc
+#define MIX_TRIG                       0x10
+#define MIX_TRIG_CRC_EN                                BIT(8)
+#define MIX_TRIG_CRC_RST                       BIT(9)
 #define MIX_RST                                0x14
 #define MIX_ROI_SIZE                   0x18
 #define MIX_DATAPATH_CON               0x1c
@@ -39,6 +42,11 @@
 #define PREMULTI_SOURCE                                (3 << 12)
 #define MIX_L_SRC_SIZE(n)              (0x30 + 0x18 * (n))
 #define MIX_L_SRC_OFFSET(n)            (0x34 + 0x18 * (n))
+
+/* CRC register offsets for odd and even lines */
+#define MIX_CRC_ODD                    0x110
+#define MIX_CRC_EVEN                   0x114
+
 #define MIX_FUNC_DCM0                  0x120
 #define MIX_FUNC_DCM1                  0x124
 #define MIX_FUNC_DCM_ENABLE                    0xffffffff
@@ -82,6 +90,7 @@ struct mtk_ethdr {
        void                    *vblank_cb_data;
        int                     irq;
        struct reset_control    *reset_ctl;
+       struct mtk_drm_crc      crc;
 };
 
 static const char * const ethdr_clk_str[] = {
@@ -100,6 +109,32 @@ static const char * const ethdr_clk_str[] = {
        "vdo_be_async",
 };
 
+static const u32 ethdr_crc_ofs[] = {
+       MIX_CRC_ODD,
+       MIX_CRC_EVEN,
+};
+
+size_t mtk_ethdr_crc_cnt(struct device *dev)
+{
+       struct mtk_ethdr *priv = dev_get_drvdata(dev);
+
+       return priv->crc.cnt;
+}
+
+u32 *mtk_ethdr_crc_entry(struct device *dev)
+{
+       struct mtk_ethdr *priv = dev_get_drvdata(dev);
+
+       return priv->crc.va;
+}
+
+void mtk_ethdr_crc_read(struct device *dev)
+{
+       struct mtk_ethdr *priv = dev_get_drvdata(dev);
+
+       mtk_drm_crc_read(&priv->crc, priv->ethdr_comp[ETHDR_MIXER].regs);
+}
+
 void mtk_ethdr_register_vblank_cb(struct device *dev,
                                  void (*vblank_cb)(void *),
                                  void *vblank_cb_data)
@@ -256,6 +291,13 @@ void mtk_ethdr_start(struct device *dev)
        struct mtk_ethdr_comp *mixer = &priv->ethdr_comp[ETHDR_MIXER];
 
        writel(1, mixer->regs + MIX_EN);
+
+       if (priv->crc.cnt) {
+               writel(MIX_TRIG_CRC_EN, mixer->regs + MIX_TRIG);
+#if IS_REACHABLE(CONFIG_MTK_CMDQ)
+               mtk_drm_crc_cmdq_start(&priv->crc);
+#endif
+       }
 }
 
 void mtk_ethdr_stop(struct device *dev)
@@ -263,6 +305,9 @@ void mtk_ethdr_stop(struct device *dev)
        struct mtk_ethdr *priv = dev_get_drvdata(dev);
        struct mtk_ethdr_comp *mixer = &priv->ethdr_comp[ETHDR_MIXER];
 
+#if IS_REACHABLE(CONFIG_MTK_CMDQ)
+       mtk_drm_crc_cmdq_stop(&priv->crc);
+#endif
        writel(0, mixer->regs + MIX_EN);
        writel(1, mixer->regs + MIX_RST);
        reset_control_reset(priv->reset_ctl);
@@ -317,6 +362,10 @@ static int mtk_ethdr_probe(struct platform_device *pdev)
        if (!priv)
                return -ENOMEM;
 
+       mtk_drm_crc_init(&priv->crc,
+                        ethdr_crc_ofs, ARRAY_SIZE(ethdr_crc_ofs),
+                        MIX_TRIG, MIX_TRIG_CRC_RST);
+
        for (i = 0; i < ETHDR_ID_MAX; i++) {
                priv->ethdr_comp[i].dev = dev;
                priv->ethdr_comp[i].regs = of_iomap(dev->of_node, i);
@@ -325,6 +374,16 @@ static int mtk_ethdr_probe(struct platform_device *pdev)
                                              &priv->ethdr_comp[i].cmdq_base, 
i);
                if (ret)
                        dev_dbg(dev, "get mediatek,gce-client-reg fail!\n");
+
+               if (i == ETHDR_MIXER) {
+                       if (of_property_read_u32_index(dev->of_node,
+                                                      "mediatek,gce-events", i,
+                                                      &priv->crc.cmdq_event)) {
+                               dev_warn(dev, "failed to get gce-events for 
crc\n");
+                       }
+                       priv->crc.cmdq_reg = &priv->ethdr_comp[i].cmdq_base;
+                       mtk_drm_crc_cmdq_create(dev, &priv->crc);
+               }
 #endif
                dev_dbg(dev, "[DRM]regs:0x%p, node:%d\n", 
priv->ethdr_comp[i].regs, i);
        }
@@ -365,6 +424,9 @@ static int mtk_ethdr_probe(struct platform_device *pdev)
 
 static int mtk_ethdr_remove(struct platform_device *pdev)
 {
+       struct mtk_ethdr *priv = dev_get_drvdata(&pdev->dev);
+
+       mtk_drm_crc_destroy(&priv->crc);
        component_del(&pdev->dev, &mtk_ethdr_component_ops);
        return 0;
 }
diff --git a/drivers/gpu/drm/mediatek/mtk_ethdr.h 
b/drivers/gpu/drm/mediatek/mtk_ethdr.h
index 81af9edea3f74..d17d7256bd120 100644
--- a/drivers/gpu/drm/mediatek/mtk_ethdr.h
+++ b/drivers/gpu/drm/mediatek/mtk_ethdr.h
@@ -22,4 +22,9 @@ void mtk_ethdr_register_vblank_cb(struct device *dev,
 void mtk_ethdr_unregister_vblank_cb(struct device *dev);
 void mtk_ethdr_enable_vblank(struct device *dev);
 void mtk_ethdr_disable_vblank(struct device *dev);
+
+size_t mtk_ethdr_crc_cnt(struct device *dev);
+u32 *mtk_ethdr_crc_entry(struct device *dev);
+void mtk_ethdr_crc_read(struct device *dev);
+
 #endif
-- 
2.18.0

Reply via email to