Remove hand rolled msm property caching to handle DPU
custom properties. This change also cleans up all its
dependencies to cache and restore respective drm
states.
Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org>
---
drivers/gpu/drm/msm/Makefile | 1 -
drivers/g
Replace custom plane zpos property with drm core zpos
property. CRTC relies on the normalized zpos values
to configure blend stages of each plane.
Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org>
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 36 +--
d
remove unwanted dpu uapi headers exposing custom
payload layouts for custom properties
Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org>
---
include/uapi/drm/dpu_drm.h| 220 ---
include/uapi/drm/msm_drm_pp.h | 345 ---
Remove dpu crtc custom properties and its handlers.
Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org>
---
drivers/gpu/drm/msm/Makefile | 1 -
drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 28 -
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
Cleanup residual connector property enumerations.
Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org>
---
drivers/gpu/drm/msm/msm_drv.h | 27 ---
1 file changed, 27 deletions(-)
diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h
index 9
]
available on the drm-next tip.
[1]https://patchwork.kernel.org/patch/10202847/
Thanks.
Jeykumar Sankaran (7):
drm/msm: remove connector custom properties
drm/msm/dpu: clean up dpu plane custom properties
drm/msm: enable zpos normalization
drm/msm/dpu: switch to drm zpos property
drm/msm/dpu
Enable drm core zpos normalization for planes.
Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org>
---
drivers/gpu/drm/msm/msm_drv.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
index ebc40a9..549359e
in v3:
- none
Reviewed-by: Sean Paul <seanp...@chromium.org>
Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org>
Signed-off-by: Sean Paul <seanp...@chromium.org>
Signed-off-by: Rajesh Yadav <rya...@codeaurora.org>
---
drivers/gpu/drm/msm/disp/dpu1/dp
compiled out non-dsi display init (Sean Paul)
- removed file changes not applicable upstream (Sean Paul)
- Split unrelated changes into seperate patch sets (Sean Paul)
changes in v3:
- fix warnings
- compile out dsi-staging with upstream dsi hook up
Jeykumar Sankaran (6):
drm/msm
Remove custom ioctl support in SDM845 which allows
user space to register/unregister for hw events.
changes in v2:
- none
changes in v3:
- none
Reviewed-by: Sean Paul <seanp...@chromium.org>
Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org>
Signed-off-by: Sean
Upstream DSI driver doesn't support DSC panels yet. Remove
the support for compression from DPU for now.
changes in v2:
- indents and unrelated change clean up (Sean Paul)
- fix compilation dependency in dsi-staging
changes in v3:
- none
Signed-off-by: Jeykumar Sankaran
compilation warning
- compile out dsi staging
Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org>
Signed-off-by: Sean Paul <seanp...@chromium.org>
Signed-off-by: Rajesh Yadav <rya...@codeaurora.org>
---
drivers/gpu/drm/msm/Makefile | 22 -
drivers/gpu
Remove autorefresh support for smart panels in SDM845 for now.
It needs more discussion to figure out the user space
communication to set preference for the feature.
changes in v2:
- none
changes in v3:
- none
Reviewed-by: Sean Paul <seanp...@chromium.org>
Signed-off-by: Je
for CRTC/PLANE and CONNECTORS but not for private_obj's?
Thanks,
Jeykumar S.
Cc: Jeykumar Sankaran <jsa...@codeaurora.org>
Reviewed-by: Jeykumar Sankaran <jsa...@codeaurora.org>
Signed-off-by: Sean Paul <seanp...@chromium.org>
---
drivers/gpu/drm/msm/disp/mdp5/mdp5_
cleans up left out scalar config definitions from headers
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h| 1 -
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h | 10 --
2 files changed, 11 deletions(-)
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h
Subclass drm private state for DPU for handling driver
specific data. Adds atomic private object and private object
lock to dpu kms. Provides helper function to retrieve DPU
private data from current atomic state.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 66
and display thread by reserving resources
only in atomic check.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 210 +++---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 59 +-
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c| 223 ++
drivers
resource pool manager utility was introduced to manage
rotator sessions. Removing the support as the rotator
feature doesn't exist.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 494 ---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 84
This patchset introduces drm private object in KMS to manage HW
resource management. It modifies the resource manager by
introducing API's to do per DRM object resource allocation/cleanups.
The patchset is based on: https://patchwork.kernel.org/patch/10461375/
Jeykumar Sankaran (4):
drm/msm
On 2018-06-13 09:29, Jordan Crouse wrote:
On Tue, Jun 12, 2018 at 06:17:44PM -0700, Jeykumar Sankaran wrote:
Subclass drm private state for DPU for handling driver
specific data. Adds atomic private object and private object
lock to dpu kms. Provides helper function to retrieve DPU
private data
On 2018-06-13 09:44, Jordan Crouse wrote:
On Tue, Jun 12, 2018 at 06:17:47PM -0700, Jeykumar Sankaran wrote:
Switch to state based resource management. This patch
overhauls the resource manager and HW allocation methods by
maintaining the global resource pool and allocated hw
blocks
Cleanup residual connector property enumerations.
changes in v2:
- none
changes in v3:
- rebased on https://gitlab.freedesktop.org/seanpaul/
dpu-staging/commit/481d29d31cd629fd216381b53de5695f645465d5
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm
remove unwanted dpu uapi headers exposing custom
payload layouts for custom properties
changs in v2:
- none
changes in v3:
- rebased on https://gitlab.freedesktop.org/seanpaul/
dpu-staging/commit/481d29d31cd629fd216381b53de5695f645465d5
Signed-off-by: Jeykumar Sankaran
Enable drm core zpos normalization for planes.
changes in v2:
- none
changes in v3:
- rebased on https://gitlab.freedesktop.org/seanpaul/
dpu-staging/commit/481d29d31cd629fd216381b53de5695f645465d5
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm
/
dpu-staging/commit/481d29d31cd629fd216381b53de5695f645465d5
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 36 +--
drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 16 +++---
2 files changed, 14 insertions(+), 38 deletions
-staging/commit/481d29d31cd629fd216381b53de5695f645465d5
Thanks.
Jeykumar Sankaran (7):
drm/msm: remove connector custom properties
drm/msm/dpu: clean up dpu plane custom properties
drm/msm: enable zpos normalization
drm/msm/dpu: switch to drm zpos property
Remove dpu crtc custom properties
/481d29d31cd629fd216381b53de5695f645465d5
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul dd
---
drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 2 -
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 239 +
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 16 -
drivers/gpu/drm/msm/disp
changes in v2:
- none
changes in v3:
- rebased on https://gitlab.freedesktop.org/seanpaul/
dpu-staging/commit/481d29d31cd629fd216381b53de5695f645465d5
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/Makefile | 1 -
drivers/gpu
On 2018-06-04 12:53, Sean Paul wrote:
On Wed, May 23, 2018 at 12:30:57PM -0700, Jeykumar Sankaran wrote:
This change removes all the dpu plane custom properties
and its handlers.
Signed-off-by: Jeykumar Sankaran
---
Makefile |2 +-
drivers/gpu
Qualcomm Snapdragon chipsets uses compressed format
to optimize BW across multiple IP's. This change adds
needed modifier support in drm for a simple 4x4 tile
based compressed variants of base formats.
Signed-off-by: Jeykumar Sankaran
---
include/uapi/drm/drm_fourcc.h | 13 +
1 file
Replace custom plane zpos property with drm core zpos
property. CRTC relies on the normalized zpos values
to configure blend stages of each plane.
changes in v2:
- Move out unrelated changes in plane init (Sean Paul)
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1
Remove dpu crtc custom properties and its handlers.
changes in v2:
- none
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/Makefile | 1 -
drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 28 -
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
Cleanup residual connector property enumerations.
changs in v2:
- none
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/msm_drv.h | 27 ---
1 file changed, 27 deletions(-)
diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu
]
available on the drm-next tip.
[1]https://patchwork.kernel.org/patch/10202847/
Thanks.
changes in v2:
- remove stale code in blend config
- move unrelated code while updating zpos property
- Makefile changes
Jeykumar Sankaran (7):
drm/msm: remove connector custom properties
remove unwanted dpu uapi headers exposing custom
payload layouts for custom properties
changs in v2:
- none
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 1 -
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.c | 1 -
include/uapi
Remove hand rolled msm property caching to handle DPU
custom properties. This change also cleans up all its
dependencies to cache and restore respective drm
states.
changs in v2:
- none
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/Makefile
Enable drm core zpos normalization for planes.
changes in v2:
- none
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/msm_drv.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
index
On 2018-06-01 09:21, Sean Paul wrote:
On Fri, May 25, 2018 at 02:26:06PM -0700, Jeykumar Sankaran wrote:
Switch DPU from dsi-staging to upstream dsi driver. To make
the switch atomic, this change includes:
- remove dpu connector layers
- clean up dpu connector dependencies in encoder/crtc
compilation warning
- compile out dsi staging
changes in v4:
- remove top_ctrl check in rm release
Signed-off-by: Jeykumar Sankaran
Signed-off-by: Sean Paul
Signed-off-by: Rajesh Yadav
---
drivers/gpu/drm/msm/Makefile | 22 -
drivers/gpu/drm/msm/disp/dpu1
-by: Jeykumar Sankaran
Signed-off-by: Sean Paul
Signed-off-by: Rajesh Yadav
---
drivers/gpu/drm/msm/disp/dpu1/dpu_connector.c | 7 -
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c| 37 +--
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h | 20 --
.../gpu/drm/msm/disp/dpu1
top_ctrl in rm release
Jeykumar Sankaran (6):
drm/msm: remove display stream compression(DSC) support for SM845
drm/msm: remove support for ping pong split topology
drm/msm: remove panel autorefresh support for SDM845
drm/msm: strip down custom event ioctl's
drm/msm: hook up DPU with upstream
Remove custom ioctl support in SDM845 which allows
user space to register/unregister for hw events.
changes in v2:
- none
changes in v3:
- none
changes in v4:
- none
Reviewed-by: Sean Paul
Signed-off-by: Jeykumar Sankaran
Signed-off-by: Sean Paul
Signed-off-by: Rajesh
-off-by: Jeykumar Sankaran
Signed-off-by: Sean Paul
Signed-off-by: Rajesh Yadav
---
drivers/gpu/drm/msm/Makefile | 1 -
drivers/gpu/drm/msm/disp/dpu1/dpu_connector.c | 4 -
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c| 476 +
drivers/gpu
in v3:
- none
changes in v4:
- none
Reviewed-by: Sean Paul
Signed-off-by: Jeykumar Sankaran
Signed-off-by: Sean Paul
Signed-off-by: Rajesh Yadav
---
drivers/gpu/drm/msm/disp/dpu1/dpu_connector.c | 2 +-
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 19
On 2018-07-25 07:09, Stanimir Varbanov wrote:
Hi,
On 07/20/2018 11:42 PM, Sean Paul wrote:
From: Jeykumar Sankaran
Qualcomm Snapdragon chipsets uses compressed format
to optimize BW across multiple IP's. This change adds
needed modifier support in drm for a simple 4x4 tile
based compressed
: Ie9d42eb3e93257816daf3d36c444a335645d65c6
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 165 ++---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 32 +-
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c| 133 ++--
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h | 2
Strip down the support for topology enums. It
can be replaced with simple hw count checks.
changes in v2:
- none
changes in v3:
- none
Change-Id: If9b2a4db5bbdf8545b99b6d90825e256d014382d
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
Encoder H_TILE values are not used for allocating the hw blocks.
no. of hw_intf blocks provides the info.
changes in v2:
- none
changes in v3:
- none
Change-Id: I1c1c13e9b9f608fbaa8c5897f9f1892029107ac5
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1
://gitlab.freedesktop.org/seanpaul/dpu-staging/commits/for-next
Change-Id: Iaab32badff224ffed024e6ef6576efc8b3af3aec
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 61 +
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 15
2 files
Prep change for state based resource management.
Rename hw_ctl to lm_ctl to mean the ctl associated
with the hw layer mixer block.
changes in v2:
- none
changes in v3:
- none
Change-Id: If6e6249e089b89225cdfafe9158f7509e97b
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu
Prep changes for state based resource management.
Moves all the hw block tracking for the crtc to the state
object.
changes in v2:
- none
changes in v3:
- none
Change-Id: I2816e9e28b27f1126b477d62eb3858a30a652747
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp
Instead of iterating for hw ctrl per physical encoder, this
patch moves the iterations and assignment to the virtual encoder.
changes in v2:
- none
changes in v3:
- none
Change-Id: I896a8c36d6353986582e9d0fe3da9b2293579d4b
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm
v2:
- none
changes in v3:
- none
Change-Id: Ib0a2253431468151355e50cbad7b91e2b77b6e54
Signed-off-by: Jeykumar Sankaran
---
.../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_vid.c | 53 +++---
1 file changed, 16 insertions(+), 37 deletions(-)
diff --git a/drivers/gpu/drm/
Identify slave-master encoders and program them explicitly.
changes in v2:
- none
changes in v3:
- none
Change-Id: I0ebfada05bd7f8437f842ad860490a678aa8f4cd
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 39 ++---
1
removes left out variables of previous ping pong
split topology cleanup.
changes in v2:
- none
changes in v3:
- none
Change-Id: I1bf9d242039ce7cfd271233fa27840e83184fb95
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 3 ---
1 file changed, 3
: Ib045f1c66269be650bce5896c459f59e1047a53f
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 205 ---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 56 -
2 files changed, 261 deletions(-)
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
b
Avoid querying RM for hw mdp block. Use the one
stored in KMS during initialization.
changes in v2:
- none
changes in v3:
- none
Change-Id: I52129b96bd561a5547507d7f567bcaa3dbe554aa
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c | 12
cleans up left out scalar config definitions from headers
changes in v2:
- none
changes in v3:
- none
Change-Id: Id824dd5075c666f97b964573c97215bb786eac75
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h| 2 --
drivers/gpu/drm/msm/disp/dpu1
/for-next
Jeykumar Sankaran (13):
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: program master-slave encoders explicitly
drm/msm/dpu: use kms stored hw mdp block
drm/msm/dpu
This change validates the physical encoder before it
is dereferenced.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
b/drivers/gpu/drm/msm/disp
On 2018-08-14 13:14, Sean Paul wrote:
On Tue, Aug 07, 2018 at 08:20:08PM -0700, Jeykumar Sankaran wrote:
Strip down the support for topology enums. It
can be replaced with simple hw count checks.
Can you remove the enum entirely? A quick scan seems like it might be
possible,
it seems like
On 2018-08-14 12:19, Sean Paul wrote:
On Tue, Aug 07, 2018 at 08:12:31PM -0700, Jeykumar Sankaran wrote:
Identify slave-master encoders and program them explicitly.
You've got the what, but could you please explain the why?
changes in v2:
- none
changes in v3:
- none
On 2018-08-14 13:26, Sean Paul wrote:
On Tue, Aug 07, 2018 at 08:20:10PM -0700, Jeykumar Sankaran wrote:
Subclass drm private state for DPU for handling driver
specific data. Adds atomic private object and private object
lock to dpu kms. Provides helper function to retrieve DPU
private data
not a connector! We really want to
use DRM_MODE_ENCODER_DSI (integer 6) instead of DRM_MODE_CONNECTOR_DSI
here, or we'll go out of bounds of the encoder array. Pass the right
thing and everything is fine.
Cc: Jeykumar Sankaran
Cc: Jordan Crouse
Cc: Sean Paul
Fixes: 25fdd5933e4c ("drm/msm
the conservative bounds checks from atomic and clips
the dst rect such that we properly display planes which go off the
screen.
Changes in v2:
- Apply the clip to src as well (taking into account scaling)
Cc: Sravanthi Kollukuduru
Cc: Jeykumar Sankaran
Signed-off-by: Sean Paul
---
drivers/gpu/drm/msm
On 2018-08-28 15:52, Sean Paul wrote:
On Tue, Aug 28, 2018, 6:04 PM Jeykumar Sankaran
wrote:
On 2018-08-28 13:50, Sean Paul wrote:
From: Sean Paul
The atomic_check is a bit too aggressive with respect to planes
which
leave the active area. This caused a bunch of log spew when the
cursor
This change gets rid of unwanted connector-encoder type
mapping used for dsi-staging driver. Now that DPU will
be using upstream DSI driver, remove the stale code.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 20 ++--
1 file changed, 6
DPU is broken with patch [1]. This patch cleans up few stale codes
and fixes the DPU bug by using encoder type to identify displays.
[1] https://patchwork.kernel.org/patch/10568269/
Thanks.
Jeykumar Sankaran (3):
drm/msm/dpu: remove stale display port programming
drm/msm/dpu: remove
Remove stale display port programming. It can be
added back with DPU support for display port.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
b/drivers/gpu/drm
With patch [1], DPU is broken since it continues to use
incorrect connector_type to identify the display type. Update
DPU to use the encoder type to get the info.
[1] https://patchwork.kernel.org/patch/10568269/
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
/
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
Strip down debugfs support for misr data read on layer mixers
and interfaces.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 139 -
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 6 -
drivers/gpu/drm/msm/disp/dpu1
Support for CDM block is not present in DPU. Remove CDM
handlers from resource manager.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/Makefile | 1 -
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h | 2 -
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h | 5
Encoder H_TILE values are not used for allocating the hw blocks.
no. of hw_intf blocks provides the info.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 5 -
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h | 3 ---
drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c
Identify slave-master encoders during initialization and enable
the encoders explicitly as the current logic has redundant and
ambiguous loops.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 46 ++---
1 file changed, 15 insertions
Rename hw_ctl to lm_ctl to mean the ctl associated
with the hw layer mixer block.
sed -i 's/\([*@.>]\)hw_ctl\([^s]\)/\1lm_ctl\2/g' dpu_crtc.c dpu_crtc.h
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 26 +-
drivers/gpu/drm/msm/disp/d
Strip down the support for topology enums. It
can be replaced with simple hw count checks.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c| 3 --
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h| 1 -
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h
Prep changes for state based resource management.
Moves all the hw block tracking for the crtc to the state
object.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 60 ++--
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 22 ++--
2
Instead of iterating for hw ctrl per physical encoder, this
patch moves the iterations and assignment to the virtual encoder.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c| 22 +++---
.../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c
Avoid querying RM for hw mdp block. Use the one
stored in KMS during initialization.
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c | 12 +---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_vid.c | 9 +
2 files
removes left out variables of previous ping pong
split topology cleanup.
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h
b/drivers/gpu/drm/msm
resource pool manager utility was introduced to manage
rotator sessions. Removing the support as the rotator
feature doesn't exist.
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 205 ---
drivers/gpu/drm/msm
Resource manager assigns hw_intf blocks for the encoder only on
modeset. If queried for hw_intf objects during init, it will be
NULL. Since hw_intf objects are needed only during encoder enable,
defer the query to encoder enable which will be triggered after
modeset.
Signed-off-by: Jeykumar
Mark CRTC get_mixer_width helper API static as it is
not used outside the file.
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 21 ++---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 18 --
2 files changed, 18 insertions(+), 21
cleans up left out scalar config definitions from headers
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h| 2 --
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h | 10 --
2 files changed, 12 deletions(-)
diff --git a/drivers/gpu/drm
On 2018-08-31 07:56, Sean Paul wrote:
On Tue, Aug 28, 2018 at 05:39:59PM -0700, Jeykumar Sankaran wrote:
Prep changes for state based resource management.
Moves all the hw block tracking for the crtc to the state
object.
Changes in v4...
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu
On 2018-08-30 09:24, Sean Paul wrote:
On Tue, Aug 28, 2018 at 05:39:54PM -0700, Jeykumar Sankaran wrote:
Identify slave-master encoders during initialization and enable
the encoders explicitly as the current logic has redundant and
ambiguous loops.
Please include a "Changes in vX"
Cc: Jeykumar Sankaran
Cc: Ville Syrjälä
Signed-off-by: Sean Paul
---
Reviewed-by: Jeykumar Sankaran
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 3 +-
drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 34 +-
drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c | 1 -
drivers/gpu/drm
On 2018-08-31 09:08, Sean Paul wrote:
On Tue, Aug 28, 2018 at 05:40:01PM -0700, Jeykumar Sankaran wrote:
Strip down the support for topology enums. It
can be replaced with simple hw count checks.
Changes in v4:
...
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1
On 2018-08-31 07:56, Sean Paul wrote:
On Tue, Aug 28, 2018 at 05:39:59PM -0700, Jeykumar Sankaran wrote:
Prep changes for state based resource management.
Moves all the hw block tracking for the crtc to the state
object.
Changes in v4...
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu
cleans up left out scalar config definitions from headers
changes in v4:
- none
changes in v5:
- none
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h| 2 --
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h | 10 --
2
only ENABLE/DISABLE events.
changes in v5:
- introduced in the series
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 28 +---
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 8 +++
drivers/gpu/drm/msm/disp/dpu1
of temporary variable for phys enc(Sean)
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 43 -
1 file changed, 12 insertions(+), 31 deletions(-)
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
b/drivers/gpu/drm
removes left out variables of previous ping pong
split topology cleanup.
changes in v4:
- none
changes in v5:
- none
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 3 ---
1 file changed, 3 deletions(-)
diff --git
iteration and mapping from physical
encoder to virtual encoder.
changes in v4:
- Fix hw_ctl initialization (Sean)
changes in v5:
- Update commit text with details on why the change is
needed (Sean)
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1
the feature is
formally implemented.
changes in v5:
- introduced in the series
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 42 +-
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 40
drivers/gpu/drm
details on commit text on why the change is needed (Sean)
changes in v5:
- Reword commit text on the usage of hw_intf objects (Sean)
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
.../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_vid.c | 53 +++---
1 file changed
resource pool manager utility was introduced to manage
rotator sessions. Removing the support as the rotator
feature doesn't exist.
changes in v4:
- none
changes in v5:
- none
Signed-off-by: Jeykumar Sankaran
Reviewed-by: Sean Paul
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
/
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
the driver.
changes in v4:
- changed introduced in the series
changes in v5:
- update commit text with the need for the change(Sean)
Signed-off-by: Jeykumar Sankaran
---
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 139 -
drivers/gpu/drm/msm/disp/dpu1
1 - 100 of 383 matches
Mail list logo