clean up DPU for RM refactor

Submitted by Jeykumar Sankaran on Aug. 29, 2018, 12:39 a.m.

Details

Reviewer None
Submitted Aug. 29, 2018, 12:39 a.m.
Last Updated Sept. 8, 2018, 12:26 a.m.
Revision 3

Cover Letter(s)

Revision 1
      Based on the comments received for the patch series[1] and to
make the review process easy a bit more easy, spliting up the 
patches for cleanup and resource manager refactor. This series 
cleans up and prepares the DPU for upcoming RM changes.

[1] https://patchwork.freedesktop.org/series/44669/

Thanks and Regards,
Jeykumar S.

Jeykumar Sankaran (14):
  drm/msm/dpu: remove debugfs support for misr
  drm/msm/dpu: remove scalar config definitions
  drm/msm/dpu: remove resource pool manager
  drm/msm/dpu: remove ping pong split topology variables
  drm/msm/dpu: enable master-slave encoders explicitly
  drm/msm/dpu: use kms stored hw mdp block
  drm/msm/dpu: iterate for assigned hw ctl in virtual encoder
  drm/msm/dpu: avoid querying for hw intf before assignment
  drm/msm/dpu: make crtc get_mixer_width helper static
  drm/msm/dpu: move hw resource tracking to crtc state
  drm/msm/dpu: rename hw_ctl to lm_ctl
  drm/msm/dpu: remove topology name
  drm/msm/dpu: remove display H_TILE from encoder
  drm/msm/dpu: remove cdm block support from resource manager

 drivers/gpu/drm/msm/Makefile                       |   1 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c           | 449 +++------------------
 drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h           | 111 +----
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c        | 203 ++--------
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h        |   6 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h   |  20 +-
 .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c   |  31 +-
 .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_vid.c   | 116 +-----
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.c         | 323 ---------------
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.h         | 139 -------
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c         |  14 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.h         |   4 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c        |  29 --
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.h        |   7 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.c          |  29 --
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.h          |   7 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.c         |  18 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.h         |  17 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.c        |   3 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h        |  16 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c             | 107 +----
 drivers/gpu/drm/msm/disp/dpu1/dpu_rm.h             |  24 --
 drivers/gpu/drm/msm/disp/dpu1/dpu_trace.h          |   4 -
 23 files changed, 150 insertions(+), 1528 deletions(-)
 delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.c
 delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.h
    
Revision 2
      Based on the comments received for the patch series[1] and to
make the review process a bit more easy, spliting up the 
patches for cleanup and resource manager refactor. This series 
cleans up and prepares the DPU for upcoming RM changes.

[1] https://patchwork.freedesktop.org/series/44669/

Thanks and Regards,
Jeykumar S.

Jeykumar Sankaran (19):
  drm/msm/dpu: remove debugfs support for misr
  drm/msm/dpu: squash power handle event types
  drm/msm/dpu: remove scalar config definitions
  drm/msm/dpu: remove resource pool manager
  drm/msm/dpu: remove ping pong split topology variables
  drm/msm/dpu: enable master-slave encoders explicitly
  drm/msm/dpu: use kms stored hw mdp block
  drm/msm/dpu: iterate for assigned hw ctl in virtual encoder
  drm/msm/dpu: avoid querying for hw intf before assignment
  drm/msm/dpu: make crtc get_mixer_width helper static
  drm/msm/dpu: move hw resource tracking to crtc state
  drm/msm/dpu: rename hw_ctl to lm_ctl
  drm/msm/dpu: clean up destination scaler residue
  drm/msm/dpu: remove cdm block support from resource manager
  drm/msm/dpu: remove LOCK/CLEAR support in RM
  drm/msm/dpu: remove display H_TILE from encoder
  drm/msm/dpu: remove RM dependency on connector state
  drm/msm/dpu: relax parameter validation in encoders
  drm/msm/dpu: remove RM topology definition

 drivers/gpu/drm/msm/Makefile                       |   1 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c           | 467 +++------------------
 drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h           | 111 +----
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c        | 216 ++--------
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h        |  10 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h   |  23 +-
 .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c   |  48 +--
 .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_vid.c   | 125 +-----
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c     |  56 +--
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h     |  56 ---
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.c         | 323 --------------
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.h         | 139 ------
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c         |  14 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.h         |   4 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c        |  29 --
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.h        |   7 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.c          |  29 --
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.h          |   7 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_mdss.h        |  33 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.c         |  18 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.h         |  17 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.c        |   3 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h        |  16 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c            |   8 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c   |  15 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h   |  14 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c             | 278 +++---------
 drivers/gpu/drm/msm/disp/dpu1/dpu_rm.h             |  44 --
 drivers/gpu/drm/msm/disp/dpu1/dpu_trace.h          |   4 -
 29 files changed, 216 insertions(+), 1899 deletions(-)
 delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.c
 delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.h
    
Revision 3
      Based on the comments received for the patch series[1] and to
make the review process a bit more easy, spliting up the 
patches for cleanup and resource manager refactor. This series 
cleans up and prepares the DPU for upcoming RM changes.

[1] https://patchwork.freedesktop.org/series/44669/

changes in v4:
	- split clean up changes in separate commits
changes in v5:
	- clean up dead code in RM
	- remove both topology enums and RM specific topology
	- update commit texts with reasons for removal
changes in v6:
	- remove parameter checks in RM reserve
	- use BIT(x) to define power handle event macros
	- avoid head allocation for RM topology

Thanks and Regards,
Jeykumar S.

Jeykumar Sankaran (19):
  drm/msm/dpu: remove debugfs support for misr
  drm/msm/dpu: squash power handle event types
  drm/msm/dpu: remove scalar config definitions
  drm/msm/dpu: remove resource pool manager
  drm/msm/dpu: remove ping pong split topology variables
  drm/msm/dpu: enable master-slave encoders explicitly
  drm/msm/dpu: use kms stored hw mdp block
  drm/msm/dpu: iterate for assigned hw ctl in virtual encoder
  drm/msm/dpu: avoid querying for hw intf before assignment
  drm/msm/dpu: make crtc get_mixer_width helper static
  drm/msm/dpu: move hw resource tracking to crtc state
  drm/msm/dpu: rename hw_ctl to lm_ctl
  drm/msm/dpu: clean up destination scaler residue
  drm/msm/dpu: remove cdm block support from resource manager
  drm/msm/dpu: remove LOCK/CLEAR support in RM
  drm/msm/dpu: remove display H_TILE from encoder
  drm/msm/dpu: remove RM dependency on connector state
  drm/msm/dpu: relax parameter validation in encoders
  drm/msm/dpu: remove RM topology definition

 drivers/gpu/drm/msm/Makefile                       |   1 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c           | 467 +++------------------
 drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h           | 111 +----
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c        | 216 ++--------
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h        |  10 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h   |  23 +-
 .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c   |  48 +--
 .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_vid.c   | 125 +-----
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c     |  56 +--
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h     |  56 ---
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.c         | 323 --------------
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.h         | 139 ------
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c         |  14 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.h         |   4 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c        |  29 --
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.h        |   7 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.c          |  29 --
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.h          |   7 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_mdss.h        |  33 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.c         |  18 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.h         |  17 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.c        |   3 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h        |  16 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c            |   8 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c   |  15 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h   |  14 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c             | 285 +++----------
 drivers/gpu/drm/msm/disp/dpu1/dpu_rm.h             |  44 --
 drivers/gpu/drm/msm/disp/dpu1/dpu_trace.h          |   4 -
 29 files changed, 214 insertions(+), 1908 deletions(-)
 delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.c
 delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.h
    

Revisions

Patches download mbox

# Name Submitter State A F R T
[v5,01/19] drm/msm/dpu: remove debugfs support for misr Jeykumar Sankaran New 1
[v5,02/19] drm/msm/dpu: squash power handle event types Jeykumar Sankaran New 1
[v5,03/19] drm/msm/dpu: remove scalar config definitions Jeykumar Sankaran New 1
[v5,04/19] drm/msm/dpu: remove resource pool manager Jeykumar Sankaran New 1
[v5,05/19] drm/msm/dpu: remove ping pong split topology variables Jeykumar Sankaran New 1
[v5,06/19] drm/msm/dpu: enable master-slave encoders explicitly Jeykumar Sankaran Accepted 1
[v5,07/19] drm/msm/dpu: use kms stored hw mdp block Jeykumar Sankaran New 1
[v5,08/19] drm/msm/dpu: iterate for assigned hw ctl in virtual encoder Jeykumar Sankaran Accepted
[v5,09/19] drm/msm/dpu: avoid querying for hw intf before assignment Jeykumar Sankaran New 1
[v5,10/19] drm/msm/dpu: make crtc get_mixer_width helper static Jeykumar Sankaran Accepted
[v5,11/19] drm/msm/dpu: move hw resource tracking to crtc state Jeykumar Sankaran Accepted
[v5,12/19] drm/msm/dpu: rename hw_ctl to lm_ctl Jeykumar Sankaran Accepted 1
[v5,13/19] drm/msm/dpu: clean up destination scaler residue Jeykumar Sankaran Accepted
[v5,14/19] drm/msm/dpu: remove cdm block support from resource manager Jeykumar Sankaran Accepted
[v5,15/19] drm/msm/dpu: remove LOCK/CLEAR support in RM Jeykumar Sankaran Accepted
[v5,16/19] drm/msm/dpu: remove display H_TILE from encoder Jeykumar Sankaran Accepted 1
[v5,17/19] drm/msm/dpu: remove RM dependency on connector state Jeykumar Sankaran New 1
[v5,18/19] drm/msm/dpu: relax parameter validation in encoders Jeykumar Sankaran Accepted 1
[v5,19/19] drm/msm/dpu: remove RM topology definition Jeykumar Sankaran New

Patches download mbox

# Name Submitter State A F R T
[v6,01/19] drm/msm/dpu: remove debugfs support for misr Jeykumar Sankaran New 1
[v6,02/19] drm/msm/dpu: squash power handle event types Jeykumar Sankaran New 1
[v6,03/19] drm/msm/dpu: remove scalar config definitions Jeykumar Sankaran New 1
[v6,04/19] drm/msm/dpu: remove resource pool manager Jeykumar Sankaran New 1
[v6,05/19] drm/msm/dpu: remove ping pong split topology variables Jeykumar Sankaran New 1
[v6,06/19] drm/msm/dpu: enable master-slave encoders explicitly Jeykumar Sankaran New 1
[v6,07/19] drm/msm/dpu: use kms stored hw mdp block Jeykumar Sankaran New 1
[v6,08/19] drm/msm/dpu: iterate for assigned hw ctl in virtual encoder Jeykumar Sankaran New 1
[v6,09/19] drm/msm/dpu: avoid querying for hw intf before assignment Jeykumar Sankaran New 1
[v6,10/19] drm/msm/dpu: make crtc get_mixer_width helper static Jeykumar Sankaran New 1
[v6,11/19] drm/msm/dpu: move hw resource tracking to crtc state Jeykumar Sankaran New 1
[v6,12/19] drm/msm/dpu: rename hw_ctl to lm_ctl Jeykumar Sankaran New 1
[v6,13/19] drm/msm/dpu: clean up destination scaler residue Jeykumar Sankaran New 1
[v6,14/19] drm/msm/dpu: remove cdm block support from resource manager Jeykumar Sankaran New 1
[v6,15/19] drm/msm/dpu: remove LOCK/CLEAR support in RM Jeykumar Sankaran New 1
[v6,16/19] drm/msm/dpu: remove display H_TILE from encoder Jeykumar Sankaran New 1
[v6,17/19] drm/msm/dpu: remove RM dependency on connector state Jeykumar Sankaran Accepted 1
[v6,18/19] drm/msm/dpu: relax parameter validation in encoders Jeykumar Sankaran New 1
[v6,19/19] drm/msm/dpu: remove RM topology definition Jeykumar Sankaran Accepted