mbox series

[GIT,PULL] mediatek drm next for 5.8

Message ID CAAOTY_8gJNpWXcMnHiZzg_sgyiJ6BDsBYRxaLX0OXWRuLF1NCg@mail.gmail.com (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL] mediatek drm next for 5.8 | expand

Pull-request

ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git tags/mediatek-drm-next-5.8

Message

Chun-Kuang Hu May 13, 2020, 4:16 p.m. UTC
Hi, Dave & Daniel:

This include dpi pin mode swap, config mipi_tx current and impedance,
convert mtk-dpi to drm_bridge API, and some fixup.

The following changes since commit 8f3d9f354286745c751374f5f1fcafee6b3f3136:
  Linux 5.7-rc1 (2020-04-12 12:35:55 -0700)
are available in the Git repository at:
  ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git
tags/mediatek-drm-next-5.8
for you to fetch changes up to 23b29ff9f13a8fb5a12be9fedb8b614b9dede46a:
  drm/mediatek: Eliminate the magic number in array size (2020-05-10
09:08:30 +0800)
----------------------------------------------------------------
Mediatek DRM Next for Linux 5.8
----------------------------------------------------------------
Anand K Mistry (1):
      drm/mediatek: Stop iterating dma addresses when sg_dma_len() == 0
Bernard Zhao (2):
      drm/mediatek: Cleanup coding style in mediatek a bit
      drm/mediatek: Eliminate the magic number in array size
Chun-Kuang Hu (1):
      Merge tag 'v5.7-next-drm-stable' of
ssh://gitolite.kernel.org/.../matthias.bgg/linux into
mediatek-drm-next
Enric Balletbo i Serra (7):
      dt-bindings: mediatek: Update mmsys binding to reflect it is a
system controller
      soc / drm: mediatek: Move routing control to mmsys device
      soc / drm: mediatek: Fix mediatek-drm device probing
      drm/mediatek: Remove debug messages for function calls
      drm/mediatek: mtk_dpi: Rename bridge to next_bridge
      drm/mediatek: mtk_dpi: Convert to bridge driver
      drm/mediatek: mtk_dpi: Use simple encoder
Hsin-Yi Wang (1):
      drm/mediatek: Fix device passed to cmdq
Jitao Shi (6):
      dt-bindings: display: mediatek: control dpi pins mode to avoid leakage
      drm/mediatek: set dpi pin mode to gpio low to avoid leakage current
      dt-bindings: display: mediatek: add property to control mipi tx
drive current
      dt-bindings: display: mediatek: get mipitx calibration data from nvmem
      drm/mediatek: add the mipitx driving control
      drm/mediatek: config mipitx impedance with calibration data
Matthias Brugger (2):
      drm/mediatek: Omit warning on probe defers
      clk / soc: mediatek: Move mt8173 MMSYS to platform driver
YueHaibing (1):
      drm/mediatek: Fix Kconfig warning
 Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt   |   7 +-
 Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.txt |   6 +
 Documentation/devicetree/bindings/display/mediatek/mediatek,dsi.txt |  10 ++
 drivers/clk/mediatek/Kconfig                                        |   7 +
 drivers/clk/mediatek/Makefile                                       |   1 +
 drivers/clk/mediatek/clk-mt8173-mm.c                                |
146 +++++++++++++++++++
 drivers/clk/mediatek/clk-mt8173.c                                   |
104 --------------
 drivers/gpu/drm/mediatek/Kconfig                                    |   1 +
 drivers/gpu/drm/mediatek/mtk_disp_color.c                           |   5 +-
 drivers/gpu/drm/mediatek/mtk_disp_ovl.c                             |   5 +-
 drivers/gpu/drm/mediatek/mtk_disp_rdma.c                            |   5 +-
 drivers/gpu/drm/mediatek/mtk_dpi.c                                  |
127 ++++++++++-------
 drivers/gpu/drm/mediatek/mtk_drm_crtc.c                             |  27 ++--
 drivers/gpu/drm/mediatek/mtk_drm_ddp.c                              |
259 +---------------------------------
 drivers/gpu/drm/mediatek/mtk_drm_ddp.h                              |   7 -
 drivers/gpu/drm/mediatek/mtk_drm_drv.c                              |
 50 +++----
 drivers/gpu/drm/mediatek/mtk_drm_drv.h                              |   2 +-
 drivers/gpu/drm/mediatek/mtk_drm_gem.c                              |   3 +
 drivers/gpu/drm/mediatek/mtk_dsi.c                                  |   8 +-
 drivers/gpu/drm/mediatek/mtk_hdmi.c                                 |  22 ++-
 drivers/gpu/drm/mediatek/mtk_mipi_tx.c                              |
 54 +++++++
 drivers/gpu/drm/mediatek/mtk_mipi_tx.h                              |   4 +
 drivers/gpu/drm/mediatek/mtk_mt8183_mipi_tx.c                       |  28 ++++
 drivers/soc/mediatek/Kconfig                                        |   8 ++
 drivers/soc/mediatek/Makefile                                       |   1 +
 drivers/soc/mediatek/mtk-mmsys.c                                    |
337 ++++++++++++++++++++++++++++++++++++++++++++
 include/linux/soc/mediatek/mtk-mmsys.h                              |  20 +++
 27 files changed, 778 insertions(+), 476 deletions(-)
 create mode 100644 drivers/clk/mediatek/clk-mt8173-mm.c
 create mode 100644 drivers/soc/mediatek/mtk-mmsys.c
 create mode 100644 include/linux/soc/mediatek/mtk-mmsys.h

Comments

Daniel Vetter May 13, 2020, 7:45 p.m. UTC | #1
On Thu, May 14, 2020 at 12:16:59AM +0800, Chun-Kuang Hu wrote:
> Hi, Dave & Daniel:
> 
> This include dpi pin mode swap, config mipi_tx current and impedance,
> convert mtk-dpi to drm_bridge API, and some fixup.
> 
> The following changes since commit 8f3d9f354286745c751374f5f1fcafee6b3f3136:
>   Linux 5.7-rc1 (2020-04-12 12:35:55 -0700)
> are available in the Git repository at:
>   ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git
> tags/mediatek-drm-next-5.8

Your pull request is badly line wrapped, and I don't have an ssh account
on that machine ... Maybe airlied has, not sure. http:// or git:// would
be a bunch easier, and then signed that. Plus if your pull isn't
linewrapped we can stuff it directly into scripts.

All I quickly wanted to quickly check is your new bridge driver.
- was that reviewed/acked by bridge maintainers? There's a lot of rework
  going on right now, need to make sure we're not adding new bridge
  drivers that use old style

- why is this bridge driver not in drm/bridge/ directory?

Cheers, Daniel

> for you to fetch changes up to 23b29ff9f13a8fb5a12be9fedb8b614b9dede46a:
>   drm/mediatek: Eliminate the magic number in array size (2020-05-10
> 09:08:30 +0800)
> ----------------------------------------------------------------
> Mediatek DRM Next for Linux 5.8
> ----------------------------------------------------------------
> Anand K Mistry (1):
>       drm/mediatek: Stop iterating dma addresses when sg_dma_len() == 0
> Bernard Zhao (2):
>       drm/mediatek: Cleanup coding style in mediatek a bit
>       drm/mediatek: Eliminate the magic number in array size
> Chun-Kuang Hu (1):
>       Merge tag 'v5.7-next-drm-stable' of
> ssh://gitolite.kernel.org/.../matthias.bgg/linux into
> mediatek-drm-next
> Enric Balletbo i Serra (7):
>       dt-bindings: mediatek: Update mmsys binding to reflect it is a
> system controller
>       soc / drm: mediatek: Move routing control to mmsys device
>       soc / drm: mediatek: Fix mediatek-drm device probing
>       drm/mediatek: Remove debug messages for function calls
>       drm/mediatek: mtk_dpi: Rename bridge to next_bridge
>       drm/mediatek: mtk_dpi: Convert to bridge driver
>       drm/mediatek: mtk_dpi: Use simple encoder
> Hsin-Yi Wang (1):
>       drm/mediatek: Fix device passed to cmdq
> Jitao Shi (6):
>       dt-bindings: display: mediatek: control dpi pins mode to avoid leakage
>       drm/mediatek: set dpi pin mode to gpio low to avoid leakage current
>       dt-bindings: display: mediatek: add property to control mipi tx
> drive current
>       dt-bindings: display: mediatek: get mipitx calibration data from nvmem
>       drm/mediatek: add the mipitx driving control
>       drm/mediatek: config mipitx impedance with calibration data
> Matthias Brugger (2):
>       drm/mediatek: Omit warning on probe defers
>       clk / soc: mediatek: Move mt8173 MMSYS to platform driver
> YueHaibing (1):
>       drm/mediatek: Fix Kconfig warning
>  Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt   |   7 +-
>  Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.txt |   6 +
>  Documentation/devicetree/bindings/display/mediatek/mediatek,dsi.txt |  10 ++
>  drivers/clk/mediatek/Kconfig                                        |   7 +
>  drivers/clk/mediatek/Makefile                                       |   1 +
>  drivers/clk/mediatek/clk-mt8173-mm.c                                |
> 146 +++++++++++++++++++
>  drivers/clk/mediatek/clk-mt8173.c                                   |
> 104 --------------
>  drivers/gpu/drm/mediatek/Kconfig                                    |   1 +
>  drivers/gpu/drm/mediatek/mtk_disp_color.c                           |   5 +-
>  drivers/gpu/drm/mediatek/mtk_disp_ovl.c                             |   5 +-
>  drivers/gpu/drm/mediatek/mtk_disp_rdma.c                            |   5 +-
>  drivers/gpu/drm/mediatek/mtk_dpi.c                                  |
> 127 ++++++++++-------
>  drivers/gpu/drm/mediatek/mtk_drm_crtc.c                             |  27 ++--
>  drivers/gpu/drm/mediatek/mtk_drm_ddp.c                              |
> 259 +---------------------------------
>  drivers/gpu/drm/mediatek/mtk_drm_ddp.h                              |   7 -
>  drivers/gpu/drm/mediatek/mtk_drm_drv.c                              |
>  50 +++----
>  drivers/gpu/drm/mediatek/mtk_drm_drv.h                              |   2 +-
>  drivers/gpu/drm/mediatek/mtk_drm_gem.c                              |   3 +
>  drivers/gpu/drm/mediatek/mtk_dsi.c                                  |   8 +-
>  drivers/gpu/drm/mediatek/mtk_hdmi.c                                 |  22 ++-
>  drivers/gpu/drm/mediatek/mtk_mipi_tx.c                              |
>  54 +++++++
>  drivers/gpu/drm/mediatek/mtk_mipi_tx.h                              |   4 +
>  drivers/gpu/drm/mediatek/mtk_mt8183_mipi_tx.c                       |  28 ++++
>  drivers/soc/mediatek/Kconfig                                        |   8 ++
>  drivers/soc/mediatek/Makefile                                       |   1 +
>  drivers/soc/mediatek/mtk-mmsys.c                                    |
> 337 ++++++++++++++++++++++++++++++++++++++++++++
>  include/linux/soc/mediatek/mtk-mmsys.h                              |  20 +++
>  27 files changed, 778 insertions(+), 476 deletions(-)
>  create mode 100644 drivers/clk/mediatek/clk-mt8173-mm.c
>  create mode 100644 drivers/soc/mediatek/mtk-mmsys.c
>  create mode 100644 include/linux/soc/mediatek/mtk-mmsys.h
Chun-Kuang Hu May 13, 2020, 11:33 p.m. UTC | #2
Hi, Daniel:

Daniel Vetter <daniel@ffwll.ch> 於 2020年5月14日 週四 上午3:45寫道:
>
> On Thu, May 14, 2020 at 12:16:59AM +0800, Chun-Kuang Hu wrote:
> > Hi, Dave & Daniel:
> >
> > This include dpi pin mode swap, config mipi_tx current and impedance,
> > convert mtk-dpi to drm_bridge API, and some fixup.
> >
> > The following changes since commit 8f3d9f354286745c751374f5f1fcafee6b3f3136:
> >   Linux 5.7-rc1 (2020-04-12 12:35:55 -0700)
> > are available in the Git repository at:
> >   ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git
> > tags/mediatek-drm-next-5.8
>
> Your pull request is badly line wrapped, and I don't have an ssh account
> on that machine ... Maybe airlied has, not sure. http:// or git:// would
> be a bunch easier, and then signed that. Plus if your pull isn't
> linewrapped we can stuff it directly into scripts.

This is my first time to request-pull on that server, I would try to
make things easier and resend request.

>
> All I quickly wanted to quickly check is your new bridge driver.
> - was that reviewed/acked by bridge maintainers? There's a lot of rework
>   going on right now, need to make sure we're not adding new bridge
>   drivers that use old style
>
> - why is this bridge driver not in drm/bridge/ directory?

I does not notice that drm bridge has maintainer, I would drop bridge
series first, and wait for ack of bridge maintainer.
mtk-dpi is also a drm encoder, and I grep 'drm_bridge_add' and find
many out side of bridge folder, so maybe we need to discuss more about
this.

Regards,
Chun-Kuang.

>
> Cheers, Daniel
>
> > for you to fetch changes up to 23b29ff9f13a8fb5a12be9fedb8b614b9dede46a:
> >   drm/mediatek: Eliminate the magic number in array size (2020-05-10
> > 09:08:30 +0800)
> > ----------------------------------------------------------------
> > Mediatek DRM Next for Linux 5.8
> > ----------------------------------------------------------------
> > Anand K Mistry (1):
> >       drm/mediatek: Stop iterating dma addresses when sg_dma_len() == 0
> > Bernard Zhao (2):
> >       drm/mediatek: Cleanup coding style in mediatek a bit
> >       drm/mediatek: Eliminate the magic number in array size
> > Chun-Kuang Hu (1):
> >       Merge tag 'v5.7-next-drm-stable' of
> > ssh://gitolite.kernel.org/.../matthias.bgg/linux into
> > mediatek-drm-next
> > Enric Balletbo i Serra (7):
> >       dt-bindings: mediatek: Update mmsys binding to reflect it is a
> > system controller
> >       soc / drm: mediatek: Move routing control to mmsys device
> >       soc / drm: mediatek: Fix mediatek-drm device probing
> >       drm/mediatek: Remove debug messages for function calls
> >       drm/mediatek: mtk_dpi: Rename bridge to next_bridge
> >       drm/mediatek: mtk_dpi: Convert to bridge driver
> >       drm/mediatek: mtk_dpi: Use simple encoder
> > Hsin-Yi Wang (1):
> >       drm/mediatek: Fix device passed to cmdq
> > Jitao Shi (6):
> >       dt-bindings: display: mediatek: control dpi pins mode to avoid leakage
> >       drm/mediatek: set dpi pin mode to gpio low to avoid leakage current
> >       dt-bindings: display: mediatek: add property to control mipi tx
> > drive current
> >       dt-bindings: display: mediatek: get mipitx calibration data from nvmem
> >       drm/mediatek: add the mipitx driving control
> >       drm/mediatek: config mipitx impedance with calibration data
> > Matthias Brugger (2):
> >       drm/mediatek: Omit warning on probe defers
> >       clk / soc: mediatek: Move mt8173 MMSYS to platform driver
> > YueHaibing (1):
> >       drm/mediatek: Fix Kconfig warning
> >  Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt   |   7 +-
> >  Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.txt |   6 +
> >  Documentation/devicetree/bindings/display/mediatek/mediatek,dsi.txt |  10 ++
> >  drivers/clk/mediatek/Kconfig                                        |   7 +
> >  drivers/clk/mediatek/Makefile                                       |   1 +
> >  drivers/clk/mediatek/clk-mt8173-mm.c                                |
> > 146 +++++++++++++++++++
> >  drivers/clk/mediatek/clk-mt8173.c                                   |
> > 104 --------------
> >  drivers/gpu/drm/mediatek/Kconfig                                    |   1 +
> >  drivers/gpu/drm/mediatek/mtk_disp_color.c                           |   5 +-
> >  drivers/gpu/drm/mediatek/mtk_disp_ovl.c                             |   5 +-
> >  drivers/gpu/drm/mediatek/mtk_disp_rdma.c                            |   5 +-
> >  drivers/gpu/drm/mediatek/mtk_dpi.c                                  |
> > 127 ++++++++++-------
> >  drivers/gpu/drm/mediatek/mtk_drm_crtc.c                             |  27 ++--
> >  drivers/gpu/drm/mediatek/mtk_drm_ddp.c                              |
> > 259 +---------------------------------
> >  drivers/gpu/drm/mediatek/mtk_drm_ddp.h                              |   7 -
> >  drivers/gpu/drm/mediatek/mtk_drm_drv.c                              |
> >  50 +++----
> >  drivers/gpu/drm/mediatek/mtk_drm_drv.h                              |   2 +-
> >  drivers/gpu/drm/mediatek/mtk_drm_gem.c                              |   3 +
> >  drivers/gpu/drm/mediatek/mtk_dsi.c                                  |   8 +-
> >  drivers/gpu/drm/mediatek/mtk_hdmi.c                                 |  22 ++-
> >  drivers/gpu/drm/mediatek/mtk_mipi_tx.c                              |
> >  54 +++++++
> >  drivers/gpu/drm/mediatek/mtk_mipi_tx.h                              |   4 +
> >  drivers/gpu/drm/mediatek/mtk_mt8183_mipi_tx.c                       |  28 ++++
> >  drivers/soc/mediatek/Kconfig                                        |   8 ++
> >  drivers/soc/mediatek/Makefile                                       |   1 +
> >  drivers/soc/mediatek/mtk-mmsys.c                                    |
> > 337 ++++++++++++++++++++++++++++++++++++++++++++
> >  include/linux/soc/mediatek/mtk-mmsys.h                              |  20 +++
> >  27 files changed, 778 insertions(+), 476 deletions(-)
> >  create mode 100644 drivers/clk/mediatek/clk-mt8173-mm.c
> >  create mode 100644 drivers/soc/mediatek/mtk-mmsys.c
> >  create mode 100644 include/linux/soc/mediatek/mtk-mmsys.h
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
Daniel Vetter May 14, 2020, 5:25 a.m. UTC | #3
On Thu, May 14, 2020 at 1:33 AM Chun-Kuang Hu <chunkuang.hu@kernel.org> wrote:
>
> Hi, Daniel:
>
> Daniel Vetter <daniel@ffwll.ch> 於 2020年5月14日 週四 上午3:45寫道:
> >
> > On Thu, May 14, 2020 at 12:16:59AM +0800, Chun-Kuang Hu wrote:
> > > Hi, Dave & Daniel:
> > >
> > > This include dpi pin mode swap, config mipi_tx current and impedance,
> > > convert mtk-dpi to drm_bridge API, and some fixup.
> > >
> > > The following changes since commit 8f3d9f354286745c751374f5f1fcafee6b3f3136:
> > >   Linux 5.7-rc1 (2020-04-12 12:35:55 -0700)
> > > are available in the Git repository at:
> > >   ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git
> > > tags/mediatek-drm-next-5.8
> >
> > Your pull request is badly line wrapped, and I don't have an ssh account
> > on that machine ... Maybe airlied has, not sure. http:// or git:// would
> > be a bunch easier, and then signed that. Plus if your pull isn't
> > linewrapped we can stuff it directly into scripts.
>
> This is my first time to request-pull on that server, I would try to
> make things easier and resend request.
>
> >
> > All I quickly wanted to quickly check is your new bridge driver.
> > - was that reviewed/acked by bridge maintainers? There's a lot of rework
> >   going on right now, need to make sure we're not adding new bridge
> >   drivers that use old style
> >
> > - why is this bridge driver not in drm/bridge/ directory?
>
> I does not notice that drm bridge has maintainer, I would drop bridge
> series first, and wait for ack of bridge maintainer.
> mtk-dpi is also a drm encoder, and I grep 'drm_bridge_add' and find
> many out side of bridge folder, so maybe we need to discuss more about
> this.

Hm indeed. Maybe we need to extend the MAINTAINERS entry for bridges
so that it matches bridge drivers outside of drm/bridge. Sounds like
we have tons of those already.

Andrzej, Neil, Laurent, Sam, any thoughts?

I guess meanwhile this should be ok.
-Daniel

>
> Regards,
> Chun-Kuang.
>
> >
> > Cheers, Daniel
> >
> > > for you to fetch changes up to 23b29ff9f13a8fb5a12be9fedb8b614b9dede46a:
> > >   drm/mediatek: Eliminate the magic number in array size (2020-05-10
> > > 09:08:30 +0800)
> > > ----------------------------------------------------------------
> > > Mediatek DRM Next for Linux 5.8
> > > ----------------------------------------------------------------
> > > Anand K Mistry (1):
> > >       drm/mediatek: Stop iterating dma addresses when sg_dma_len() == 0
> > > Bernard Zhao (2):
> > >       drm/mediatek: Cleanup coding style in mediatek a bit
> > >       drm/mediatek: Eliminate the magic number in array size
> > > Chun-Kuang Hu (1):
> > >       Merge tag 'v5.7-next-drm-stable' of
> > > ssh://gitolite.kernel.org/.../matthias.bgg/linux into
> > > mediatek-drm-next
> > > Enric Balletbo i Serra (7):
> > >       dt-bindings: mediatek: Update mmsys binding to reflect it is a
> > > system controller
> > >       soc / drm: mediatek: Move routing control to mmsys device
> > >       soc / drm: mediatek: Fix mediatek-drm device probing
> > >       drm/mediatek: Remove debug messages for function calls
> > >       drm/mediatek: mtk_dpi: Rename bridge to next_bridge
> > >       drm/mediatek: mtk_dpi: Convert to bridge driver
> > >       drm/mediatek: mtk_dpi: Use simple encoder
> > > Hsin-Yi Wang (1):
> > >       drm/mediatek: Fix device passed to cmdq
> > > Jitao Shi (6):
> > >       dt-bindings: display: mediatek: control dpi pins mode to avoid leakage
> > >       drm/mediatek: set dpi pin mode to gpio low to avoid leakage current
> > >       dt-bindings: display: mediatek: add property to control mipi tx
> > > drive current
> > >       dt-bindings: display: mediatek: get mipitx calibration data from nvmem
> > >       drm/mediatek: add the mipitx driving control
> > >       drm/mediatek: config mipitx impedance with calibration data
> > > Matthias Brugger (2):
> > >       drm/mediatek: Omit warning on probe defers
> > >       clk / soc: mediatek: Move mt8173 MMSYS to platform driver
> > > YueHaibing (1):
> > >       drm/mediatek: Fix Kconfig warning
> > >  Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt   |   7 +-
> > >  Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.txt |   6 +
> > >  Documentation/devicetree/bindings/display/mediatek/mediatek,dsi.txt |  10 ++
> > >  drivers/clk/mediatek/Kconfig                                        |   7 +
> > >  drivers/clk/mediatek/Makefile                                       |   1 +
> > >  drivers/clk/mediatek/clk-mt8173-mm.c                                |
> > > 146 +++++++++++++++++++
> > >  drivers/clk/mediatek/clk-mt8173.c                                   |
> > > 104 --------------
> > >  drivers/gpu/drm/mediatek/Kconfig                                    |   1 +
> > >  drivers/gpu/drm/mediatek/mtk_disp_color.c                           |   5 +-
> > >  drivers/gpu/drm/mediatek/mtk_disp_ovl.c                             |   5 +-
> > >  drivers/gpu/drm/mediatek/mtk_disp_rdma.c                            |   5 +-
> > >  drivers/gpu/drm/mediatek/mtk_dpi.c                                  |
> > > 127 ++++++++++-------
> > >  drivers/gpu/drm/mediatek/mtk_drm_crtc.c                             |  27 ++--
> > >  drivers/gpu/drm/mediatek/mtk_drm_ddp.c                              |
> > > 259 +---------------------------------
> > >  drivers/gpu/drm/mediatek/mtk_drm_ddp.h                              |   7 -
> > >  drivers/gpu/drm/mediatek/mtk_drm_drv.c                              |
> > >  50 +++----
> > >  drivers/gpu/drm/mediatek/mtk_drm_drv.h                              |   2 +-
> > >  drivers/gpu/drm/mediatek/mtk_drm_gem.c                              |   3 +
> > >  drivers/gpu/drm/mediatek/mtk_dsi.c                                  |   8 +-
> > >  drivers/gpu/drm/mediatek/mtk_hdmi.c                                 |  22 ++-
> > >  drivers/gpu/drm/mediatek/mtk_mipi_tx.c                              |
> > >  54 +++++++
> > >  drivers/gpu/drm/mediatek/mtk_mipi_tx.h                              |   4 +
> > >  drivers/gpu/drm/mediatek/mtk_mt8183_mipi_tx.c                       |  28 ++++
> > >  drivers/soc/mediatek/Kconfig                                        |   8 ++
> > >  drivers/soc/mediatek/Makefile                                       |   1 +
> > >  drivers/soc/mediatek/mtk-mmsys.c                                    |
> > > 337 ++++++++++++++++++++++++++++++++++++++++++++
> > >  include/linux/soc/mediatek/mtk-mmsys.h                              |  20 +++
> > >  27 files changed, 778 insertions(+), 476 deletions(-)
> > >  create mode 100644 drivers/clk/mediatek/clk-mt8173-mm.c
> > >  create mode 100644 drivers/soc/mediatek/mtk-mmsys.c
> > >  create mode 100644 include/linux/soc/mediatek/mtk-mmsys.h
> >
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > http://blog.ffwll.ch
Andrzej Hajda May 14, 2020, 1:47 p.m. UTC | #4
Hi All,


On 14.05.2020 07:25, Daniel Vetter wrote:
> On Thu, May 14, 2020 at 1:33 AM Chun-Kuang Hu <chunkuang.hu@kernel.org> wrote:
>> Hi, Daniel:
>>
>> Daniel Vetter <daniel@ffwll.ch> 於 2020年5月14日 週四 上午3:45寫道:
>>> On Thu, May 14, 2020 at 12:16:59AM +0800, Chun-Kuang Hu wrote:
>>>> Hi, Dave & Daniel:
>>>>
>>>> This include dpi pin mode swap, config mipi_tx current and impedance,
>>>> convert mtk-dpi to drm_bridge API, and some fixup.
>>>>
>>>> The following changes since commit 8f3d9f354286745c751374f5f1fcafee6b3f3136:
>>>>    Linux 5.7-rc1 (2020-04-12 12:35:55 -0700)
>>>> are available in the Git repository at:
>>>>    ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git
>>>> tags/mediatek-drm-next-5.8
>>> Your pull request is badly line wrapped, and I don't have an ssh account
>>> on that machine ... Maybe airlied has, not sure. http:// or git:// would
>>> be a bunch easier, and then signed that. Plus if your pull isn't
>>> linewrapped we can stuff it directly into scripts.
>> This is my first time to request-pull on that server, I would try to
>> make things easier and resend request.
>>
>>> All I quickly wanted to quickly check is your new bridge driver.
>>> - was that reviewed/acked by bridge maintainers? There's a lot of rework
>>>    going on right now, need to make sure we're not adding new bridge
>>>    drivers that use old style
>>>
>>> - why is this bridge driver not in drm/bridge/ directory?
>> I does not notice that drm bridge has maintainer, I would drop bridge
>> series first, and wait for ack of bridge maintainer.
>> mtk-dpi is also a drm encoder, and I grep 'drm_bridge_add' and find
>> many out side of bridge folder, so maybe we need to discuss more about
>> this.
> Hm indeed. Maybe we need to extend the MAINTAINERS entry for bridges
> so that it matches bridge drivers outside of drm/bridge. Sounds like
> we have tons of those already.
>
> Andrzej, Neil, Laurent, Sam, any thoughts?
>
> I guess meanwhile this should be ok.
> -Daniel


As I looked at  matching rules in MAINTAINERS file there is no ideal one 
to fulfil this task.

Maybe the closest one is: K: drm_bridge.

Would be ideally to extend F rule to match content of file, for example:

F: drivers/gpu/drm/#drm_bridge_add - match all files in drivers/gpu/drm/ 
with content matching regex drm_bridge_add.


Any other ideas?


Regards

Andrzej


>
>> Regards,
>> Chun-Kuang.
>>
>>> Cheers, Daniel
>>>
>>>> for you to fetch changes up to 23b29ff9f13a8fb5a12be9fedb8b614b9dede46a:
>>>>    drm/mediatek: Eliminate the magic number in array size (2020-05-10
>>>> 09:08:30 +0800)
>>>> ----------------------------------------------------------------
>>>> Mediatek DRM Next for Linux 5.8
>>>> ----------------------------------------------------------------
>>>> Anand K Mistry (1):
>>>>        drm/mediatek: Stop iterating dma addresses when sg_dma_len() == 0
>>>> Bernard Zhao (2):
>>>>        drm/mediatek: Cleanup coding style in mediatek a bit
>>>>        drm/mediatek: Eliminate the magic number in array size
>>>> Chun-Kuang Hu (1):
>>>>        Merge tag 'v5.7-next-drm-stable' of
>>>> ssh://gitolite.kernel.org/.../matthias.bgg/linux into
>>>> mediatek-drm-next
>>>> Enric Balletbo i Serra (7):
>>>>        dt-bindings: mediatek: Update mmsys binding to reflect it is a
>>>> system controller
>>>>        soc / drm: mediatek: Move routing control to mmsys device
>>>>        soc / drm: mediatek: Fix mediatek-drm device probing
>>>>        drm/mediatek: Remove debug messages for function calls
>>>>        drm/mediatek: mtk_dpi: Rename bridge to next_bridge
>>>>        drm/mediatek: mtk_dpi: Convert to bridge driver
>>>>        drm/mediatek: mtk_dpi: Use simple encoder
>>>> Hsin-Yi Wang (1):
>>>>        drm/mediatek: Fix device passed to cmdq
>>>> Jitao Shi (6):
>>>>        dt-bindings: display: mediatek: control dpi pins mode to avoid leakage
>>>>        drm/mediatek: set dpi pin mode to gpio low to avoid leakage current
>>>>        dt-bindings: display: mediatek: add property to control mipi tx
>>>> drive current
>>>>        dt-bindings: display: mediatek: get mipitx calibration data from nvmem
>>>>        drm/mediatek: add the mipitx driving control
>>>>        drm/mediatek: config mipitx impedance with calibration data
>>>> Matthias Brugger (2):
>>>>        drm/mediatek: Omit warning on probe defers
>>>>        clk / soc: mediatek: Move mt8173 MMSYS to platform driver
>>>> YueHaibing (1):
>>>>        drm/mediatek: Fix Kconfig warning
>>>>   Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt   |   7 +-
>>>>   Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.txt |   6 +
>>>>   Documentation/devicetree/bindings/display/mediatek/mediatek,dsi.txt |  10 ++
>>>>   drivers/clk/mediatek/Kconfig                                        |   7 +
>>>>   drivers/clk/mediatek/Makefile                                       |   1 +
>>>>   drivers/clk/mediatek/clk-mt8173-mm.c                                |
>>>> 146 +++++++++++++++++++
>>>>   drivers/clk/mediatek/clk-mt8173.c                                   |
>>>> 104 --------------
>>>>   drivers/gpu/drm/mediatek/Kconfig                                    |   1 +
>>>>   drivers/gpu/drm/mediatek/mtk_disp_color.c                           |   5 +-
>>>>   drivers/gpu/drm/mediatek/mtk_disp_ovl.c                             |   5 +-
>>>>   drivers/gpu/drm/mediatek/mtk_disp_rdma.c                            |   5 +-
>>>>   drivers/gpu/drm/mediatek/mtk_dpi.c                                  |
>>>> 127 ++++++++++-------
>>>>   drivers/gpu/drm/mediatek/mtk_drm_crtc.c                             |  27 ++--
>>>>   drivers/gpu/drm/mediatek/mtk_drm_ddp.c                              |
>>>> 259 +---------------------------------
>>>>   drivers/gpu/drm/mediatek/mtk_drm_ddp.h                              |   7 -
>>>>   drivers/gpu/drm/mediatek/mtk_drm_drv.c                              |
>>>>   50 +++----
>>>>   drivers/gpu/drm/mediatek/mtk_drm_drv.h                              |   2 +-
>>>>   drivers/gpu/drm/mediatek/mtk_drm_gem.c                              |   3 +
>>>>   drivers/gpu/drm/mediatek/mtk_dsi.c                                  |   8 +-
>>>>   drivers/gpu/drm/mediatek/mtk_hdmi.c                                 |  22 ++-
>>>>   drivers/gpu/drm/mediatek/mtk_mipi_tx.c                              |
>>>>   54 +++++++
>>>>   drivers/gpu/drm/mediatek/mtk_mipi_tx.h                              |   4 +
>>>>   drivers/gpu/drm/mediatek/mtk_mt8183_mipi_tx.c                       |  28 ++++
>>>>   drivers/soc/mediatek/Kconfig                                        |   8 ++
>>>>   drivers/soc/mediatek/Makefile                                       |   1 +
>>>>   drivers/soc/mediatek/mtk-mmsys.c                                    |
>>>> 337 ++++++++++++++++++++++++++++++++++++++++++++
>>>>   include/linux/soc/mediatek/mtk-mmsys.h                              |  20 +++
>>>>   27 files changed, 778 insertions(+), 476 deletions(-)
>>>>   create mode 100644 drivers/clk/mediatek/clk-mt8173-mm.c
>>>>   create mode 100644 drivers/soc/mediatek/mtk-mmsys.c
>>>>   create mode 100644 include/linux/soc/mediatek/mtk-mmsys.h
>>> --
>>> Daniel Vetter
>>> Software Engineer, Intel Corporation
>>> https://protect2.fireeye.com/url?k=6b3cd950-36f75e6e-6b3d521f-0cc47a31309a-78c27b43412cd927&q=1&u=http%3A%2F%2Fblog.ffwll.ch%2F
>
>
Laurent Pinchart May 14, 2020, 2:19 p.m. UTC | #5
Hi Daniel,

On Thu, May 14, 2020 at 07:25:10AM +0200, Daniel Vetter wrote:
> On Thu, May 14, 2020 at 1:33 AM Chun-Kuang Hu <chunkuang.hu@kernel.org> wrote:
> > Daniel Vetter <daniel@ffwll.ch> 於 2020年5月14日 週四 上午3:45寫道:
> > > On Thu, May 14, 2020 at 12:16:59AM +0800, Chun-Kuang Hu wrote:
> > > > Hi, Dave & Daniel:
> > > >
> > > > This include dpi pin mode swap, config mipi_tx current and impedance,
> > > > convert mtk-dpi to drm_bridge API, and some fixup.
> > > >
> > > > The following changes since commit 8f3d9f354286745c751374f5f1fcafee6b3f3136:
> > > >   Linux 5.7-rc1 (2020-04-12 12:35:55 -0700)
> > > > are available in the Git repository at:
> > > >   ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git
> > > > tags/mediatek-drm-next-5.8
> > >
> > > Your pull request is badly line wrapped, and I don't have an ssh account
> > > on that machine ... Maybe airlied has, not sure. http:// or git:// would
> > > be a bunch easier, and then signed that. Plus if your pull isn't
> > > linewrapped we can stuff it directly into scripts.
> >
> > This is my first time to request-pull on that server, I would try to
> > make things easier and resend request.
> >
> > > All I quickly wanted to quickly check is your new bridge driver.
> > > - was that reviewed/acked by bridge maintainers? There's a lot of rework
> > >   going on right now, need to make sure we're not adding new bridge
> > >   drivers that use old style
> > >
> > > - why is this bridge driver not in drm/bridge/ directory?
> >
> > I does not notice that drm bridge has maintainer, I would drop bridge
> > series first, and wait for ack of bridge maintainer.
> > mtk-dpi is also a drm encoder, and I grep 'drm_bridge_add' and find
> > many out side of bridge folder, so maybe we need to discuss more about
> > this.
> 
> Hm indeed. Maybe we need to extend the MAINTAINERS entry for bridges
> so that it matches bridge drivers outside of drm/bridge. Sounds like
> we have tons of those already.

There are four categories of bridges:

- Standalone chips. Those are in drivers/gpu/drm/bridge/ and don't cause
  any issue. An example is the adv7511 driver.

- Third-party IP cores (Synopsys, Cadence, ...) shared by multiple SoCs,
  with SoC-specific glue layers. The main driver is in
  drivers/gpu/drm/bridge/, with glue layers typically stored in
  SoC-specific directories in drivers/gpu/drm/. An example is dw-hdmi.

  I'm toying with the idea of moving those glue layers to
  drivers/gpu/drm/bridge/, as many of them are entangled with the rest
  of the corresponding display driver (I've just had a look at the meson
  dw-hdmi glue recently to try and rework the dw-hdmi core, and the
  .mode_valid() check there calls in the main meson display driver,
  making things quite difficult to handle). I'm not sure if full
  isolation would always be possible though, I may not have enough
  visibility on corner cases.

- SoC-specific IP cores. Those bridge drivers are stored in SoC-specific
  directories. Examples are rcar-lvds or omapdrm hdmi.

  There's a wide variety of cases here actually, with some drivers
  (mostly) standalone, such as the rcar-lvds. There are corner cases
  though, as those IP cores are designed for a particular display
  controller, and can thus be tightly integrated with it. In the
  rcar-lvds case for instance, on some of the R-Car Gen3 SoCs, the LVDS
  bridge provides the main clock to the display controller (I kid you
  not). The rcar-lvds driver exports a few functions to support that.

  Other bridges in this category are not even compiled as separate
  modules but are tightly integrated with the display controller driver.
  That's the case of omapdrm hdmi. It may be possible to rework that,
  but that wouldn't be easy, and certainly not be done overnight.

  It would thus be more difficult to move these drivers to
  drivers/gpu/drm/bridge/, and I don't believe that would be desirable.
  Usage of drm_bridge here isn't about reusability of the components,
  but about benefitting from all the bridge-related helpers instead of
  reinventing the wheel.

> Andrzej, Neil, Laurent, Sam, any thoughts?
> 
> I guess meanwhile this should be ok.
>
> > > > for you to fetch changes up to 23b29ff9f13a8fb5a12be9fedb8b614b9dede46a:
> > > >   drm/mediatek: Eliminate the magic number in array size (2020-05-10
> > > > 09:08:30 +0800)
> > > > ----------------------------------------------------------------
> > > > Mediatek DRM Next for Linux 5.8
> > > > ----------------------------------------------------------------
> > > > Anand K Mistry (1):
> > > >       drm/mediatek: Stop iterating dma addresses when sg_dma_len() == 0
> > > > Bernard Zhao (2):
> > > >       drm/mediatek: Cleanup coding style in mediatek a bit
> > > >       drm/mediatek: Eliminate the magic number in array size
> > > > Chun-Kuang Hu (1):
> > > >       Merge tag 'v5.7-next-drm-stable' of ssh://gitolite.kernel.org/.../matthias.bgg/linux into mediatek-drm-next
> > > > Enric Balletbo i Serra (7):
> > > >       dt-bindings: mediatek: Update mmsys binding to reflect it is a system controller
> > > >       soc / drm: mediatek: Move routing control to mmsys device
> > > >       soc / drm: mediatek: Fix mediatek-drm device probing
> > > >       drm/mediatek: Remove debug messages for function calls
> > > >       drm/mediatek: mtk_dpi: Rename bridge to next_bridge
> > > >       drm/mediatek: mtk_dpi: Convert to bridge driver
> > > >       drm/mediatek: mtk_dpi: Use simple encoder
> > > > Hsin-Yi Wang (1):
> > > >       drm/mediatek: Fix device passed to cmdq
> > > > Jitao Shi (6):
> > > >       dt-bindings: display: mediatek: control dpi pins mode to avoid leakage
> > > >       drm/mediatek: set dpi pin mode to gpio low to avoid leakage current
> > > >       dt-bindings: display: mediatek: add property to control mipi tx drive current
> > > >       dt-bindings: display: mediatek: get mipitx calibration data from nvmem
> > > >       drm/mediatek: add the mipitx driving control
> > > >       drm/mediatek: config mipitx impedance with calibration data
> > > > Matthias Brugger (2):
> > > >       drm/mediatek: Omit warning on probe defers
> > > >       clk / soc: mediatek: Move mt8173 MMSYS to platform driver
> > > > YueHaibing (1):
> > > >       drm/mediatek: Fix Kconfig warning
> > > >  Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt   |   7 +-
> > > >  Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.txt |   6 +
> > > >  Documentation/devicetree/bindings/display/mediatek/mediatek,dsi.txt |  10 ++
> > > >  drivers/clk/mediatek/Kconfig                                        |   7 +
> > > >  drivers/clk/mediatek/Makefile                                       |   1 +
> > > >  drivers/clk/mediatek/clk-mt8173-mm.c                                | 146 +++++++++++++++++++
> > > >  drivers/clk/mediatek/clk-mt8173.c                                   | 104 --------------
> > > >  drivers/gpu/drm/mediatek/Kconfig                                    |   1 +
> > > >  drivers/gpu/drm/mediatek/mtk_disp_color.c                           |   5 +-
> > > >  drivers/gpu/drm/mediatek/mtk_disp_ovl.c                             |   5 +-
> > > >  drivers/gpu/drm/mediatek/mtk_disp_rdma.c                            |   5 +-
> > > >  drivers/gpu/drm/mediatek/mtk_dpi.c                                  | 127 ++++++++++-------
> > > >  drivers/gpu/drm/mediatek/mtk_drm_crtc.c                             |  27 ++--
> > > >  drivers/gpu/drm/mediatek/mtk_drm_ddp.c                              | 259 +---------------------------------
> > > >  drivers/gpu/drm/mediatek/mtk_drm_ddp.h                              |   7 -
> > > >  drivers/gpu/drm/mediatek/mtk_drm_drv.c                              |  50 +++----
> > > >  drivers/gpu/drm/mediatek/mtk_drm_drv.h                              |   2 +-
> > > >  drivers/gpu/drm/mediatek/mtk_drm_gem.c                              |   3 +
> > > >  drivers/gpu/drm/mediatek/mtk_dsi.c                                  |   8 +-
> > > >  drivers/gpu/drm/mediatek/mtk_hdmi.c                                 |  22 ++-
> > > >  drivers/gpu/drm/mediatek/mtk_mipi_tx.c                              |  54 +++++++
> > > >  drivers/gpu/drm/mediatek/mtk_mipi_tx.h                              |   4 +
> > > >  drivers/gpu/drm/mediatek/mtk_mt8183_mipi_tx.c                       |  28 ++++
> > > >  drivers/soc/mediatek/Kconfig                                        |   8 ++
> > > >  drivers/soc/mediatek/Makefile                                       |   1 +
> > > >  drivers/soc/mediatek/mtk-mmsys.c                                    | 337 ++++++++++++++++++++++++++++++++++++++++++++
> > > >  include/linux/soc/mediatek/mtk-mmsys.h                              |  20 +++
> > > >  27 files changed, 778 insertions(+), 476 deletions(-)
> > > >  create mode 100644 drivers/clk/mediatek/clk-mt8173-mm.c
> > > >  create mode 100644 drivers/soc/mediatek/mtk-mmsys.c
> > > >  create mode 100644 include/linux/soc/mediatek/mtk-mmsys.h
Daniel Vetter May 14, 2020, 8:01 p.m. UTC | #6
On Thu, May 14, 2020 at 05:19:08PM +0300, Laurent Pinchart wrote:
> Hi Daniel,
> 
> On Thu, May 14, 2020 at 07:25:10AM +0200, Daniel Vetter wrote:
> > On Thu, May 14, 2020 at 1:33 AM Chun-Kuang Hu <chunkuang.hu@kernel.org> wrote:
> > > Daniel Vetter <daniel@ffwll.ch> 於 2020年5月14日 週四 上午3:45寫道:
> > > > On Thu, May 14, 2020 at 12:16:59AM +0800, Chun-Kuang Hu wrote:
> > > > > Hi, Dave & Daniel:
> > > > >
> > > > > This include dpi pin mode swap, config mipi_tx current and impedance,
> > > > > convert mtk-dpi to drm_bridge API, and some fixup.
> > > > >
> > > > > The following changes since commit 8f3d9f354286745c751374f5f1fcafee6b3f3136:
> > > > >   Linux 5.7-rc1 (2020-04-12 12:35:55 -0700)
> > > > > are available in the Git repository at:
> > > > >   ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git
> > > > > tags/mediatek-drm-next-5.8
> > > >
> > > > Your pull request is badly line wrapped, and I don't have an ssh account
> > > > on that machine ... Maybe airlied has, not sure. http:// or git:// would
> > > > be a bunch easier, and then signed that. Plus if your pull isn't
> > > > linewrapped we can stuff it directly into scripts.
> > >
> > > This is my first time to request-pull on that server, I would try to
> > > make things easier and resend request.
> > >
> > > > All I quickly wanted to quickly check is your new bridge driver.
> > > > - was that reviewed/acked by bridge maintainers? There's a lot of rework
> > > >   going on right now, need to make sure we're not adding new bridge
> > > >   drivers that use old style
> > > >
> > > > - why is this bridge driver not in drm/bridge/ directory?
> > >
> > > I does not notice that drm bridge has maintainer, I would drop bridge
> > > series first, and wait for ack of bridge maintainer.
> > > mtk-dpi is also a drm encoder, and I grep 'drm_bridge_add' and find
> > > many out side of bridge folder, so maybe we need to discuss more about
> > > this.
> > 
> > Hm indeed. Maybe we need to extend the MAINTAINERS entry for bridges
> > so that it matches bridge drivers outside of drm/bridge. Sounds like
> > we have tons of those already.
> 
> There are four categories of bridges:
> 
> - Standalone chips. Those are in drivers/gpu/drm/bridge/ and don't cause
>   any issue. An example is the adv7511 driver.
> 
> - Third-party IP cores (Synopsys, Cadence, ...) shared by multiple SoCs,
>   with SoC-specific glue layers. The main driver is in
>   drivers/gpu/drm/bridge/, with glue layers typically stored in
>   SoC-specific directories in drivers/gpu/drm/. An example is dw-hdmi.
> 
>   I'm toying with the idea of moving those glue layers to
>   drivers/gpu/drm/bridge/, as many of them are entangled with the rest
>   of the corresponding display driver (I've just had a look at the meson
>   dw-hdmi glue recently to try and rework the dw-hdmi core, and the
>   .mode_valid() check there calls in the main meson display driver,
>   making things quite difficult to handle). I'm not sure if full
>   isolation would always be possible though, I may not have enough
>   visibility on corner cases.
> 
> - SoC-specific IP cores. Those bridge drivers are stored in SoC-specific
>   directories. Examples are rcar-lvds or omapdrm hdmi.
> 
>   There's a wide variety of cases here actually, with some drivers
>   (mostly) standalone, such as the rcar-lvds. There are corner cases
>   though, as those IP cores are designed for a particular display
>   controller, and can thus be tightly integrated with it. In the
>   rcar-lvds case for instance, on some of the R-Car Gen3 SoCs, the LVDS
>   bridge provides the main clock to the display controller (I kid you
>   not). The rcar-lvds driver exports a few functions to support that.
> 
>   Other bridges in this category are not even compiled as separate
>   modules but are tightly integrated with the display controller driver.
>   That's the case of omapdrm hdmi. It may be possible to rework that,
>   but that wouldn't be easy, and certainly not be done overnight.
> 
>   It would thus be more difficult to move these drivers to
>   drivers/gpu/drm/bridge/, and I don't believe that would be desirable.
>   Usage of drm_bridge here isn't about reusability of the components,
>   but about benefitting from all the bridge-related helpers instead of
>   reinventing the wheel.

Yeah I guess these I'm somewhat worried about, if there's random strange
usage of bridge interfaces there that might get in the way of all the
refactoring that's planned. Hence the idea that maybe we should extend the
MAINTAINERS entry with a regex match. We've done that for
dma_buf/resv/fence recently to make sure people use this correctly.

But if you folks think it's all going smoothly, then totally fine with me.
-Daniel

> 
> > Andrzej, Neil, Laurent, Sam, any thoughts?
> > 
> > I guess meanwhile this should be ok.
> >
> > > > > for you to fetch changes up to 23b29ff9f13a8fb5a12be9fedb8b614b9dede46a:
> > > > >   drm/mediatek: Eliminate the magic number in array size (2020-05-10
> > > > > 09:08:30 +0800)
> > > > > ----------------------------------------------------------------
> > > > > Mediatek DRM Next for Linux 5.8
> > > > > ----------------------------------------------------------------
> > > > > Anand K Mistry (1):
> > > > >       drm/mediatek: Stop iterating dma addresses when sg_dma_len() == 0
> > > > > Bernard Zhao (2):
> > > > >       drm/mediatek: Cleanup coding style in mediatek a bit
> > > > >       drm/mediatek: Eliminate the magic number in array size
> > > > > Chun-Kuang Hu (1):
> > > > >       Merge tag 'v5.7-next-drm-stable' of ssh://gitolite.kernel.org/.../matthias.bgg/linux into mediatek-drm-next
> > > > > Enric Balletbo i Serra (7):
> > > > >       dt-bindings: mediatek: Update mmsys binding to reflect it is a system controller
> > > > >       soc / drm: mediatek: Move routing control to mmsys device
> > > > >       soc / drm: mediatek: Fix mediatek-drm device probing
> > > > >       drm/mediatek: Remove debug messages for function calls
> > > > >       drm/mediatek: mtk_dpi: Rename bridge to next_bridge
> > > > >       drm/mediatek: mtk_dpi: Convert to bridge driver
> > > > >       drm/mediatek: mtk_dpi: Use simple encoder
> > > > > Hsin-Yi Wang (1):
> > > > >       drm/mediatek: Fix device passed to cmdq
> > > > > Jitao Shi (6):
> > > > >       dt-bindings: display: mediatek: control dpi pins mode to avoid leakage
> > > > >       drm/mediatek: set dpi pin mode to gpio low to avoid leakage current
> > > > >       dt-bindings: display: mediatek: add property to control mipi tx drive current
> > > > >       dt-bindings: display: mediatek: get mipitx calibration data from nvmem
> > > > >       drm/mediatek: add the mipitx driving control
> > > > >       drm/mediatek: config mipitx impedance with calibration data
> > > > > Matthias Brugger (2):
> > > > >       drm/mediatek: Omit warning on probe defers
> > > > >       clk / soc: mediatek: Move mt8173 MMSYS to platform driver
> > > > > YueHaibing (1):
> > > > >       drm/mediatek: Fix Kconfig warning
> > > > >  Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt   |   7 +-
> > > > >  Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.txt |   6 +
> > > > >  Documentation/devicetree/bindings/display/mediatek/mediatek,dsi.txt |  10 ++
> > > > >  drivers/clk/mediatek/Kconfig                                        |   7 +
> > > > >  drivers/clk/mediatek/Makefile                                       |   1 +
> > > > >  drivers/clk/mediatek/clk-mt8173-mm.c                                | 146 +++++++++++++++++++
> > > > >  drivers/clk/mediatek/clk-mt8173.c                                   | 104 --------------
> > > > >  drivers/gpu/drm/mediatek/Kconfig                                    |   1 +
> > > > >  drivers/gpu/drm/mediatek/mtk_disp_color.c                           |   5 +-
> > > > >  drivers/gpu/drm/mediatek/mtk_disp_ovl.c                             |   5 +-
> > > > >  drivers/gpu/drm/mediatek/mtk_disp_rdma.c                            |   5 +-
> > > > >  drivers/gpu/drm/mediatek/mtk_dpi.c                                  | 127 ++++++++++-------
> > > > >  drivers/gpu/drm/mediatek/mtk_drm_crtc.c                             |  27 ++--
> > > > >  drivers/gpu/drm/mediatek/mtk_drm_ddp.c                              | 259 +---------------------------------
> > > > >  drivers/gpu/drm/mediatek/mtk_drm_ddp.h                              |   7 -
> > > > >  drivers/gpu/drm/mediatek/mtk_drm_drv.c                              |  50 +++----
> > > > >  drivers/gpu/drm/mediatek/mtk_drm_drv.h                              |   2 +-
> > > > >  drivers/gpu/drm/mediatek/mtk_drm_gem.c                              |   3 +
> > > > >  drivers/gpu/drm/mediatek/mtk_dsi.c                                  |   8 +-
> > > > >  drivers/gpu/drm/mediatek/mtk_hdmi.c                                 |  22 ++-
> > > > >  drivers/gpu/drm/mediatek/mtk_mipi_tx.c                              |  54 +++++++
> > > > >  drivers/gpu/drm/mediatek/mtk_mipi_tx.h                              |   4 +
> > > > >  drivers/gpu/drm/mediatek/mtk_mt8183_mipi_tx.c                       |  28 ++++
> > > > >  drivers/soc/mediatek/Kconfig                                        |   8 ++
> > > > >  drivers/soc/mediatek/Makefile                                       |   1 +
> > > > >  drivers/soc/mediatek/mtk-mmsys.c                                    | 337 ++++++++++++++++++++++++++++++++++++++++++++
> > > > >  include/linux/soc/mediatek/mtk-mmsys.h                              |  20 +++
> > > > >  27 files changed, 778 insertions(+), 476 deletions(-)
> > > > >  create mode 100644 drivers/clk/mediatek/clk-mt8173-mm.c
> > > > >  create mode 100644 drivers/soc/mediatek/mtk-mmsys.c
> > > > >  create mode 100644 include/linux/soc/mediatek/mtk-mmsys.h
> 
> -- 
> Regards,
> 
> Laurent Pinchart