Re: [PATCH v4, 22/33] drm/mediatek: add background color input select function for ovl/ovl_2l

2019-07-16 Thread CK Hu
t; ovl0_2l read layer4~5 > layer5 is at the top ot all these layers. > > the decision of how to setting ovl0/ovl0_2l read these layer data > is controlled in mtk crtc, which will be another patch > Reviewed-by: CK Hu > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/

Re: [PATCH v4, 21/33] drm/mediatek: add function to background color input select for ovl/ovl_2l direct link

2019-07-16 Thread CK Hu
t select for these hardware. > this is preparation patch for ovl/ovl_2l usecase > Reviewed-by: CK Hu > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.h | 14 ++ > 1 file changed, 14 insertions(+) > > diff --git a/drivers/gpu/drm/me

Re: [PATCH v4, 13/33] drm/mediatek: add mmsys private data for ddp path config

2019-07-16 Thread CK Hu
Hi, Yongqiang: On Tue, 2019-07-09 at 06:33 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add mmsys private data for ddp path config > all these register offset and value will be different in future SOC > add these define into mmsys private data > u32 ovl0_mo

Re: [PATCH v4, 12/33] drm/mediatek: split DISP_REG_CONFIG_DSI_SEL setting into another use case

2019-07-16 Thread CK Hu
Hi, Yongqiang: On Tue, 2019-07-09 at 06:33 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > Here is two modifition in this patch: > 1.bls->dpi0 and rdma1->dsi are differen usecase, > Split DISP_REG_CONFIG_DSI_SEL setting into anther usecase > 2.remove DISP_REG_CONFIG_DPI_SEL se

Re: [PATCH v4, 11/33] drm/mediatek: add mutex sof register offset into ddp private data

2019-07-16 Thread CK Hu
Hi, Yongqiang: On Tue, 2019-07-09 at 06:33 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > mutex sof register offset will be private data of ddp > Reviewed-by: CK Hu > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_drm_ddp.c |

Re: [PATCH v4, 10/33] drm/mediatek: add mutex sof into ddp private data

2019-07-16 Thread CK Hu
Hi, Yongqiang: On Tue, 2019-07-09 at 06:33 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > mutex sof will be ddp private data > Reviewed-by: CK Hu > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediat

Re: [PATCH v4, 09/33] drm/mediatek: add mutex mod register offset into ddp private data

2019-07-16 Thread CK Hu
Hi, Yongqiang: On Tue, 2019-07-09 at 06:33 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > mutex mod register offset will be private data of ddp. > Reviewed-by: CK Hu > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediat

Re: [PATCH v4, 08/33] drm/mediatek: add mutex mod into ddp private data

2019-07-16 Thread CK Hu
Hi, Yongqiang: On Tue, 2019-07-09 at 06:33 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > except mutex mod, mutex mod reg,mutex sof reg, > and mutex sof id will be ddp private data Reviewed-by: CK Hu > > Signed-off-by: Yongqiang Niu > --- >

Re: [PATCH v4, 04/33] dt-bindings: mediatek: add dither description for mt8183 display

2019-07-16 Thread CK Hu
Hi, Yongqiang: This version is identical to previous version, and Rob has gave a 'Reviwed-by' tag on previous version, so you should keep that tag on this version, so we don't need to review this patch again. Regards, CK On Tue, 2019-07-09 at 06:33 +0800, yongqiang@mediatek.com wrote: > Fr

Re: [PATCH v4, 03/33] dt-bindings: mediatek: add ccorr description for mt8183 display

2019-07-16 Thread CK Hu
Hi, Yongqiang: This version is identical to previous version, and Rob has gave a 'Reviwed-by' tag on previous version, so you should keep that tag on this version, so we don't need to review this patch again. Regards, CK On Tue, 2019-07-09 at 06:33 +0800, yongqiang@mediatek.com wrote: > Fr

Re: [PATCH v4, 01/33] dt-bindings: mediatek: add binding for mt8183 display

2019-07-16 Thread CK Hu
Hi, Yongqiang: On Tue, 2019-07-09 at 06:33 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > Update device tree binding documention for the display subsystem for > Mediatek MT8183 SOCs > > Signed-off-by: Yongqiang Niu > --- > .../bindings/display/mediatek/mediatek,display.txt

Re: [PATCH v2 19/19] drm/mediatek: drop use of drmP.h

2019-07-16 Thread CK Hu
ude "" > > And sort the includes in the blocks > Add the necessary includes to fix build after removal of drmP.h Reviewed-by: CK Hu Thanks. > > Signed-off-by: Sam Ravnborg > Acked-by: Emil Velikov > Cc: CK Hu > Cc: Philipp Zabel > Cc: David

Re: [PATCH v7 4/4] drm/mtk: add panel orientation property

2019-07-10 Thread CK Hu
Hi, Derek: On Tue, 2019-07-09 at 19:16 -0700, Derek Basehore wrote: > This inits the panel orientation property for the mediatek dsi driver > if the panel orientation (connector.display_info.panel_orientation) is > not DRM_MODE_PANEL_ORIENTATION_UNKNOWN. > Reviewed-by: CK Hu >

Re: [PATCH v5 08/12] dt-bindings: mediatek: Change the binding for mmsys clocks

2019-07-04 Thread CK Hu
Hi, Uli: On Thu, 2019-07-04 at 17:33 +0200, Ulrich Hecht wrote: > > On July 4, 2019 at 11:08 AM Matthias Brugger wrote: > > You are right, it took far too long for me to respond with a new version of > > the > > series. The problem I face is, that I use my mt8173 based chromebook for > > testing

Re: [PATCH v5 08/12] dt-bindings: mediatek: Change the binding for mmsys clocks

2019-06-30 Thread CK Hu
Hi, Matthias: On Fri, 2018-11-30 at 16:59 +0800, Matthias Brugger wrote: > > On 30/11/2018 07:43, Stephen Boyd wrote: > > Quoting Matthias Brugger (2018-11-21 09:09:52) > >> > >> > >> On 21/11/2018 17:46, Stephen Boyd wrote: > >>> Quoting Rob Herring (2018-11-19 11:15:16) > On Sun, Nov 18, 2

Re: [PATCH v4 4/4] drm/mtk: add panel orientation property

2019-06-30 Thread CK Hu
Hi, Derek: On Fri, 2019-06-21 at 20:41 -0700, Derek Basehore wrote: > This inits the panel orientation property for the mediatek dsi driver > if the panel orientation (connector.display_info.panel_orientation) is > not DRM_MODE_PANEL_ORIENTATION_UNKNOWN. > Reviewed-by: CK Hu >

Re: [v5 6/7] drm/mediatek: change the dsi phytiming calculate method

2019-06-30 Thread CK Hu
pixel > > Signed-off-by: Jitao Shi > Tested-by: Ryan Case > Reviewed-by: CK Hu This version is different than previous version, so you should remove Reviewed-by tag. For this version, I still give you a Reviewed-by: CK Hu > --- > drivers/gpu/drm/mediatek/mtk_dsi.c | 117 +++

Re: [v5 4/7] drm/mediatek: add frame size control

2019-06-30 Thread CK Hu
Hi, Jitao: On Thu, 2019-06-27 at 16:01 +0800, Jitao Shi wrote: > Our new DSI chip has frame size control. > So add the driver data to control for different chips. > > Signed-off-by: Jitao Shi > Reviewed-by: CK Hu This version is different than previous version, so you sh

Re: [v5 1/3] dt-bindings: display: mediatek: update dsi supported chips

2019-06-27 Thread CK Hu
Hi, Jitao: On Thu, 2019-06-27 at 10:58 +0800, Jitao Shi wrote: > Update device tree binding documentation for the dsi for > Mediatek MT8183 SoCs. > > Signed-off-by: Jitao Shi > Acked-by: Rob Herring This version is different than previous version, so I think you should remove the Acked-by tag.

Re: [v5 1/7] drm/mediatek: move mipi_dsi_host_register to probe

2019-06-27 Thread CK Hu
er to probe from bind. > > Signed-off-by: Jitao Shi > Reviewed-by: CK Hu This version is different than v4, so please remove reviewed-by tag when this patch change. When I see a reviewed-by tag of mine, I would just skip review it again because I assume this patch is the same as previous version.

Re: [v5 2/7] drm/mediatek: fixes CMDQ reg address of mt8173 is different with mt2701

2019-06-27 Thread CK Hu
Hi, Jitao: On Thu, 2019-06-27 at 16:01 +0800, Jitao Shi wrote: > Config the different CMDQ reg address in driver data. > > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/mtk_dsi.c | 29 - > 1 file changed, 24 insertions(+), 5 deletions(-) > > diff --git a

Re: [v5 2/3] drm/mediatek: separate mipi_tx to different file

2019-06-27 Thread CK Hu
Hi, Jitao: On Thu, 2019-06-27 at 10:59 +0800, Jitao Shi wrote: > Different IC has different mipi_tx setting of dsi. > This patch separates the mipi_tx hardware relate part for mt8173. > > Signed-off-by: Jitao Shi > Reviewed-by: CK Hu > --- > drivers/gpu/drm/mediatek/Makef

Re: [v5 3/3] drm/mediatek: add mipi_tx driver for mt8183

2019-06-27 Thread CK Hu
Hi, Jitao: On Thu, 2019-06-27 at 10:59 +0800, Jitao Shi wrote: > This patch add mt8183 mipi_tx driver. > And also support other chips that use the same binding and driver. > > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/Makefile | 1 + > drivers/gpu/drm/mediatek/mtk_

Re: [PATCH 44/59] drm/mediatek: Use drm_atomic_helper_wait_for_fences

2019-06-16 Thread CK Hu
Note that this relies on mtk setting drm_fb->obj, which is already > done in mtk_drm_framebuffer_init(). Reviewed-by: CK Hu > > Aside: Probably can use the default commit_tail with this again, but I > didn't check for that. > > Signed-off-by: Daniel Vetter > Cc: CK Hu

Re: [PATCH v3, 27/27] drm/mediatek: add support for mediatek SOC MT8183

2019-06-16 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:43 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add support for mediatek SOC MT8183 > 1.ovl_2l share driver with ovl > 2.rdma1 share drive with rdma0, but fifo size is different > 3.add mt8183 mutex private data, and mmsys

Re: [PATCH v3, 26/27] drm/mediatek: add clock property check before get it

2019-06-16 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:43 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add clock property check before get it In the binding document [1], clock is required property. In this patch, you change it to optional property. I think you should change t

Re: [PATCH v3, 19/27] drm/mediatek: add function to background color input select for ovl/ovl_2l direct link

2019-06-16 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:42 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add function to background color input select for ovl/ovl_2l > direct link > for ovl/ovl_2l direct link usecase, we need set background color > input select for these hardwar

Re: [PATCH 19/59] drm/mtk: Drop drm_gem_prime_export/import

2019-06-16 Thread CK Hu
Hi, Daniel: On Fri, 2019-06-14 at 22:35 +0200, Daniel Vetter wrote: > They're the default. > > Aside: Would be really nice to switch the others over to > drm_gem_object_funcs. Reviewed-by: CK Hu > > Signed-off-by: Daniel Vetter > Cc: CK Hu > Cc: Philipp Zabel

Re: [PATCH v3, 20/27] drm/mediatek: add background color input select function for ovl/ovl_2l

2019-06-13 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:42 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add background color input select function for ovl/ovl_2l > > ovl include 4 DRAM layer and 1 background color layer > ovl_2l include 4 DRAM layer and 1 background color layer

Re: [PATCH v3, 18/27] drm/medaitek: add layer_nr for ovl private data

2019-06-13 Thread CK Hu
reparation for ovl-2l and > ovl share the same driver. This patch is identical to v2, and I've give a 'Reviewed-by' for v2, so you should keep this 'Reviewed-by' tag in this patch, so I still give you a Reviewed-by: CK Hu > > Signed-off-by: Yongqiang Niu

Re: [PATCH v3, 17/27] drm/mediatek: add gmc_bits for ovl private data

2019-06-13 Thread CK Hu
hd_l not > used. This patch is identical to v2, and I've give a 'Reviewed-by' for v2, so you should keep this 'Reviewed-by' tag in this patch, so I still give you a Reviewed-by: CK Hu > > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediate

Re: [PATCH v3, 16/27] drm/mediatek: add component DITHER

2019-06-13 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:42 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add component DITHER Reviewed-by: CK Hu > > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mt

Re: [PATCH v3, 15/27] drm/mediatek: add component OVL_2L1

2019-06-13 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:42 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add component OVL_2L1 Reviewed-by: CK Hu > > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 1 + >

Re: [PATCH v3, 14/27] drm/mediatek: add commponent OVL_2L0

2019-06-13 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:42 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add commponent OVL_2L0 Reviewed-by: CK Hu > > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 2 ++ >

Re: [PATCH v3, 13/27] drm/mediatek: add ddp component CCORR

2019-06-13 Thread CK Hu
in this patch, so I still give you a Reviewed-by: CK Hu [1] https://patchwork.kernel.org/patch/10872697/ > > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 32 > + > drivers/gpu/drm/mediatek/mtk_drm_

Re: [PATCH v3, 11/27] drm/mediatek: add mmsys private data for ddp path config

2019-06-13 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:42 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add mmsys private data for ddp path config > all these register offset and value will be different in future SOC > add these define into mmsys private data > u32 ovl0_mo

Re: [PATCH v3, 10/27] drm/mediatek: split DISP_REG_CONFIG_DSI_SEL setting into another use case

2019-06-13 Thread CK Hu
+Bibby: Hi, Yongqiang: On Wed, 2019-06-05 at 19:42 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > Here is two modifition in this patch: > 1.bls->dpi0 and rdma1->dsi are differen usecase, > Split DISP_REG_CONFIG_DSI_SEL setting into anther usecase > 2.remove DISP_REG_CONFIG_D

Re: [GIT,PULL] mediatek drm fixes for 5.2

2019-06-13 Thread CK Hu
On Thu, 2019-06-13 at 10:27 +0200, Daniel Vetter wrote: > On Thu, Jun 13, 2019 at 02:31:18PM +0800, CK Hu wrote: > > Hi, Daniel: > > > > On Wed, 2019-06-12 at 18:25 +0200, Daniel Vetter wrote: > > > On Wed, Jun 12, 2019 at 03:51:08PM +0800, CK Hu

Re: [PATCH v3, 09/27] drm/mediatek: add mutex sof register offset into ddp private data

2019-06-13 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:42 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > mutex sof register offset will be private data of ddp > > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_drm_ddp.c | 13 ++--- > 1 file changed, 10 insertio

Re: [GIT,PULL] mediatek drm fixes for 5.2

2019-06-12 Thread CK Hu
Hi, Daniel: On Wed, 2019-06-12 at 18:25 +0200, Daniel Vetter wrote: > On Wed, Jun 12, 2019 at 03:51:08PM +0800, CK Hu wrote: > > Hi Dave, Daniel: > > > > This include unbind error fix, clock control flow refinement, and PRIME > > mmap with page offset. > >

[GIT,PULL] mediatek drm fixes for 5.2

2019-06-12 Thread CK Hu
Hi Dave, Daniel: This include unbind error fix, clock control flow refinement, and PRIME mmap with page offset. Regards, CK The following changes since commit a188339ca5a396acc588e5851ed7e19f66b0ebd9: Linux 5.2-rc1 (2019-05-19 15:47:09 -0700) are available in the Git repository at: https:

Re: [v4 5/7] drm/mediatek: add mt8183 dsi driver support

2019-06-10 Thread CK Hu
Hi, Jitao: On Sat, 2019-06-01 at 17:26 +0800, Jitao Shi wrote: > Add mt8183 dsi driver data. Enable size control and > reg commit control. > > Signed-off-by: Jitao Shi > Reviewed-by: CK Hu > --- > drivers/gpu/drm/mediatek/mtk_dsi.c | 8 > 1 file changed, 8

Re: [PATCH 5/5] drm/mtk: add panel orientation property

2019-06-10 Thread CK Hu
Hi, Derek: On Mon, 2019-06-10 at 17:22 -0700, Derek Basehore wrote: > This inits the panel orientation property for the mediatek dsi driver > if the panel orientation (connector.display_info.panel_orientation) is > not DRM_MODE_PANEL_ORIENTATION_UNKNOWN. > Looks good to me, Ack

Re: [PATCH v3, 08/27] drm/mediatek: add mutex sof into ddp private data

2019-06-05 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:42 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > mutex sof will be ddp private data > > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_drm_ddp.c | 44 > +++--- > 1 file changed, 36 ins

Re: [PATCH v3, 06/27] drm/mediatek: add mutex mod into ddp private data

2019-06-05 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:42 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > except mutex mod, mutex mod reg,mutex sof reg, > and mutex sof id will be ddp private data > > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_drm_ddp.c | 53 >

Re: [PATCH v3, 01/27] dt-bindings: mediatek: add binding for mt8183 display

2019-06-05 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-06-05 at 19:42 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > Update device tree binding documention for the display subsystem for > Mediatek MT8183 SOCs > > Signed-off-by: Yongqiang Niu > --- > .../bindings/display/mediatek/mediatek,disp.txt

Re: [PATCH v4] gpu/drm: mediatek: call mtk_dsi_stop() after mtk_drm_crtc_atomic_disable()

2019-06-03 Thread CK Hu
Hi, Hsin-Yi: On Thu, 2019-05-30 at 17:18 +0800, Hsin-Yi Wang wrote: > mtk_dsi_stop() should be called after mtk_drm_crtc_atomic_disable(), which > needs > ovl irq for drm_crtc_wait_one_vblank(), since after mtk_dsi_stop() is called, > ovl irq will be disabled. If drm_crtc_wait_one_vblank() is cal

Re: [PATCH v2 0/4] fix mediatek drm, dis, and disp-* unbind/bind

2019-06-03 Thread CK Hu
Hi, Hsin-Yi: On Wed, 2019-05-29 at 18:25 +0800, Hsin-Yi Wang wrote: > There are some errors when unbinding and rebinding mediatek drm, dsi, > and disp-* drivers. This series is to fix those errors and warnings. > > Hsin-Yi Wang (4): > drm: mediatek: fix unbind functions > drm: mediatek: unbin

Re: [v4 3/3] drm/mediatek: add mipi_tx driver for mt8183

2019-06-02 Thread CK Hu
Hi, Jitao: On Sat, 2019-06-01 at 17:52 +0800, Jitao Shi wrote: > This patch add mt8183 mipi_tx driver. > And also support other chips that use the same binding and driver. > > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/Makefile | 1 + > drivers/gpu/drm/mediatek/mtk_

Re: [v4 6/7] drm/mediatek: change the dsi phytiming calculate method

2019-06-02 Thread CK Hu
gt;regs + DSI_PHY_TIMECON1); > @@ -418,7 +451,8 @@ static void mtk_dsi_config_vdo_timing(struct mtk_dsi *dsi) > u32 horizontal_sync_active_byte; > u32 horizontal_backporch_byte; > u32 horizontal_frontporch_byte; > - u32 dsi_tmp_buf_bpp; > + u32 dsi_tmp_

Re: [v4 3/7] drm/mediatek: add dsi reg commit disable control

2019-06-02 Thread CK Hu
lt on. But this driver doesn't use this > function. So add the disable control. Reviewed-by: CK Hu > > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/mtk_dsi.c | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git a/drivers/gpu/drm/medi

Re: [v4 2/7] drm/mediatek: fixes CMDQ reg address of mt8173 is different with mt2701

2019-06-02 Thread CK Hu
Hi, Jitao: On Sat, 2019-06-01 at 17:26 +0800, Jitao Shi wrote: > Config the different CMDQ reg address in driver data. > > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/mtk_dsi.c | 29 - > 1 file changed, 24 insertions(+), 5 deletions(-) > > diff --git a

Re: [v4 1/7] drm/mediatek: move mipi_dsi_host_register to probe

2019-06-02 Thread CK Hu
;host); > -err_ddp_comp_unregister: > mtk_ddp_comp_unregister(drm, &dsi->ddp_comp); > return ret; > } > @@ -1097,31 +1089,37 @@ static int mtk_dsi_probe(struct platform_device *pdev) > > dsi->host.ops = &mtk_dsi_ops; > dsi->h

Re: [v3 7/7] drm: mediatek: adjust dsi and mipi_tx probe sequence

2019-05-30 Thread CK Hu
Hi, Jitao: On Sun, 2019-05-19 at 17:25 +0800, Jitao Shi wrote: > mtk_mipi_tx is the phy of mtk_dsi. > mtk_dsi get the phy(mtk_mipi_tx) in probe(). > > So, mtk_mipi_tx init should be ahead of mtk_dsi. Or mtk_dsi will > defer to wait mtk_mipi_tx probe done. Reviewed-by: CK Hu

Re: [v3 3/7] drm/mediatek: add dsi reg commit disable control

2019-05-30 Thread CK Hu
Hi, Jitao: On Sun, 2019-05-19 at 17:25 +0800, Jitao Shi wrote: > New DSI IP has shadow register and working reg. The register > values are writen to shadow register. And then trigger with > commit reg, the register values will be moved working register. > > This fucntion is defualt on. But this d

Re: [PATCH] drm/mediatek: add dsi module reset driver

2019-05-30 Thread CK Hu
Hi, Jitao: On Sun, 2019-05-19 at 19:15 +0800, Jitao Shi wrote: > Reset dsi HW to default when power on. Prevent the setting differet > between bootloader and kernel. > > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/mtk_dsi.c | 35 ++ > 1 file changed, 35

Re: [PATCH v4] gpu/drm: mediatek: call mtk_dsi_stop() after mtk_drm_crtc_atomic_disable()

2019-05-30 Thread CK Hu
gt; ... >--> mtk_dsi_ddp_stop() > --> mtk_dsi_poweroff(); > > mtk_dsi_poweroff() has reference count design, change to make mtk_dsi_stop() > called in mtk_dsi_poweroff() when refcount is 0. Reviewed-by: CK Hu > > Fixes: 0707632b5bac ("drm/mediatek: update

Re: [PATCH v3] gpu/drm: mediatek: call mtk_dsi_stop() after mtk_drm_crtc_atomic_disable()

2019-05-30 Thread CK Hu
Hi, Hsin-Yi: On Thu, 2019-05-30 at 10:55 +0800, Hsin-Yi Wang wrote: > On Tue, May 28, 2019 at 4:53 PM CK Hu wrote: > > > I think we've already discussed in [1]. I need a reason to understand > > this is hardware behavior or software bug. If this is a software bug, we >

Re: [PATCH v2 4/4] drm: mediatek: clear num_pipes when unbind driver

2019-05-29 Thread CK Hu
> 10 mutex id. Clear this number so it starts from 0 in every rebind. Reviewed-by: CK Hu > > Fixes: 119f5173628a ("drm/mediatek: Add DRM Driver for Mediatek SoC MT8173.") > Signed-off-by: Hsin-Yi Wang > --- > drivers/gpu/drm/mediatek/mtk_drm_drv.c | 1 + > 1 file chan

Re: [PATCH v2 3/4] drm: mediatek: call drm_atomic_helper_shutdown() when unbinding driver

2019-05-29 Thread CK Hu
Hi, Hsin-Yi: On Wed, 2019-05-29 at 18:25 +0800, Hsin-Yi Wang wrote: > shutdown all CRTC when unbinding drm driver. > Reviewed-by: CK Hu > Fixes: 119f5173628a ("drm/mediatek: Add DRM Driver for Mediatek SoC MT8173.") > Signed-off-by: Hsin-Yi Wang > --- &g

Re: [PATCH v2 2/4] drm: mediatek: unbind components in mtk_drm_unbind()

2019-05-29 Thread CK Hu
tk_drm_kms_init() is called, and the components are added back. > > .unbind() should call mtk_drm_kms_deinit() to unbind components. > > And since component_master_del() in .remove() will trigger .unbind(), > which will also unregister device, it's fine to remove origin

Re: [PATCH v2 1/4] drm: mediatek: fix unbind functions

2019-05-29 Thread CK Hu
Hi, Hsin-Yi: On Wed, 2019-05-29 at 18:25 +0800, Hsin-Yi Wang wrote: > detatch panel in mtk_dsi_destroy_conn_enc(), since .bind will try to > attach it again. > Reviewed-by: CK Hu > Fixes: 2e54c14e310f ("drm/mediatek: Add DSI sub driver") > Signed-off-by: Hsin-Yi Wa

Re: [PATCH 3/3] drm: mediatek: unbind components in mtk_drm_unbind()

2019-05-29 Thread CK Hu
Hi, Hsin-Yi: On Mon, 2019-05-27 at 12:50 +0800, Hsin-Yi Wang wrote: > Unbinding components (i.e. mtk_dsi and mtk_disp_ovl/rdma/color) will > trigger master(mtk_drm)'s .unbind(), and currently mtk_drm's unbind > won't actually unbind components. During the next bind, > mtk_drm_kms_init() is called,

Re: [PATCH 1/3] drm: mediatek: fix unbind functions

2019-05-29 Thread CK Hu
Hi, Hsin-Yi: On Wed, 2019-05-29 at 15:06 +0800, Hsin-Yi Wang wrote: > On Wed, May 29, 2019 at 9:35 AM CK Hu wrote: > > > > > I think mtk_dsi_destroy_conn_enc() has much thing to do and I would like > > you to do more. You could refer to [2] for complete implementatio

Re: [PATCH 2/3] drm: mediatek: remove clk_unprepare() in mtk_drm_crtc_destroy()

2019-05-29 Thread CK Hu
Hi, Hsin-Yi: On Wed, 2019-05-29 at 14:08 +0800, Hsin-Yi Wang wrote: > On Wed, May 29, 2019 at 1:58 PM CK Hu wrote: > > > > Hi, Hsin-Yi: > > > > On Mon, 2019-05-27 at 12:50 +0800, Hsin-Yi Wang wrote: > > > There is no clk_prepare() called in mtk_drm_crtc_rese

Re: [PATCH 2/3] drm: mediatek: remove clk_unprepare() in mtk_drm_crtc_destroy()

2019-05-28 Thread CK Hu
Hi, Hsin-Yi: On Mon, 2019-05-27 at 12:50 +0800, Hsin-Yi Wang wrote: > There is no clk_prepare() called in mtk_drm_crtc_reset(), when unbinding > drm device, mtk_drm_crtc_destroy() will be triggered, and the clocks will > be disabled and unprepared in mtk_crtc_ddp_clk_disable. If clk_unprepare() >

Re: [PATCH 1/3] drm: mediatek: fix unbind functions

2019-05-28 Thread CK Hu
Hi, Hsin-yi: On Mon, 2019-05-27 at 12:50 +0800, Hsin-Yi Wang wrote: > move mipi_dsi_host_unregister() to .remove since mipi_dsi_host_register() > is called in .probe. In the latest kernel [1], mipi_dsi_host_register() is called in mtk_dsi_bind(), I think we don't need this part. [1] https://git.

Re: [PATCH v3] gpu/drm: mediatek: call mtk_dsi_stop() after mtk_drm_crtc_atomic_disable()

2019-05-28 Thread CK Hu
Hi, Hsin-Yi: On Tue, 2019-05-28 at 15:39 +0800, Hsin-Yi Wang wrote: > mtk_dsi_stop() should be called after mtk_drm_crtc_atomic_disable(), which > needs > ovl irq for drm_crtc_wait_one_vblank(), since after mtk_dsi_stop() is called, > ovl irq will be disabled. If drm_crtc_wait_one_vblank() is cal

Re: [PATCH v2 24/25] drm/mediatek: respect page offset for PRIME mmap calls

2019-05-27 Thread CK Hu
Hi, Yongqiang: On Tue, 2019-04-16 at 16:33 +0800, CK Hu wrote: > Hi, Yongqiang: > > On Wed, 2019-03-27 at 14:19 +0800, yongqiang@mediatek.com wrote: > > From: Yongqiang Niu > > > > Respect page offset for PRIME mmap calls > > Reviewed-by: CK Hu This

Re: [PATCH v2 22/25] drm/mediatek: adjust ddp clock control flow

2019-05-27 Thread CK Hu
Hi, Yongqiang: On Tue, 2019-04-16 at 16:24 +0800, CK Hu wrote: > Hi, Yongqiang: > > On Wed, 2019-03-27 at 14:19 +0800, yongqiang@mediatek.com wrote: > > From: Yongqiang Niu > > > > display hardware clock will not unprepare when > > crtc is disable, until

Re: [PATCH] drm/mediatek: Fix warning about unhandled enum value

2019-05-22 Thread CK Hu
with the addition of HDMI_INFOFRAME_TYPE_DRM in the commit > below, but the code really should have been future-proofed from the > start. Acked-by: CK Hu > > Fixes: 2cdbfd66a829 ("drm: Enable HDR infoframe support") I think "drm: Enable HDR infoframe support" exist o

Re: [v2 2/5] drm/mediatek: CMDQ reg address of mt8173 is different with mt2701

2019-05-20 Thread CK Hu
Hi, Jitao: On Sun, 2019-05-19 at 17:33 +0800, Jitao Shi wrote: > On Wed, 2019-05-08 at 10:39 +0800, CK Hu wrote: > > On Tue, 2019-04-16 at 14:04 +0800, Jitao Shi wrote: > > > Config the different CMDQ reg address in driver data. > > > > > For MT8173, you chang

Re: [v3 1/7] drm/mediatek: move mipi_dsi_host_register to probe

2019-05-20 Thread CK Hu
Hi, Jitao: On Sun, 2019-05-19 at 17:25 +0800, Jitao Shi wrote: > DSI panel driver need attach function which is inculde in > mipi_dsi_host_ops. > > If mipi_dsi_host_register is not in probe, dsi panel will > probe fail or more delay. In [1], you have agreed this patch just for delay not for prob

Re: [v2 1/5] drm/mediatek: move mipi_dsi_host_register to probe

2019-05-20 Thread CK Hu
On Sun, 2019-05-19 at 17:36 +0800, Jitao Shi wrote: > On Tue, 2019-05-07 at 17:52 +0800, CK Hu wrote: > > Hi, Jitao: > > > > On Tue, 2019-04-16 at 14:04 +0800, Jitao Shi wrote: > > > DSI panel driver need attach function which is inculde in > >

Re: [v4 4/5] drm/mediatek: control dpi pins dpi or gpio mode in on or off

2019-05-19 Thread CK Hu
Hi, Jitao: On Sat, 2019-05-18 at 17:56 +0800, Jitao Shi wrote: > Pull dpi pins low when dpi has nothing to display. Aovid leakage > current from some dpi pins (Hsync Vsync DE ... ). > > Some chips have dpi pins, but there are some chip don't have pins. > So this function is controlled by chips dr

Re: [v4 2/5] drm/mediatek: dpi dual edge support

2019-05-19 Thread CK Hu
Hi, Jitao: On Sat, 2019-05-18 at 17:56 +0800, Jitao Shi wrote: > DPI sample the data both rising and falling edge. > It can reduce half data io pins. All the registers which you control in this patch exist in MT8173. So I think this is not a SoC-level feature. This feature depends on how much io

Re: [v2 3/3] drm/mediatek: add mipi_tx driver for mt8183

2019-05-19 Thread CK Hu
On Sat, 2019-05-18 at 15:51 +0800, Jitao Shi wrote: > On Mon, 2019-05-06 at 17:17 +0800, CK Hu wrote: > > Hi, Jitao: > > > > On Tue, 2019-04-16 at 13:42 +0800, Jitao Shi wrote: > > > This patch add mt8183 mipi_tx driver. > > > And also support other chi

Re: [v3 2/3] drm/mediatek: separate mipi_tx to different file

2019-05-19 Thread CK Hu
Reviewed-by: CK Hu > > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/Makefile | 1 + > drivers/gpu/drm/mediatek/mtk_mipi_tx.c| 342 ++ > drivers/gpu/drm/mediatek/mtk_mipi_tx.h| 49 +++ > drivers/gpu/drm/med

Re: [v2 5/5] drm/mediatek: add mt8183 dsi driver support

2019-05-07 Thread CK Hu
Hi, Jitao: On Tue, 2019-04-16 at 14:05 +0800, Jitao Shi wrote: > Add mt8183 dsi driver data. Enable size control and > reg commit control. > Reviewed-by: CK Hu > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/mtk_dsi.c | 8 > 1 file changed, 8 in

Re: [v2 4/5] drm/mediatek: add frame size control

2019-05-07 Thread CK Hu
Hi, Jitao: On Tue, 2019-04-16 at 14:05 +0800, Jitao Shi wrote: > Our new DSI chip has frame size control. > So add the driver data to control for different chips. > Reviewed-by: CK Hu > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/mtk_dsi.c | 5 + &g

Re: [v2 3/5] drm/mediatek: add dsi reg commit control

2019-05-07 Thread CK Hu
Hi, Jitao: On Tue, 2019-04-16 at 14:04 +0800, Jitao Shi wrote: > New DSI IP has shadow register and working reg. The register > values are writen to shadow register. And then trigger with > commit reg, the register values will be moved working register. This patch looks good, but the message is n

Re: [v2 2/5] drm/mediatek: CMDQ reg address of mt8173 is different with mt2701

2019-05-07 Thread CK Hu
On Tue, 2019-04-16 at 14:04 +0800, Jitao Shi wrote: > Config the different CMDQ reg address in driver data. > For MT8173, you change reg_cmd_off from 0x180 to 0x200, so this patch is a bug fix. You should add a 'Fixes' tag. > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/mtk_dsi.c |

Re: [v2 1/5] drm/mediatek: move mipi_dsi_host_register to probe

2019-05-07 Thread CK Hu
Hi, Jitao: On Tue, 2019-04-16 at 14:04 +0800, Jitao Shi wrote: > DSI panel driver need attach function which is inculde in > mipi_dsi_host_ops. > > If mipi_dsi_host_register is not in probe, dsi panel will > probe fail or more delay. I think this patch just prevent delay, not to prevent dsi pane

Re: [v3 3/3] drm/mediatek: add mt8183 dpi support

2019-05-07 Thread CK Hu
Hi, Jitao: On Tue, 2019-04-16 at 13:52 +0800, Jitao Shi wrote: I need the commit message. Even though the code is easy to understand, words for this patch is still necessary. Regards, CK > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/mtk_dpi.c | 19 +++ > 1 file c

Re: [v3 2/3] drm/mediatek: dpi dual edge support

2019-05-07 Thread CK Hu
Hi, Jitao: On Tue, 2019-04-16 at 13:52 +0800, Jitao Shi wrote: Where is the commit message? I think you could introduce what is dual edge (Maybe it's trivial for you, but not for me) Regards, CK > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/mtk_dpi.c | 10 ++ > 1 file ch

Re: [v2 3/3] drm/mediatek: add mipi_tx driver for mt8183

2019-05-06 Thread CK Hu
Hi, Jitao: On Tue, 2019-04-16 at 13:42 +0800, Jitao Shi wrote: > This patch add mt8183 mipi_tx driver. > And also support other chips that use the same binding and driver. > > Signed-off-by: Jitao Shi > --- > drivers/gpu/drm/mediatek/Makefile | 1 + > drivers/gpu/drm/mediatek/mtk_

Re: [PATCH 1/2] arm64: dts: mt8183: add dsi node

2019-04-19 Thread CK Hu
Hi, Jitao: On Tue, 2019-04-16 at 16:54 +0800, Jitao Shi wrote: > Add dsi and mipitx nodes to the mt8183 > > Signed-off-by: Jitao Shi > --- > arch/arm64/boot/dts/mediatek/mt8183.dtsi | 25 > 1 file changed, 25 insertions(+) > > diff --git a/arch/arm64/boot/dts/mediatek/

Re: [PATCH v1 3/4] drm/mediatek: fix boot up for 720 and 480 but 1080

2019-04-16 Thread CK Hu
Hi, Frank: On Tue, 2019-04-16 at 16:58 +0200, Frank Wunderlich wrote: > From: chunhui dai > > - 1080 plg in/out with ng/ok > - support other resolutions like 1280x1024 The description is so simple and I could not understand why pll_default_off could fix this problem. And why only MT2701 has thi

Re: Aw: Re: [PATCH v1 0/4] make hdmi work on bananapi-r2

2019-04-16 Thread CK Hu
Hi, Frank: On Wed, 2019-04-17 at 07:30 +0200, Frank Wunderlich wrote: > Hi CK Hu, > > you mean the problematic patch is fix possible_crtcs (4/4) and the others are > ok? > > can you push the first 3 while working on the last one? I think 3 patches is related to possible

Re: [PATCH v1 0/4] make hdmi work on bananapi-r2

2019-04-16 Thread CK Hu
Hi, Frank: On Tue, 2019-04-16 at 16:58 +0200, Frank Wunderlich wrote: > This Patch-Series adds missing Patches/Bugfixes to get hdmi working on BPI-R2 > > first 2 Patches were already posted, but not yet merged into mainline > i found no hint why > - config component output by device node port >

Re: [PATCH v2 24/25] drm/mediatek: respect page offset for PRIME mmap calls

2019-04-16 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-03-27 at 14:19 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > Respect page offset for PRIME mmap calls Reviewed-by: CK Hu > > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_drm_gem.c | 7 ++-

Re: [PATCH v2 23/25] drm/mediatek: add vmap support for mediatek drm

2019-04-16 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-03-27 at 14:19 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add add vmap support for mediatek drm I think the upstreamed patch 'drm/mediatek: Implement gem prime vmap/vunmap function' [1] has the same function of this patch. You could

Re: [PATCH v2 22/25] drm/mediatek: adjust ddp clock control flow

2019-04-16 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-03-27 at 14:19 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > display hardware clock will not unprepare when > crtc is disable, until crtc is destroyed. > with this patch, hard clock will disable and unprepare > at the same time

Re: [PATCH v2 20/25] drm/mediatek: add ovl0/ovl0_2l usecase

2019-04-16 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-03-27 at 14:19 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add ovl0/ovl0_2l usecase > in ovl->ovl0_2l direct link usecase: > 1. the crtc support layer number will 4+2 > 2. ovl0_2l background color input select ovl0 when crtc init > an

Re: [PATCH v2 18/25] drm/mediatek: add RDMA fifo size error handle

2019-04-16 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-03-27 at 14:19 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add RDMA fifo size error handle > rdma fifo size will not always bigger than the calculated threshold > if that case happened, we need set fifo size as the threshold > > Sign

Re: [PATCH v2 17/25] drm/mediatek: add background color input select function for ovl/ovl_2l

2019-04-16 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-03-27 at 14:19 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add background color input select function for ovl/ovl_2l > > ovl include 4 DRAM layer and 1 background color layer > ovl_2l include 4 DRAM layer and 1 background color layer

Re: [PATCH v2 16/25] drm/mediatek: add ddp write register common api

2019-04-11 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-03-27 at 14:19 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add ddp write register common api > this is preparation patch for ovl/ovl_2l direct link > usecase. > in that case, we need this funtion to set one bit of ovl_2l > register T

Re: [PATCH v2 15/25] drm/mediatek: add function to background color input select for ovl/ovl_2l direct link

2019-04-11 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-03-27 at 14:19 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add function to background color input select for ovl/ovl_2l > direct link > for ovl/ovl_2l direct link usecase, we need set background color > input select for these hardwar

Re: [PATCH v2 14/25] drm/medaitek: add layer_nr for ovl private data

2019-04-11 Thread CK Hu
reparation for ovl-2l and > ovl share the same driver. Reviewed-by: CK Hu > > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/mediatek/mt

Re: [PATCH v2 09/25] drm/mediatek: add mmsys private data for ddp path config

2019-04-11 Thread CK Hu
Hi, Yongqiang: On Wed, 2019-03-27 at 14:19 +0800, yongqiang@mediatek.com wrote: > From: Yongqiang Niu > > This patch add mmsys private data for ddp path config > all these register offset and value will be different in future SOC > add these define into mmsys private data > u32 ovl0_mo

<    1   2   3   4   5   6   7   8   >