mbox series

[v5,00/11] drm: Add self refresh helpers

Message ID 20190611160844.257498-1-sean@poorly.run (mailing list archive)
Headers show
Series drm: Add self refresh helpers | expand

Message

Sean Paul June 11, 2019, 4:08 p.m. UTC
From: Sean Paul <seanpaul@chromium.org>

Back again with a new version. Very little has changed from v5, in fact
I was just going to resend patch 2, but I ran into some collisions while
rebasing. I figured I'd take the opportunity to collect and apply the
tags I've accumulated so far and send the whole thing again.

I need reviews on patch 2 and patches 6 through 11 (the rockchip bits).

Link to v4 cover: https://patchwork.freedesktop.org/patch/msgid/20190508160920.144739-1-sean@poorly.run

Thanks,

Sean


Laurent Pinchart (1):
  drm: Add drm_atomic_get_(old|new_connector_for_encoder() helpers

Sean Paul (10):
  drm: Add atomic variants of enable/disable to encoder helper funcs
  drm: Add atomic variants for bridge enable/disable
  drm: Convert connector_helper_funcs->atomic_check to accept
    drm_atomic_state
  drm: Add helpers to kick off self refresh mode in drivers
  drm/rockchip: Use dirtyfb helper
  drm/rockchip: Check for fast link training before enabling psr
  drm/rockchip: Use the helpers for PSR
  drm/rockchip: Use vop_win in vop_win_disable instead of vop_win_data
  drm/rockchip: Don't fully disable vop on self refresh
  drm/rockchip: Use drm_atomic_helper_commit_tail_rpm

 Documentation/gpu/drm-kms-helpers.rst         |   9 +
 drivers/gpu/drm/Makefile                      |   2 +-
 .../drm/bridge/analogix/analogix_dp_core.c    | 292 +++++++++++++-----
 .../drm/bridge/analogix/analogix_dp_core.h    |   2 +-
 drivers/gpu/drm/drm_atomic.c                  |  71 +++++
 drivers/gpu/drm/drm_atomic_helper.c           |  55 +++-
 drivers/gpu/drm/drm_atomic_state_helper.c     |   4 +
 drivers/gpu/drm/drm_atomic_uapi.c             |   7 +-
 drivers/gpu/drm/drm_bridge.c                  | 110 +++++++
 drivers/gpu/drm/drm_self_refresh_helper.c     | 213 +++++++++++++
 drivers/gpu/drm/i915/intel_atomic.c           |   8 +-
 drivers/gpu/drm/i915/intel_atomic.h           |   2 +-
 drivers/gpu/drm/i915/intel_dp_mst.c           |   7 +-
 drivers/gpu/drm/i915/intel_sdvo.c             |   9 +-
 drivers/gpu/drm/i915/intel_tv.c               |   8 +-
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |   5 +-
 drivers/gpu/drm/rcar-du/rcar_lvds.c           |  12 +-
 drivers/gpu/drm/rockchip/Makefile             |   3 +-
 .../gpu/drm/rockchip/analogix_dp-rockchip.c   | 106 ++++---
 drivers/gpu/drm/rockchip/rockchip_drm_fb.c    |  39 +--
 drivers/gpu/drm/rockchip/rockchip_drm_psr.c   | 290 -----------------
 drivers/gpu/drm/rockchip/rockchip_drm_psr.h   |  30 --
 drivers/gpu/drm/rockchip/rockchip_drm_vop.c   |  84 ++++-
 drivers/gpu/drm/vc4/vc4_txp.c                 |   7 +-
 include/drm/bridge/analogix_dp.h              |   4 -
 include/drm/drm_atomic.h                      |  22 ++
 include/drm/drm_bridge.h                      | 106 +++++++
 include/drm/drm_connector.h                   |  19 ++
 include/drm/drm_crtc.h                        |  19 ++
 include/drm/drm_modeset_helper_vtables.h      |  50 ++-
 include/drm/drm_self_refresh_helper.h         |  22 ++
 31 files changed, 1066 insertions(+), 551 deletions(-)
 create mode 100644 drivers/gpu/drm/drm_self_refresh_helper.c
 delete mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_psr.c
 delete mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_psr.h
 create mode 100644 include/drm/drm_self_refresh_helper.h

Comments

Sean Paul June 13, 2019, 7:05 p.m. UTC | #1
On Tue, Jun 11, 2019 at 12:08:14PM -0400, Sean Paul wrote:
> From: Sean Paul <seanpaul@chromium.org>

I've pushed the first 6 (all reviewed) patches to drm-misc-next, I'll wait for a
rockchip review on the last 5 and apply them when that comes through.

Thanks all for the comments!
Sean

> 
> Back again with a new version. Very little has changed from v5, in fact
> I was just going to resend patch 2, but I ran into some collisions while
> rebasing. I figured I'd take the opportunity to collect and apply the
> tags I've accumulated so far and send the whole thing again.
> 
> I need reviews on patch 2 and patches 6 through 11 (the rockchip bits).
> 
> Link to v4 cover: https://patchwork.freedesktop.org/patch/msgid/20190508160920.144739-1-sean@poorly.run
> 
> Thanks,
> 
> Sean
> 
> 
> Laurent Pinchart (1):
>   drm: Add drm_atomic_get_(old|new_connector_for_encoder() helpers
> 
> Sean Paul (10):
>   drm: Add atomic variants of enable/disable to encoder helper funcs
>   drm: Add atomic variants for bridge enable/disable
>   drm: Convert connector_helper_funcs->atomic_check to accept
>     drm_atomic_state
>   drm: Add helpers to kick off self refresh mode in drivers
>   drm/rockchip: Use dirtyfb helper
>   drm/rockchip: Check for fast link training before enabling psr
>   drm/rockchip: Use the helpers for PSR
>   drm/rockchip: Use vop_win in vop_win_disable instead of vop_win_data
>   drm/rockchip: Don't fully disable vop on self refresh
>   drm/rockchip: Use drm_atomic_helper_commit_tail_rpm
> 
>  Documentation/gpu/drm-kms-helpers.rst         |   9 +
>  drivers/gpu/drm/Makefile                      |   2 +-
>  .../drm/bridge/analogix/analogix_dp_core.c    | 292 +++++++++++++-----
>  .../drm/bridge/analogix/analogix_dp_core.h    |   2 +-
>  drivers/gpu/drm/drm_atomic.c                  |  71 +++++
>  drivers/gpu/drm/drm_atomic_helper.c           |  55 +++-
>  drivers/gpu/drm/drm_atomic_state_helper.c     |   4 +
>  drivers/gpu/drm/drm_atomic_uapi.c             |   7 +-
>  drivers/gpu/drm/drm_bridge.c                  | 110 +++++++
>  drivers/gpu/drm/drm_self_refresh_helper.c     | 213 +++++++++++++
>  drivers/gpu/drm/i915/intel_atomic.c           |   8 +-
>  drivers/gpu/drm/i915/intel_atomic.h           |   2 +-
>  drivers/gpu/drm/i915/intel_dp_mst.c           |   7 +-
>  drivers/gpu/drm/i915/intel_sdvo.c             |   9 +-
>  drivers/gpu/drm/i915/intel_tv.c               |   8 +-
>  drivers/gpu/drm/nouveau/dispnv50/disp.c       |   5 +-
>  drivers/gpu/drm/rcar-du/rcar_lvds.c           |  12 +-
>  drivers/gpu/drm/rockchip/Makefile             |   3 +-
>  .../gpu/drm/rockchip/analogix_dp-rockchip.c   | 106 ++++---
>  drivers/gpu/drm/rockchip/rockchip_drm_fb.c    |  39 +--
>  drivers/gpu/drm/rockchip/rockchip_drm_psr.c   | 290 -----------------
>  drivers/gpu/drm/rockchip/rockchip_drm_psr.h   |  30 --
>  drivers/gpu/drm/rockchip/rockchip_drm_vop.c   |  84 ++++-
>  drivers/gpu/drm/vc4/vc4_txp.c                 |   7 +-
>  include/drm/bridge/analogix_dp.h              |   4 -
>  include/drm/drm_atomic.h                      |  22 ++
>  include/drm/drm_bridge.h                      | 106 +++++++
>  include/drm/drm_connector.h                   |  19 ++
>  include/drm/drm_crtc.h                        |  19 ++
>  include/drm/drm_modeset_helper_vtables.h      |  50 ++-
>  include/drm/drm_self_refresh_helper.h         |  22 ++
>  31 files changed, 1066 insertions(+), 551 deletions(-)
>  create mode 100644 drivers/gpu/drm/drm_self_refresh_helper.c
>  delete mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_psr.c
>  delete mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_psr.h
>  create mode 100644 include/drm/drm_self_refresh_helper.h
> 
> -- 
> Sean Paul, Software Engineer, Google / Chromium OS
>