mbox series

[GIT,PULL] drm/imx: fixes and drm managed resources

Message ID c745fc1596898932c9454fd2979297b4242566a2.camel@pengutronix.de (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL] drm/imx: fixes and drm managed resources | expand

Pull-request

git://git.pengutronix.de/git/pza/linux.git tags/imx-drm-next-2021-01-04

Message

Philipp Zabel Jan. 4, 2021, 5:10 p.m. UTC
Hi Dave, Daniel,

this PR includes the drmm encoder/plane/crtc allocation functions and
converts the imx-drm driver to use them.

The following changes since commit 5c8fe583cce542aa0b84adc939ce85293de36e5e:

  Linux 5.11-rc1 (2020-12-27 15:30:22 -0800)

are available in the Git repository at:

  git://git.pengutronix.de/git/pza/linux.git tags/imx-drm-next-2021-01-04

for you to fetch changes up to 16da8e9a7767ac77720f49bfa870def61a250cda:

  drm/imx: ipuv3-crtc: use drm managed resources (2021-01-04 12:59:34 +0100)

----------------------------------------------------------------
drm/imx: fixes and drm managed resources

- Reduce stack usage in ipu-di.
- Fix imx-ldb for compile tests.
- Make drm encoder control functions optional.
- Add drm managed variants drmm_encoder_alloc(),
  drmm_simple_encoder_alloc(), drmm_universal_plane_alloc(), and
  drmm_crtc_alloc_with_planes() for drm_encoder_init(),
  drm_simple_encoder_init(), drm_universal_plane_init(), and
  drm_crtc_init_with_planes(), respectively.
- Update imx-drm to use the new functions for drm managed resource
  allocation, moving initialization from bind to probe where possible.
- Fix imx-tve clock provider leak.

----------------------------------------------------------------
Krzysztof Kozlowski (1):
      drm/imx: depend on COMMON_CLK to fix compile tests

Lee Jones (1):
      gpu/ipu-v3/ipu-di: Strip out 2 unused 'di_sync_config' entries

Philipp Zabel (19):
      drm/encoder: make encoder control functions optional
      drm: add drmm_encoder_alloc()
      drm/simple_kms_helper: add drmm_simple_encoder_alloc()
      drm/plane: add drmm_universal_plane_alloc()
      drm/crtc: add drmm_crtc_alloc_with_planes()
      drm/imx: dw_hdmi-imx: move initialization into probe
      drm/imx: imx-ldb: use local connector variable
      drm/imx: imx-ldb: move initialization into probe
      drm/imx: imx-tve: use local encoder and connector variables
      drm/imx: imx-tve: move initialization into probe
      drm/imx: imx-tve: use devm_clk_register
      drm/imx: parallel-display: use local bridge and connector variables
      drm/imx: parallel-display: move initialization into probe
      drm/imx: dw_hdmi-imx: use drm managed resources
      drm/imx: imx-ldb: use drm managed resources
      drm/imx: imx-tve: use drm managed resources
      drm/imx: parallel-display: use drm managed resources
      drm/imx: ipuv3-plane: use drm managed resources
      drm/imx: ipuv3-crtc: use drm managed resources

Zheng Yongjun (1):
      gpu: drm: imx: convert comma to semicolon

 drivers/gpu/drm/drm_crtc.c              | 125 ++++++++++++++++++++++-------
 drivers/gpu/drm/drm_encoder.c           | 113 +++++++++++++++++++++------
 drivers/gpu/drm/drm_mode_config.c       |   2 +-
 drivers/gpu/drm/drm_plane.c             | 134 +++++++++++++++++++++++++-------
 drivers/gpu/drm/drm_simple_kms_helper.c |  14 +++-
 drivers/gpu/drm/imx/Kconfig             |   1 +
 drivers/gpu/drm/imx/dw_hdmi-imx.c       |  95 ++++++++++------------
 drivers/gpu/drm/imx/imx-ldb.c           | 109 +++++++++++++++-----------
 drivers/gpu/drm/imx/imx-tve.c           | 109 +++++++++++++-------------
 drivers/gpu/drm/imx/ipuv3-crtc.c        | 131 ++++++++++---------------------
 drivers/gpu/drm/imx/ipuv3-plane.c       |  69 ++++++++--------
 drivers/gpu/drm/imx/ipuv3-plane.h       |   3 -
 drivers/gpu/drm/imx/parallel-display.c  |  93 +++++++++++-----------
 drivers/gpu/ipu-v3/ipu-di.c             |   4 -
 include/drm/drm_crtc.h                  |  33 ++++++++
 include/drm/drm_encoder.h               |  32 +++++++-
 include/drm/drm_plane.h                 |  42 ++++++++++
 include/drm/drm_simple_kms_helper.h     |  24 ++++++
 18 files changed, 712 insertions(+), 421 deletions(-)

Comments

Daniel Vetter Jan. 7, 2021, 10:25 a.m. UTC | #1
On Mon, Jan 04, 2021 at 06:10:31PM +0100, Philipp Zabel wrote:
> Hi Dave, Daniel,
> 
> this PR includes the drmm encoder/plane/crtc allocation functions and
> converts the imx-drm driver to use them.
> 
> The following changes since commit 5c8fe583cce542aa0b84adc939ce85293de36e5e:
> 
>   Linux 5.11-rc1 (2020-12-27 15:30:22 -0800)
> 
> are available in the Git repository at:
> 
>   git://git.pengutronix.de/git/pza/linux.git tags/imx-drm-next-2021-01-04

Pulled, thanks. Daniel

> 
> for you to fetch changes up to 16da8e9a7767ac77720f49bfa870def61a250cda:
> 
>   drm/imx: ipuv3-crtc: use drm managed resources (2021-01-04 12:59:34 +0100)
> 
> ----------------------------------------------------------------
> drm/imx: fixes and drm managed resources
> 
> - Reduce stack usage in ipu-di.
> - Fix imx-ldb for compile tests.
> - Make drm encoder control functions optional.
> - Add drm managed variants drmm_encoder_alloc(),
>   drmm_simple_encoder_alloc(), drmm_universal_plane_alloc(), and
>   drmm_crtc_alloc_with_planes() for drm_encoder_init(),
>   drm_simple_encoder_init(), drm_universal_plane_init(), and
>   drm_crtc_init_with_planes(), respectively.
> - Update imx-drm to use the new functions for drm managed resource
>   allocation, moving initialization from bind to probe where possible.
> - Fix imx-tve clock provider leak.
> 
> ----------------------------------------------------------------
> Krzysztof Kozlowski (1):
>       drm/imx: depend on COMMON_CLK to fix compile tests
> 
> Lee Jones (1):
>       gpu/ipu-v3/ipu-di: Strip out 2 unused 'di_sync_config' entries
> 
> Philipp Zabel (19):
>       drm/encoder: make encoder control functions optional
>       drm: add drmm_encoder_alloc()
>       drm/simple_kms_helper: add drmm_simple_encoder_alloc()
>       drm/plane: add drmm_universal_plane_alloc()
>       drm/crtc: add drmm_crtc_alloc_with_planes()
>       drm/imx: dw_hdmi-imx: move initialization into probe
>       drm/imx: imx-ldb: use local connector variable
>       drm/imx: imx-ldb: move initialization into probe
>       drm/imx: imx-tve: use local encoder and connector variables
>       drm/imx: imx-tve: move initialization into probe
>       drm/imx: imx-tve: use devm_clk_register
>       drm/imx: parallel-display: use local bridge and connector variables
>       drm/imx: parallel-display: move initialization into probe
>       drm/imx: dw_hdmi-imx: use drm managed resources
>       drm/imx: imx-ldb: use drm managed resources
>       drm/imx: imx-tve: use drm managed resources
>       drm/imx: parallel-display: use drm managed resources
>       drm/imx: ipuv3-plane: use drm managed resources
>       drm/imx: ipuv3-crtc: use drm managed resources
> 
> Zheng Yongjun (1):
>       gpu: drm: imx: convert comma to semicolon
> 
>  drivers/gpu/drm/drm_crtc.c              | 125 ++++++++++++++++++++++-------
>  drivers/gpu/drm/drm_encoder.c           | 113 +++++++++++++++++++++------
>  drivers/gpu/drm/drm_mode_config.c       |   2 +-
>  drivers/gpu/drm/drm_plane.c             | 134 +++++++++++++++++++++++++-------
>  drivers/gpu/drm/drm_simple_kms_helper.c |  14 +++-
>  drivers/gpu/drm/imx/Kconfig             |   1 +
>  drivers/gpu/drm/imx/dw_hdmi-imx.c       |  95 ++++++++++------------
>  drivers/gpu/drm/imx/imx-ldb.c           | 109 +++++++++++++++-----------
>  drivers/gpu/drm/imx/imx-tve.c           | 109 +++++++++++++-------------
>  drivers/gpu/drm/imx/ipuv3-crtc.c        | 131 ++++++++++---------------------
>  drivers/gpu/drm/imx/ipuv3-plane.c       |  69 ++++++++--------
>  drivers/gpu/drm/imx/ipuv3-plane.h       |   3 -
>  drivers/gpu/drm/imx/parallel-display.c  |  93 +++++++++++-----------
>  drivers/gpu/ipu-v3/ipu-di.c             |   4 -
>  include/drm/drm_crtc.h                  |  33 ++++++++
>  include/drm/drm_encoder.h               |  32 +++++++-
>  include/drm/drm_plane.h                 |  42 ++++++++++
>  include/drm/drm_simple_kms_helper.h     |  24 ++++++
>  18 files changed, 712 insertions(+), 421 deletions(-)