Re: [PATCH v1 06/10] drm/mediatek: add ETHDR support for MT8195

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi, Nancy:

Nancy.Lin <nancy.lin@xxxxxxxxxxxx> 於 2021年7月17日 週六 下午5:04寫道:
>
> Add ETHDR module files:
> ETHDR is designed for HDR video and graphics conversion in the external
> display path. It handles multiple HDR input types and performs tone
> mapping, color space/color format conversion, and then combines
> different layers, output the required HDR or SDR signal to the
> subsequent display path.
>
> Signed-off-by: Nancy.Lin <nancy.lin@xxxxxxxxxxxx>
> ---
>  drivers/gpu/drm/mediatek/Makefile           |   3 +-
>  drivers/gpu/drm/mediatek/mtk_disp_drv.h     |   8 +
>  drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c |  11 +
>  drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.h |   1 +
>  drivers/gpu/drm/mediatek/mtk_drm_drv.c      |   4 +
>  drivers/gpu/drm/mediatek/mtk_drm_drv.h      |   1 +
>  drivers/gpu/drm/mediatek/mtk_ethdr.c        | 537 ++++++++++++++++++++
>  drivers/gpu/drm/mediatek/mtk_ethdr.h        |  20 +
>  8 files changed, 584 insertions(+), 1 deletion(-)
>  create mode 100644 drivers/gpu/drm/mediatek/mtk_ethdr.c
>  create mode 100644 drivers/gpu/drm/mediatek/mtk_ethdr.h
>
> diff --git a/drivers/gpu/drm/mediatek/Makefile b/drivers/gpu/drm/mediatek/Makefile
> index 27c89847d43b..fcce08710cef 100644
> --- a/drivers/gpu/drm/mediatek/Makefile
> +++ b/drivers/gpu/drm/mediatek/Makefile
> @@ -13,7 +13,8 @@ mediatek-drm-y := mtk_disp_ccorr.o \
>                   mtk_drm_gem.o \
>                   mtk_drm_plane.o \
>                   mtk_dsi.o \
> -                 mtk_dpi.o
> +                 mtk_dpi.o \
> +                 mtk_ethdr.o
>
>  obj-$(CONFIG_DRM_MEDIATEK) += mediatek-drm.o
>
> diff --git a/drivers/gpu/drm/mediatek/mtk_disp_drv.h b/drivers/gpu/drm/mediatek/mtk_disp_drv.h
> index 3e27ce7fef57..7227ffbc3eae 100644
> --- a/drivers/gpu/drm/mediatek/mtk_disp_drv.h
> +++ b/drivers/gpu/drm/mediatek/mtk_disp_drv.h
> @@ -105,4 +105,12 @@ void mtk_rdma_enable_vblank(struct device *dev,
>                             void *vblank_cb_data);
>  void mtk_rdma_disable_vblank(struct device *dev);
>
> +int mtk_ethdr_clk_enable(struct device *dev);
> +void mtk_ethdr_clk_disable(struct device *dev);
> +void mtk_ethdr_config(struct device *dev, unsigned int w,
> +                     unsigned int h, unsigned int vrefresh,
> +                     unsigned int bpc, struct cmdq_pkt *cmdq_pkt);
> +void mtk_ethdr_start(struct device *dev);
> +void mtk_ethdr_stop(struct device *dev);
> +
>  #endif
> diff --git a/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c b/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c
> index 9125d0f6352f..3fa86f12feb4 100644
> --- a/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c
> +++ b/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c
> @@ -355,6 +355,14 @@ static const struct mtk_ddp_comp_funcs ddp_ufoe = {
>         .start = mtk_ufoe_start,
>  };
>
> +static const struct mtk_ddp_comp_funcs ddp_ethdr = {
> +       .clk_enable = mtk_ethdr_clk_enable,
> +       .clk_disable = mtk_ethdr_clk_disable,
> +       .config = mtk_ethdr_config,
> +       .start = mtk_ethdr_start,
> +       .stop = mtk_ethdr_stop,
> +};
> +
>  static const char * const mtk_ddp_comp_stem[MTK_DDP_COMP_TYPE_MAX] = {
>         [MTK_DISP_OVL] = "ovl",
>         [MTK_DISP_OVL_2L] = "ovl-2l",
> @@ -363,6 +371,7 @@ static const char * const mtk_ddp_comp_stem[MTK_DDP_COMP_TYPE_MAX] = {
>         [MTK_DISP_COLOR] = "color",
>         [MTK_DISP_CCORR] = "ccorr",
>         [MTK_DISP_AAL] = "aal",
> +       [MTK_DISP_ETHDR] = "ethdr",
>         [MTK_DISP_GAMMA] = "gamma",
>         [MTK_DISP_DITHER] = "dither",
>         [MTK_DISP_UFOE] = "ufoe",
> @@ -399,6 +408,7 @@ static const struct mtk_ddp_comp_match mtk_ddp_matches[DDP_COMPONENT_ID_MAX] = {
>         [DDP_COMPONENT_DSI1]    = { MTK_DSI,            1, &ddp_dsi },
>         [DDP_COMPONENT_DSI2]    = { MTK_DSI,            2, &ddp_dsi },
>         [DDP_COMPONENT_DSI3]    = { MTK_DSI,            3, &ddp_dsi },
> +       [DDP_COMPONENT_ETHDR]   = { MTK_DISP_ETHDR,     0, &ddp_ethdr},
>         [DDP_COMPONENT_GAMMA]   = { MTK_DISP_GAMMA,     0, &ddp_gamma },
>         [DDP_COMPONENT_MERGE0]  = { MTK_DISP_MERGE,     0, &ddp_merge },
>         [DDP_COMPONENT_MERGE1]  = { MTK_DISP_MERGE,     1, &ddp_merge },
> @@ -536,6 +546,7 @@ int mtk_ddp_comp_init(struct device_node *node, struct mtk_ddp_comp *comp,
>             type == MTK_DISP_CCORR ||
>             type == MTK_DISP_COLOR ||
>             type == MTK_DISP_DSC ||
> +           type == MTK_DISP_ETHDR ||
>             type == MTK_DISP_GAMMA ||
>             type == MTK_DISP_MERGE ||
>             type == MTK_DISP_OVL ||
> diff --git a/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.h b/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.h
> index 0afd78c0bc92..f55efba6e744 100644
> --- a/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.h
> +++ b/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.h
> @@ -36,6 +36,7 @@ enum mtk_ddp_comp_type {
>         MTK_DISP_BLS,
>         MTK_DISP_DSC,
>         MTK_DISP_MERGE,
> +       MTK_DISP_ETHDR,
>         MTK_DDP_COMP_TYPE_MAX,
>  };
>
> diff --git a/drivers/gpu/drm/mediatek/mtk_drm_drv.c b/drivers/gpu/drm/mediatek/mtk_drm_drv.c
> index 11c25daf05d8..ace958a34bb5 100644
> --- a/drivers/gpu/drm/mediatek/mtk_drm_drv.c
> +++ b/drivers/gpu/drm/mediatek/mtk_drm_drv.c
> @@ -480,6 +480,8 @@ static const struct of_device_id mtk_ddp_comp_dt_ids[] = {
>           .data = (void *)MTK_DISP_PWM },
>         { .compatible = "mediatek,mt8173-disp-od",
>           .data = (void *)MTK_DISP_OD },
> +       { .compatible = "mediatek,mt8195-disp-ethdr",
> +         .data = (void *)MTK_DISP_ETHDR },
>         { }
>  };
>
> @@ -567,6 +569,7 @@ static int mtk_drm_probe(struct platform_device *pdev)
>                 if (comp_type == MTK_DISP_CCORR ||
>                     comp_type == MTK_DISP_COLOR ||
>                     comp_type == MTK_DISP_DSC ||
> +                   comp_type == MTK_DISP_ETHDR ||
>                     comp_type == MTK_DISP_GAMMA ||
>                     comp_type == MTK_DISP_MERGE ||
>                     comp_type == MTK_DISP_OVL ||
> @@ -676,6 +679,7 @@ static struct platform_driver * const mtk_drm_drivers[] = {
>         &mtk_dpi_driver,
>         &mtk_drm_platform_driver,
>         &mtk_dsi_driver,
> +       &mtk_ethdr_driver,
>  };
>
>  static int __init mtk_drm_init(void)
> diff --git a/drivers/gpu/drm/mediatek/mtk_drm_drv.h b/drivers/gpu/drm/mediatek/mtk_drm_drv.h
> index c4d802a43531..c87ebb5309d0 100644
> --- a/drivers/gpu/drm/mediatek/mtk_drm_drv.h
> +++ b/drivers/gpu/drm/mediatek/mtk_drm_drv.h
> @@ -55,5 +55,6 @@ extern struct platform_driver mtk_disp_dsc_driver;
>  extern struct platform_driver mtk_disp_merge_driver;
>  extern struct platform_driver mtk_dpi_driver;
>  extern struct platform_driver mtk_dsi_driver;
> +extern struct platform_driver mtk_ethdr_driver;
>
>  #endif /* MTK_DRM_DRV_H */
> diff --git a/drivers/gpu/drm/mediatek/mtk_ethdr.c b/drivers/gpu/drm/mediatek/mtk_ethdr.c
> new file mode 100644
> index 000000000000..ceadb28169b8
> --- /dev/null
> +++ b/drivers/gpu/drm/mediatek/mtk_ethdr.c
> @@ -0,0 +1,537 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +/*
> + * Copyright (c) 2021 MediaTek Inc.
> + */
> +
> +#include <drm/drm_fourcc.h>
> +#include <linux/clk.h>
> +#include <linux/component.h>
> +#include <linux/of_device.h>
> +#include <linux/of_address.h>
> +#include <linux/platform_device.h>
> +#include <linux/soc/mediatek/mtk-cmdq.h>
> +
> +#include "mtk_drm_crtc.h"
> +#include "mtk_drm_ddp_comp.h"
> +#include "mtk_drm_drv.h"
> +#include "mtk_ethdr.h"
> +
> +#define MIX_EN         0xc
> +#define MIX_RST                0x14
> +#define MIX_ROI_SIZE   0x18
> +#define MIX_DATAPATH_CON       0x1c
> +#define MIX_ROI_BGCLR  0x20
> +#define MIX_SRC_CON    0x24
> +#define MIX_L0_CON     0x28
> +#define MIX_L0_SRC_SIZE        0x30
> +#define MIX_L0_OFFSET  0x34
> +#define MIX_L1_CON     0x40
> +#define MIX_L1_SRC_SIZE        0x48
> +#define MIX_L1_OFFSET  0x4c
> +#define MIX_L2_CON     0x58
> +#define MIX_L2_SRC_SIZE        0x60
> +#define MIX_L2_OFFSET  0x64
> +#define MIX_L3_CON     0x70
> +#define MIX_L3_SRC_SIZE        0x78
> +#define MIX_L3_OFFSET  0x7c
> +#define MIX_FUNC_DCM0  0x120
> +#define MIX_FUNC_DCM1  0x124
> +
> +#define HDR_VDO_FE_0804_HDR_DM_FE      0x804
> +#define HDR_VDO_FE_081C_HDR_DM_FE      0x81c
> +#define HDR_VDO_FE_09EC_HDR_DM_FE      0x9ec
> +#define HDR_VDO_FE_0618_HDR_TOP_FE     0x618
> +#define HDR_VDO_FE_061C_HDR_TOP_FE     0x61c
> +#define HDR_VDO_FE_06D0_HDR_TOP_FE     0x6d0
> +#define HDR_VDO_FE_0634_HDR_TOP_FE     0x634
> +
> +#define HDR_GFX_FE_0100_GFX_DV_WP      0x100
> +#define HDR_GFX_FE_012C_GFX_DV_WP      0x12c
> +#define HDR_GFX_FE_0134_GFX_DV_WP      0x134
> +#define HDR_GFX_FE_0138_GFX_DV_WP      0x138
> +#define HDR_GFX_FE_013C_GFX_DV_WP      0x13c
> +#define HDR_GFX_FE_0140_GFX_DV_WP      0x140
> +#define HDR_GFX_FE_0144_GFX_DV_WP      0x144
> +#define HDR_GFX_FE_0148_GFX_DV_WP      0x148
> +#define HDR_GFX_FE_014C_GFX_DV_WP      0x14c
> +#define HDR_GFX_FE_0150_GFX_DV_WP      0x150
> +#define HDR_GFX_FE_0154_GFX_DV_WP      0x154
> +#define HDR_GFX_FE_0204_GFX_HDR_FE     0x204
> +#define HDR_GFX_FE_021C_GFX_HDR_FE     0x21c
> +#define HDR_GFX_FE_03EC_GFX_HDR_FE     0x3ec
> +
> +#define HDR_VDO_BE_0204_VDO_DM_BE      0x204
> +#define HDR_VDO_BE_0320_VDO_DM_BE      0x320
> +#define HDR_VDO_BE_03C8_VDO_DM_BE      0x3c8
> +
> +#define VDO1_CONFIG_SW0_RST_B 0x1d0
> +#define VDO1_CONFIG_SW1_RST_B 0x1d4
> +       #define HDR_ASYNC_RESET_BIT (BIT(19) | BIT(20) | BIT(21) | BIT(22) | BIT(23))
> +#define VDO1_CONFIG_HDR_BE_ASYNC_CFG_WD 0xe70
> +#define VDO1_CONFIG_HDR_TOP_CFG 0xd00
> +       #define HDR_ALPHA_SEL_MIXER_IN1 BIT(20)
> +       #define HDR_ALPHA_SEL_MIXER_IN2 BIT(21)
> +       #define HDR_ALPHA_SEL_MIXER_IN3 BIT(22)
> +       #define HDR_ALPHA_SEL_MIXER_IN4 BIT(23)
> +#define VDO1_CONFIG_MIXER_IN1_ALPHA 0xd30
> +#define VDO1_CONFIG_MIXER_IN2_ALPHA 0xd34
> +#define VDO1_CONFIG_MIXER_IN3_ALPHA 0xd38
> +#define VDO1_CONFIG_MIXER_IN4_ALPHA 0xd3c
> +#define VDO1_CONFIG_MIXER_IN4_PAD   0xd4c
> +
> +#define        MIXER_ALPHA_AEN         BIT(8)
> +#define        MIXER_ALPHA             0xff
> +#define ETHDR_CLK_NUM          13
> +
> +enum mtk_ethdr_comp_id {
> +       ETHDR_MIXER,
> +       ETHDR_VDO_FE0,
> +       ETHDR_VDO_FE1,
> +       ETHDR_GFX_FE0,
> +       ETHDR_GFX_FE1,
> +       ETHDR_VDO_BE,
> +       ETHDR_ADL_DS,
> +       ETHDR_ID_MAX
> +};
> +
> +struct mtk_ethdr_comp {
> +       struct device *dev;
> +       void __iomem *regs;
> +       struct cmdq_client_reg cmdq_base;
> +};
> +
> +struct mtk_ethdr {
> +       void __iomem *top_regs;
> +       struct cmdq_client_reg top_cmdq_base;
> +       struct mtk_ethdr_comp ethdr_comp[ETHDR_ID_MAX];
> +       struct clk_bulk_data ethdr_clk[ETHDR_CLK_NUM];
> +};
> +
> +static const char * const ethdr_comp_str[] = {
> +       "ETHDR_MIXER",
> +       "ETHDR_VDO_FE0",
> +       "ETHDR_VDO_FE1",
> +       "ETHDR_GFX_FE0",
> +       "ETHDR_GFX_FE1",
> +       "ETHDR_VDO_BE",
> +       "ETHDR_ADL_DS",
> +       "ETHDR_ID_MAX"
> +};
> +
> +static const char * const ethdr_clk_str[] = {
> +       "ethdr_top",
> +       "mixer",
> +       "vdo_fe0",
> +       "vdo_fe1",
> +       "gfx_fe0",
> +       "gfx_fe1",
> +       "vdo_be",
> +       "adl_ds",
> +       "vdo_fe0_async",
> +       "vdo_fe1_async",
> +       "gfx_fe0_async",
> +       "gfx_fe1_async",
> +       "vdo_be_async",
> +};
> +
> +static const unsigned int alpha_source_sel[] = {
> +       HDR_ALPHA_SEL_MIXER_IN1,
> +       HDR_ALPHA_SEL_MIXER_IN2,
> +       HDR_ALPHA_SEL_MIXER_IN3,
> +       HDR_ALPHA_SEL_MIXER_IN4,
> +};
> +
> +void mtk_ethdr_layer_on(struct device *dev, unsigned int idx,
> +                       struct cmdq_pkt *cmdq_pkt)
> +{
> +       struct mtk_ethdr *priv = dev_get_drvdata(dev);
> +       struct mtk_ethdr_comp *mixer = &priv->ethdr_comp[ETHDR_MIXER];
> +
> +       dev_dbg(dev, "%s+ idx:%d", __func__, idx);
> +
> +       if (idx < 4)
> +               mtk_ddp_write_mask(cmdq_pkt, BIT(idx), &mixer->cmdq_base,
> +                                  mixer->regs, MIX_SRC_CON, BIT(idx));
> +}
> +
> +void mtk_ethdr_layer_off(struct device *dev, unsigned int idx,
> +                        struct cmdq_pkt *cmdq_pkt)
> +{
> +       struct mtk_ethdr *priv = dev_get_drvdata(dev);
> +       struct mtk_ethdr_comp *mixer = &priv->ethdr_comp[ETHDR_MIXER];
> +
> +       dev_dbg(dev, "%s+ idx:%d", __func__, idx);
> +
> +       switch (idx) {
> +       case 0:
> +               mtk_ddp_write_mask(cmdq_pkt, 0, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L0_SRC_SIZE, ~0);
> +               break;
> +       case 1:
> +               mtk_ddp_write_mask(cmdq_pkt, 0, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L1_SRC_SIZE, ~0);
> +               break;
> +       case 2:
> +               mtk_ddp_write_mask(cmdq_pkt, 0, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L2_SRC_SIZE, ~0);
> +               break;
> +       case 3:
> +               mtk_ddp_write_mask(cmdq_pkt, 0, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L3_SRC_SIZE, ~0);
> +               break;
> +       default:
> +               dev_dbg(dev, "%s Wrong layer ID\n", __func__);
> +               break;
> +       }

Why not just

               mtk_ddp_write_mask(cmdq_pkt, 0, &mixer->cmdq_base,
                                  mixer->regs, MIX_SRC_CON, BIT(idx));

> +}
> +
> +void mtk_ethdr_layer_config(struct device *dev, unsigned int idx,
> +                           struct mtk_plane_state *state,
> +                           struct cmdq_pkt *cmdq_pkt)

Because ethdr has layers and vblank interrupt, I think ethdr should be
part of pseudo ovl, so squash these two into one.

> +{
> +       struct mtk_ethdr *priv = dev_get_drvdata(dev);
> +       struct mtk_ethdr_comp *mixer = &priv->ethdr_comp[ETHDR_MIXER];
> +       struct mtk_plane_pending_state *pending = &state->pending;
> +       unsigned int src_size = (pending->height << 16) | pending->width;
> +       unsigned int offset = (pending->y << 16) | pending->x;
> +       unsigned int alpha_con = 0;
> +       unsigned int fmt = 0;
> +
> +       fmt = state->pending.format;

fmt is useless, so remove.

> +
> +       if (state->base.fb && state->base.fb->format->has_alpha) {
> +               alpha_con = MIXER_ALPHA_AEN | MIXER_ALPHA;
> +               mtk_ddp_write_mask(cmdq_pkt, 0, &priv->top_cmdq_base, priv->top_regs,
> +                                  VDO1_CONFIG_HDR_TOP_CFG, alpha_source_sel[idx]);

Set vdosys1 register in vdosys1 driver, and vdosys1 driver provide
interface for this driver to use.

> +       } else {
> +               mtk_ddp_write_mask(cmdq_pkt, ~0, &priv->top_cmdq_base, priv->top_regs,
> +                                  VDO1_CONFIG_HDR_TOP_CFG, alpha_source_sel[idx]);
> +       }
> +
> +       switch (idx) {
> +       case 0:
> +               mtk_ddp_write_mask(cmdq_pkt, src_size, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L0_SRC_SIZE, ~0);

#define MIX_L_SRC_SIZE(n)  (0x30 + 0x18 * (n))

And you could get rid of this switch case.

> +               mtk_ddp_write_mask(cmdq_pkt, offset, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L0_OFFSET, ~0);
> +               mtk_ddp_write_mask(cmdq_pkt, alpha_con, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L0_CON, 0x1ff);
> +               break;
> +       case 1:
> +               mtk_ddp_write_mask(cmdq_pkt, src_size, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L1_SRC_SIZE, ~0);
> +               mtk_ddp_write_mask(cmdq_pkt, offset, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L1_OFFSET, ~0);
> +               mtk_ddp_write_mask(cmdq_pkt, alpha_con, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L1_CON, 0x1ff);
> +               break;
> +       case 2:
> +               mtk_ddp_write_mask(cmdq_pkt, src_size, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L2_SRC_SIZE, ~0);
> +               mtk_ddp_write_mask(cmdq_pkt, offset, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L2_OFFSET, ~0);
> +               mtk_ddp_write_mask(cmdq_pkt, alpha_con, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L2_CON, 0x1ff);
> +               break;
> +       case 3:
> +               mtk_ddp_write_mask(cmdq_pkt, src_size, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L3_SRC_SIZE, ~0);
> +               mtk_ddp_write_mask(cmdq_pkt, offset, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L3_OFFSET, ~0);
> +               mtk_ddp_write_mask(cmdq_pkt, alpha_con, &mixer->cmdq_base,
> +                                  mixer->regs, MIX_L3_CON, 0x1ff);
> +               break;
> +       default:
> +               dev_dbg(dev, "%s Wrong layer ID\n", __func__);
> +               break;
> +       }
> +}
> +
> +void mtk_ethdr_config(struct device *dev, unsigned int w,
> +                     unsigned int h, unsigned int vrefresh,
> +                     unsigned int bpc, struct cmdq_pkt *cmdq_pkt)
> +{
> +       struct mtk_ethdr *priv = dev_get_drvdata(dev);
> +       struct mtk_ethdr_comp *mixer = &priv->ethdr_comp[ETHDR_MIXER];
> +
> +       dev_dbg(dev, "%s-w:%d, h:%d\n", __func__, w, h);
> +       mtk_ddp_write_mask(cmdq_pkt, (h << 16) | (w / 2), &priv->top_cmdq_base,
> +                          priv->top_regs, VDO1_CONFIG_HDR_BE_ASYNC_CFG_WD, ~0);
> +       mtk_ddp_write_mask(cmdq_pkt, 0, &priv->top_cmdq_base,
> +                          priv->top_regs, VDO1_CONFIG_MIXER_IN4_PAD, ~0);
> +       mtk_ddp_write_mask(cmdq_pkt, (h << 16 | w), &mixer->cmdq_base,
> +                          mixer->regs, MIX_ROI_SIZE, ~0);

      mtk_ddp_write(cmdq_pkt, (h << 16 | w), &mixer->cmdq_base,
                         mixer->regs, MIX_ROI_SIZE);

> +       mtk_ddp_write_mask(cmdq_pkt, 0x01000100, &priv->top_cmdq_base,
> +                          priv->top_regs, VDO1_CONFIG_MIXER_IN1_ALPHA, ~0);
> +       mtk_ddp_write_mask(cmdq_pkt, 0x01000100, &priv->top_cmdq_base,
> +                          priv->top_regs, VDO1_CONFIG_MIXER_IN2_ALPHA, ~0);
> +       mtk_ddp_write_mask(cmdq_pkt, 0x01000100, &priv->top_cmdq_base,
> +                          priv->top_regs, VDO1_CONFIG_MIXER_IN3_ALPHA, ~0);
> +       mtk_ddp_write_mask(cmdq_pkt, 0x01000100, &priv->top_cmdq_base,
> +                          priv->top_regs, VDO1_CONFIG_MIXER_IN4_ALPHA, ~0);
> +}
> +
> +void mtk_ethdr_start(struct device *dev)
> +{
> +       struct mtk_ethdr *priv = dev_get_drvdata(dev);
> +       struct mtk_ethdr_comp *vdo_fe0 = &priv->ethdr_comp[ETHDR_VDO_FE0];
> +       struct mtk_ethdr_comp *vdo_fe1 = &priv->ethdr_comp[ETHDR_VDO_FE1];
> +       struct mtk_ethdr_comp *gfx_fe0 = &priv->ethdr_comp[ETHDR_GFX_FE0];
> +       struct mtk_ethdr_comp *gfx_fe1 = &priv->ethdr_comp[ETHDR_GFX_FE1];
> +       struct mtk_ethdr_comp *vdo_be = &priv->ethdr_comp[ETHDR_VDO_BE];
> +       struct mtk_ethdr_comp *mixer = &priv->ethdr_comp[ETHDR_MIXER];
> +
> +       mtk_ddp_write_mask(NULL, 0xfd, &vdo_fe0->cmdq_base, vdo_fe0->regs,
> +                          HDR_VDO_FE_0804_HDR_DM_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x80, &vdo_fe0->cmdq_base, vdo_fe0->regs,
> +                          HDR_VDO_FE_09EC_HDR_DM_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x12e, &vdo_fe0->cmdq_base, vdo_fe0->regs,
> +                          HDR_VDO_FE_081C_HDR_DM_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x0, &vdo_fe0->cmdq_base, vdo_fe0->regs,
> +                          HDR_VDO_FE_0618_HDR_TOP_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x2, &vdo_fe0->cmdq_base, vdo_fe0->regs,
> +                          HDR_VDO_FE_061C_HDR_TOP_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x8001, &vdo_fe0->cmdq_base, vdo_fe0->regs,
> +                          HDR_VDO_FE_06D0_HDR_TOP_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x8000, &vdo_fe0->cmdq_base, vdo_fe0->regs,
> +                          HDR_VDO_FE_0634_HDR_TOP_FE, ~0);

Explain what these value mean.

> +
> +       mtk_ddp_write_mask(NULL, 0xfd, &vdo_fe1->cmdq_base, vdo_fe1->regs,
> +                          HDR_VDO_FE_0804_HDR_DM_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x80, &vdo_fe1->cmdq_base, vdo_fe1->regs,
> +                          HDR_VDO_FE_09EC_HDR_DM_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x12e, &vdo_fe1->cmdq_base, vdo_fe1->regs,
> +                          HDR_VDO_FE_081C_HDR_DM_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x0, &vdo_fe1->cmdq_base, vdo_fe1->regs,
> +                          HDR_VDO_FE_0618_HDR_TOP_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x2, &vdo_fe1->cmdq_base, vdo_fe1->regs,
> +                          HDR_VDO_FE_061C_HDR_TOP_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x8001, &vdo_fe1->cmdq_base, vdo_fe1->regs,
> +                          HDR_VDO_FE_06D0_HDR_TOP_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x8000, &vdo_fe1->cmdq_base, vdo_fe1->regs,
> +                          HDR_VDO_FE_0634_HDR_TOP_FE, ~0);
> +
> +       mtk_ddp_write_mask(NULL, 0x8001, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_0100_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0xe030, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_012C_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1c0, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_0134_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1e69, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_0138_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1fd7, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_013C_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0xba, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_0140_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x275, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_0144_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x3f, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_0148_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1f99, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_014C_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1ea6, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_0150_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1c2, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_0154_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0xfd, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_0204_GFX_HDR_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x80, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_03EC_GFX_HDR_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x20, &gfx_fe0->cmdq_base, gfx_fe0->regs,
> +                          HDR_GFX_FE_021C_GFX_HDR_FE, ~0);
> +
> +       mtk_ddp_write_mask(NULL, 0x8001, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_0100_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0xe030, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_012C_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1c0, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_0134_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1e69, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_0138_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1fd7, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_013C_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0xba, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_0140_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x275, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_0144_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x3f, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_0148_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1f99, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_014C_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1ea6, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_0150_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0x1c2, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_0154_GFX_DV_WP, ~0);
> +       mtk_ddp_write_mask(NULL, 0xfd, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_0204_GFX_HDR_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x80, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_03EC_GFX_HDR_FE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x20, &gfx_fe1->cmdq_base, gfx_fe1->regs,
> +                          HDR_GFX_FE_021C_GFX_HDR_FE, ~0);
> +
> +       mtk_ddp_write_mask(NULL, 0x7e, &vdo_be->cmdq_base, vdo_be->regs,
> +                          HDR_VDO_BE_0204_VDO_DM_BE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x00, &vdo_be->cmdq_base, vdo_be->regs,
> +                          HDR_VDO_BE_0320_VDO_DM_BE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x01, &vdo_be->cmdq_base, vdo_be->regs,
> +                          HDR_VDO_BE_03C8_VDO_DM_BE, ~0);
> +
> +       mtk_ddp_write_mask(NULL, 0xffffffff, &mixer->cmdq_base, mixer->regs,
> +                          MIX_FUNC_DCM0, ~0);
> +       mtk_ddp_write_mask(NULL, 0xffffffff, &mixer->cmdq_base, mixer->regs,
> +                          MIX_FUNC_DCM1, ~0);
> +       mtk_ddp_write_mask(NULL, 0x00000888, &mixer->cmdq_base, mixer->regs,
> +                          MIX_DATAPATH_CON, ~0);
> +       mtk_ddp_write_mask(NULL, 0x000021ff, &mixer->cmdq_base, mixer->regs,
> +                          MIX_L0_CON, ~0);
> +       mtk_ddp_write_mask(NULL, 0x000021ff, &mixer->cmdq_base, mixer->regs,
> +                          MIX_L1_CON, ~0);
> +       mtk_ddp_write_mask(NULL, 0x000021ff, &mixer->cmdq_base, mixer->regs,
> +                          MIX_L2_CON, ~0);
> +       mtk_ddp_write_mask(NULL, 0x000021ff, &mixer->cmdq_base, mixer->regs,
> +                          MIX_L3_CON, ~0);
> +       mtk_ddp_write_mask(NULL, 0x0, &mixer->cmdq_base, mixer->regs,
> +                          MIX_L0_SRC_SIZE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x0, &mixer->cmdq_base, mixer->regs,
> +                          MIX_L1_SRC_SIZE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x0, &mixer->cmdq_base, mixer->regs,
> +                          MIX_L2_SRC_SIZE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x0, &mixer->cmdq_base, mixer->regs,
> +                          MIX_L3_SRC_SIZE, ~0);
> +       mtk_ddp_write_mask(NULL, 0x0fa50001, &mixer->cmdq_base, mixer->regs,
> +                          MIX_SRC_CON, ~0);
> +       mtk_ddp_write_mask(NULL, 0xFF000000, &mixer->cmdq_base, mixer->regs,
> +                          MIX_ROI_BGCLR, ~0);
> +       mtk_ddp_write_mask(NULL, 0x00000001, &mixer->cmdq_base, mixer->regs,
> +                          MIX_EN, ~0);

I think only the latest setting is about start, move other setting to config.


> +}
> +
> +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];
> +
> +       mtk_ddp_write_mask(NULL, 0, &mixer->cmdq_base, mixer->regs, MIX_EN, ~0);
> +       mtk_ddp_write_mask(NULL, 1, &mixer->cmdq_base, mixer->regs, MIX_RST, ~0);
> +       mtk_ddp_write_mask(NULL, 0, &priv->top_cmdq_base, priv->top_regs,
> +                          VDO1_CONFIG_SW1_RST_B, HDR_ASYNC_RESET_BIT);
> +       mtk_ddp_write_mask(NULL, 0, &priv->top_cmdq_base, priv->top_regs,
> +                          VDO1_CONFIG_SW0_RST_B, BIT(29));
> +       mtk_ddp_write_mask(NULL, 0, &mixer->cmdq_base, mixer->regs, MIX_RST, ~0);
> +       mtk_ddp_write_mask(NULL, HDR_ASYNC_RESET_BIT, &priv->top_cmdq_base,
> +                          priv->top_regs, VDO1_CONFIG_SW1_RST_B,
> +                          HDR_ASYNC_RESET_BIT);
> +       mtk_ddp_write_mask(NULL, BIT(29), &priv->top_cmdq_base, priv->top_regs,
> +                          VDO1_CONFIG_SW0_RST_B, BIT(29));

Make vdosys1 a reset controller like [1].

[1] https://patchwork.kernel.org/project/linux-mediatek/list/?series=515355

Regards,
Chun-Kuang.

> +}
> +
> +int mtk_ethdr_clk_enable(struct device *dev)
> +{
> +       int i, ret;
> +       struct mtk_ethdr *priv = dev_get_drvdata(dev);
> +
> +       ret = clk_bulk_prepare_enable(ETHDR_CLK_NUM, priv->ethdr_clk);
> +       if (ret)
> +               dev_err(dev,
> +                       "ethdr_clk prepare enable failed\n");
> +       return ret;
> +}
> +
> +void mtk_ethdr_clk_disable(struct device *dev)
> +{
> +       struct mtk_ethdr *priv = dev_get_drvdata(dev);
> +       int i;
> +
> +       clk_bulk_disable_unprepare(ETHDR_CLK_NUM, priv->ethdr_clk);
> +}
> +
> +static int mtk_ethdr_bind(struct device *dev, struct device *master,
> +                         void *data)
> +{
> +       struct mtk_ethdr *priv = dev_get_drvdata(dev);
> +       struct drm_device *drm_dev = data;
> +       struct mtk_drm_private *drm_private = drm_dev->dev_private;
> +       struct device *mmsys_dev = drm_private->mmsys_dev;
> +
> +       priv->top_regs = of_iomap(mmsys_dev->of_node, 0);
> +#if IS_REACHABLE(CONFIG_MTK_CMDQ)
> +       if (cmdq_dev_get_client_reg(mmsys_dev, &priv->top_cmdq_base, 0))
> +               dev_dbg(dev, "get mediatek,gce-client-reg fail!\n");
> +#endif
> +       return 0;
> +}
> +
> +static void mtk_ethdr_unbind(struct device *dev, struct device *master, void *data)
> +{
> +}
> +
> +static const struct component_ops mtk_ethdr_component_ops = {
> +       .bind   = mtk_ethdr_bind,
> +       .unbind = mtk_ethdr_unbind,
> +};
> +
> +static int mtk_ethdr_probe(struct platform_device *pdev)
> +{
> +       struct device *dev = &pdev->dev;
> +       struct mtk_ethdr *priv;
> +       int ret;
> +       int i;
> +
> +       dev_info(dev, "%s+\n", __func__);
> +
> +       priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
> +       if (!priv)
> +               return -ENOMEM;
> +
> +       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);
> +#if IS_REACHABLE(CONFIG_MTK_CMDQ)
> +               ret = cmdq_dev_get_client_reg(dev,
> +                                             &priv->ethdr_comp[i].cmdq_base, i);
> +               if (ret)
> +                       dev_dbg(dev, "get mediatek,gce-client-reg fail!\n");
> +#endif
> +               dev_info(dev, "[DRM]regs:0x%x, node:%s\n",
> +                        priv->ethdr_comp[i].regs, ethdr_comp_str[i]);
> +       }
> +
> +       for (i = 0; i < ETHDR_CLK_NUM; i++)
> +               priv->ethdr_clk[i].id = ethdr_clk_str[i];
> +       ret = devm_clk_bulk_get_optional(dev, ETHDR_CLK_NUM, priv->ethdr_clk);
> +       if (ret)
> +               return ret;
> +
> +       platform_set_drvdata(pdev, priv);
> +
> +       ret = component_add(dev, &mtk_ethdr_component_ops);
> +       if (ret)
> +               dev_notice(dev, "Failed to add component: %d\n", ret);
> +
> +       dev_info(dev, "%s-\n", __func__);
> +
> +       return ret;
> +}
> +
> +static int mtk_ethdr_remove(struct platform_device *pdev)
> +{
> +       component_del(&pdev->dev, &mtk_ethdr_component_ops);
> +       return 0;
> +}
> +
> +static const struct of_device_id mtk_ethdr_driver_dt_match[] = {
> +       { .compatible = "mediatek,mt8195-disp-ethdr"},
> +       {},
> +};
> +
> +MODULE_DEVICE_TABLE(of, mtk_ethdr_driver_dt_match);
> +
> +struct platform_driver mtk_ethdr_driver = {
> +       .probe = mtk_ethdr_probe,
> +       .remove = mtk_ethdr_remove,
> +       .driver = {
> +                       .name = "mediatek-disp-ethdr",
> +                       .owner = THIS_MODULE,
> +                       .of_match_table = mtk_ethdr_driver_dt_match,
> +               },
> +};
> diff --git a/drivers/gpu/drm/mediatek/mtk_ethdr.h b/drivers/gpu/drm/mediatek/mtk_ethdr.h
> new file mode 100644
> index 000000000000..c8fc0581a632
> --- /dev/null
> +++ b/drivers/gpu/drm/mediatek/mtk_ethdr.h
> @@ -0,0 +1,20 @@
> +/* SPDX-License-Identifier: GPL-2.0-only */
> +/*
> + * Copyright (c) 2021 MediaTek Inc.
> + */
> +
> +#ifndef __MTK_DISP_ETHDR_H__
> +#define __MTK_DISP_ETHDR_H__
> +
> +#include <linux/uaccess.h>
> +#include <drm/mediatek_drm.h>
> +
> +void mtk_ethdr_layer_config(struct device *dev, unsigned int idx,
> +                           struct mtk_plane_state *state,
> +                           struct cmdq_pkt *cmdq_pkt);
> +void mtk_ethdr_layer_on(struct device *dev, unsigned int idx,
> +                       struct cmdq_pkt *cmdq_pkt);
> +void mtk_ethdr_layer_off(struct device *dev, unsigned int idx,
> +                        struct cmdq_pkt *cmdq_pkt);
> +#endif
> +
> --
> 2.18.0
>




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux