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

Nancy.Lin nancy.lin at mediatek.com
Wed Jul 21 18:32:38 PDT 2021


Hi Chun-Kuang,

On Mon, 2021-07-19 at 07:56 +0800, Chun-Kuang Hu wrote:
> Hi, Nancy:
> 
> Nancy.Lin <nancy.lin at mediatek.com> 於 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 at mediatek.com>
> > ---
> >  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));
> 

There are two modes in Mixer.
1. Background relay mode: all layers off
2. Normal mix mode: at least one layer on
The timing of the two modes is different, so keep using the normal mix
mode.
Just set the layer region to 0 when the layer off.

> > +}
> > +
> > +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.
> 
OK, I will merge the pseudo_ovl and ethdr components 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.
> 
OK, I will remove it.

> > +
> > +       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.
> 
OK, I will add a new mmsys API for the config.

> > +       } 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.

OK, I will modify it.
> 
> > +               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);
> 
OK, I will modify it.

> > +       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.
> 
OK, I will explain these settings.
> > +
> > +       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.
> 
OK, I will move it.
> 
> > +}
> > +
> > +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://urldefense.com/v3/__https://patchwork.kernel.org/project/linux-mediatek/list/?series=515355__;!!CTRNKA9wMg0ARbw!2eXsbwb3k-IvGimKXZxCCXGJiGQowmNyxuwHetN4Jr6jhe8Bgu4aHuLTYcvgVrxy$
> 
OK, I will change to use the reset framework.
>  
> 
> 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
> > 


More information about the Linux-mediatek mailing list