[Freedreno] [PATCH v3 07/13] drm/msm/dpu: avoid querying for hw intf before assignment

2018-08-07 Thread Jeykumar Sankaran
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/

[Freedreno] [PATCH v3 06/13] drm/msm/dpu: iterate for assigned hw ctl in virtual encoder

2018-08-07 Thread Jeykumar Sankaran
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

[Freedreno] [PATCH v3 05/13] drm/msm/dpu: use kms stored hw mdp block

2018-08-07 Thread Jeykumar Sankaran
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

[Freedreno] [PATCH v3 02/13] drm/msm/dpu: remove resource pool manager

2018-08-07 Thread Jeykumar Sankaran
: 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

[Freedreno] [PATCH v3 03/13] drm/msm/dpu: remove ping pong split topology variables

2018-08-07 Thread Jeykumar Sankaran
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

[Freedreno] [PATCH v3 00/13] Atomic resource management

2018-08-07 Thread Jeykumar Sankaran
/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

[Freedreno] [PATCH v3 01/13] drm/msm/dpu: remove scalar config definitions

2018-08-07 Thread Jeykumar Sankaran
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

[Freedreno] [PATCH] drm/msm/disp/dpu: fix early dereference of physical encoder

2018-08-07 Thread Jeykumar Sankaran
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

Re: [Freedreno] [bug report] drm/msm: Add SDM845 DPU support

2018-08-07 Thread Jeykumar Sankaran
On 2018-08-07 05:09, Dan Carpenter wrote: Hello Jeykumar Sankaran, This is a semi-automatic email about new static checker warnings. The patch 25fdd5933e4c: "drm/msm: Add SDM845 DPU support" from Jun 27, 2018, leads to the following Smatch complaint: ./drivers/gpu/drm/msm

Re: [Freedreno] [PATCH v3 03/19] drm: add msm compressed format modifiers

2018-07-25 Thread Jeykumar Sankaran
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

Re: [Freedreno] [DPU PATCH 4/4] drm/msm/dpu: use private obj to track hw resources

2018-06-13 Thread Jeykumar Sankaran
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

Re: [Freedreno] [DPU PATCH 1/4] drm/msm/dpu: add atomic private object to dpu kms

2018-06-13 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH 4/4] drm/msm/dpu: use private obj to track hw resources

2018-06-12 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH 1/4] drm/msm/dpu: add atomic private object to dpu kms

2018-06-12 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH 3/4] drm/msm/dpu: remove resource pool manager

2018-06-12 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH 2/4] drm/msm/dpu: remove scalar config definitions

2018-06-12 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH 0/4] Atomic resource management

2018-06-12 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v3 3/7] drm/msm: enable zpos normalization

2018-06-11 Thread 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

[Freedreno] [DPU PATCH v3 0/7] clean up DPU custom properties

2018-06-11 Thread Jeykumar Sankaran
-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

[Freedreno] [DPU PATCH v3 1/7] drm/msm: remove connector custom properties

2018-06-11 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v3 5/7] Remove dpu crtc custom properties and its handlers.

2018-06-11 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v3 7/7] drm/msm: remove dpu specific uapi header

2018-06-11 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v2 7/7] drm/msm: remove dpu specific uapi header

2018-06-05 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v2 5/7] drm/msm/dpu: clean up dpu crtc custom properties

2018-06-05 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v2 6/7] drm/msm: remove msm_prop files

2018-06-05 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v2 1/7] drm/msm: remove connector custom properties

2018-06-05 Thread Jeykumar Sankaran
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

Re: [Freedreno] [DPU PATCH 2/7] drm/msm/dpu: clean up dpu plane custom properties

2018-06-05 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v4 0/6] Switch DPU to use upstream DSI driver for SDM845

2018-06-01 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v4 5/6] drm/msm: hook up DPU with upstream DSI

2018-06-01 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v4 2/6] drm/msm: remove support for ping pong split topology

2018-06-01 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v4 4/6] drm/msm: strip down custom event ioctl's

2018-06-01 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v4 3/6] drm/msm: remove panel autorefresh support for SDM845

2018-06-01 Thread Jeykumar Sankaran
-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

[Freedreno] [DPU PATCH v4 1/6] drm/msm: remove display stream compression(DSC) support for SM845

2018-06-01 Thread Jeykumar Sankaran
-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

Re: [Freedreno] [DPU PATCH v3 5/6] drm/msm: hook up DPU with upstream DSI

2018-06-01 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v3 2/6] drm/msm: remove support for ping pong split topology

2018-05-25 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v3 3/6] drm/msm: remove panel autorefresh support for SDM845

2018-05-25 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v3 5/6] drm/msm: hook up DPU with upstream DSI

2018-05-25 Thread 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

[Freedreno] [DPU PATCH v3 1/6] drm/msm: remove display stream compression(DSC) support for SM845

2018-05-25 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH v3 4/6] drm/msm: strip down custom event ioctl's

2018-05-25 Thread Jeykumar Sankaran
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

Re: [Freedreno] [DPU PATCH v3] drm/msm: Use atomic private_obj instead of subclassing

2018-05-24 Thread Jeykumar Sankaran
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_

[Freedreno] [DPU PATCH 6/7] drm/msm: remove msm_prop files

2018-05-23 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH 4/7] drm/msm/dpu: switch to drm zpos property

2018-05-23 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH 7/7] drm/msm: remove dpu specific uapi header

2018-05-23 Thread Jeykumar Sankaran
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 ---

[Freedreno] [DPU PATCH 0/7] clean up DPU custom properties

2018-05-23 Thread Jeykumar Sankaran
] 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

[Freedreno] [DPU PATCH 5/7] drm/msm/dpu: clean up dpu crtc custom properties

2018-05-23 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH 1/7] drm/msm: remove connector custom properties

2018-05-23 Thread Jeykumar Sankaran
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

[Freedreno] [DPU PATCH 3/7] drm/msm: enable zpos normalization

2018-05-23 Thread Jeykumar Sankaran
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

Re: [Freedreno] [DPU PATCH] drm/msm: dpu: Fix build warnings

2018-04-20 Thread Jeykumar Sankaran
On 2018-04-20 09:07, Sean Paul wrote: Signed-off-by: Sean Paul <seanp...@chromium.org> --- Reviewed-by: Jeykumar Sankaran <jsa...@codeaurora.org> .../gpu/drm/msm/disp/dpu1/dpu_color_processing.c | 5 +++-- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 4 ++-- drivers

[Freedreno] [DPU PATCH v2 1/6] drm/msm: remove display stream compression(DSC) support for SM845

2018-04-19 Thread Jeykumar Sankaran
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 Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.

[Freedreno] [DPU PATCH v2 3/6] drm/msm: remove panel autorefresh support for SDM845

2018-04-19 Thread Jeykumar Sankaran
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 Reviewed-by: Sean Paul <seanp...@chromium.org> Signed-off-by: Jeykumar Sankara

[Freedreno] [DPU PATCH v2 4/6] drm/msm: strip down custom event ioctl's

2018-04-19 Thread Jeykumar Sankaran
Remove custom ioctl support in SDM845 which allows user space to register/unregister for hw events. changes in v2: - none Reviewed-by: Sean Paul <seanp...@chromium.org> Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/disp/dpu1/dpu_

[Freedreno] [DPU PATCH v2 2/6] drm/msm: remove support for ping pong split topology

2018-04-19 Thread Jeykumar Sankaran
Reviewed-by: Sean Paul <seanp...@chromium.org> Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/disp/dpu1/dpu_connector.c | 2 +- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 19 --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c

[Freedreno] [DPU PATCH v2 0/6] Switch DPU to use upstream DSI driver for SDM845

2018-04-19 Thread Jeykumar Sankaran
) - removed 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) Jeykumar Sankaran (6): drm/msm: remove display stream compression(DSC) support for SM845 drm/msm

[Freedreno] [DPU PATCH] drm/msm: populate aspace in msm_kms

2018-04-19 Thread Jeykumar Sankaran
display drivers may need to access aspace to allocate internal buffers (e.g. DSI to allocate cmd buffer) This change populates the needed info. Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 7 +-- 1 file changed, 5 insertions

Re: [Freedreno] [DPU PATCH 5/6] drm/msm: hook up DPU with upstream DSI

2018-04-19 Thread Jeykumar Sankaran
On 2018-04-19 08:38, Sean Paul wrote: On Mon, Apr 16, 2018 at 11:22:20AM -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

[Freedreno] [DPU PATCH 3/6] drm/msm: remove panel autorefresh support for SDM845

2018-04-16 Thread Jeykumar Sankaran
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. Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/disp/dpu1/dpu_connector.c | 7 - d

[Freedreno] [DPU PATCH 0/6] Switch DPU to use upstream DSI driver for SDM845

2018-04-16 Thread Jeykumar Sankaran
driver by removing the dsi-staging driver and it's dependent sources. The patch series is based on: [1]https://www.spinics.net/lists/dri-devel/msg172315.html [2]https://www.spinics.net/lists/dri-devel/msg172395.html Jeykumar Sankaran (6): drm/msm: remove display stream compression(DSC) support

[Freedreno] [DPU PATCH 5/6] drm/msm: hook up DPU with upstream DSI

2018-04-16 Thread Jeykumar Sankaran
to remove the driver) - adapt upstream device hierarchy Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- .../config/arm64/chromiumos-arm64.flavour.config |4 +- .../arm64/chromiumos-qualcomm.flavour.config |4 +- drivers/gpu/drm/msm/Makefile

[Freedreno] [DPU PATCH 2/6] drm/msm: remove support for ping pong split topology

2018-04-16 Thread Jeykumar Sankaran
Ping pong split topology was meant for low end soc's which doesn't have enough layer mixers to support split panels. Considering how uncommon the topology is for current chipset's and also to simply the driver programming, striping off the support for SDM845. Signed-off-by: Jeykumar Sankaran <

[Freedreno] [DPU PATCH 4/6] drm/msm: strip down custom event ioctl's

2018-04-16 Thread Jeykumar Sankaran
Remove custom ioctl support in SDM845 which allows user space to register/unregister for hw events. Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 218 +-- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h

[Freedreno] [DPU PATCH 1/6] drm/msm: remove display stream compression(DSC) support for SM845

2018-04-16 Thread Jeykumar Sankaran
Upstream DSI driver doesn't support DSC panels yet. Remove the support for compression from DPU for now. Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/Makefile | 1 - drivers/gpu/drm/msm/disp/dpu1/dpu_connector.c | 4 - d

Re: [Freedreno] [DPU PATCH 06/11] drm/msm: Remove msm_commit/kthread, use atomic helper commit

2018-03-19 Thread Jeykumar Sankaran
On 2018-03-19 08:01, Sean Paul wrote: On Mon, Mar 12, 2018 at 04:23:10PM -0400, Sean Paul wrote: On Thu, Mar 08, 2018 at 05:08:03PM -0800, Jeykumar Sankaran wrote: > On 2018-03-02 06:56, Sean Paul wrote: > > On Thu, Mar 01, 2018 at 07:37:10PM -0500, Rob Clark wrote: > > > O

Re: [Freedreno] [DPU PATCH v2] drm/msm: Use atomic private_obj instead of subclassing

2018-03-19 Thread Jeykumar Sankaran
ate callback (Jeykumar) Cc: Jeykumar Sankaran <jsa...@codeaurora.org> Signed-off-by: Sean Paul <seanp...@chromium.org> With the below comment updated Reviewed-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 37 ++

Re: [Freedreno] [DPU PATCH v2] drm/msm: Don't duplicate modeset_enables atomic helper

2018-03-16 Thread Jeykumar Sankaran
On 2018-03-16 12:45, Sean Paul wrote: Instead, shuffle things around so we kickoff crtc after enabling encoder during modesets. Also moves the vblank wait to after the frame. Changes in v2: - Remove the encoder.commit hack, it's not required (Jeykumar) Cc: Jeykumar Sankaran <

Re: [Freedreno] [DPU PATCH 02/11] drm/msm: Don't duplicate modeset_enables atomic helper

2018-03-14 Thread Jeykumar Sankaran
On 2018-03-14 08:14, Sean Paul wrote: On Tue, Mar 13, 2018 at 04:57:35PM -0700, Jeykumar Sankaran wrote: On 2018-03-12 13:21, Sean Paul wrote: > On Thu, Mar 08, 2018 at 04:56:01PM -0800, Jeykumar Sankaran wrote: > > On 2018-02-28 11:18, Sean Paul wrote: > > > Instead, shuffl

Re: [Freedreno] [DPU PATCH 02/11] drm/msm: Don't duplicate modeset_enables atomic helper

2018-03-13 Thread Jeykumar Sankaran
On 2018-03-12 13:21, Sean Paul wrote: On Thu, Mar 08, 2018 at 04:56:01PM -0800, Jeykumar Sankaran wrote: On 2018-02-28 11:18, Sean Paul wrote: > Instead, shuffle things around so we kickoff crtc after enabling encoder > during modesets. Also moves the vblank wait to after the

Re: [Freedreno] [DPU PATCH 01/11] drm/msm: Skip seamless disables in crtc/encoder

2018-03-13 Thread Jeykumar Sankaran
sables in the > .disable hooks. > > Change-Id: I7bd9183ae60624204fb1de9550656b776efc7202 > Signed-off-by: Sean Paul <seanp...@chromium.org> Reviewed-by: Jeykumar Sankaran <jsa...@codeaurora.org> Can you consider getting rid of these checks? Do you mean the Change-Id? Yeah, I f

Re: [Freedreno] [DPU PATCH 07/11] drm/msm: Use atomic private_obj instead of subclassing

2018-03-08 Thread Jeykumar Sankaran
On 2018-02-28 11:19, Sean Paul wrote: Instead of subclassing atomic state, store driver private data in private_obj/state. This allows us to remove the swap_state driver hook for mdp5 and get closer to using the atomic helpers entirely. Change-Id: I65a4a2887593ae257d584e00b352b5daf00e4e61

Re: [Freedreno] [DPU PATCH 06/11] drm/msm: Remove msm_commit/kthread, use atomic helper commit

2018-03-08 Thread Jeykumar Sankaran
On 2018-03-02 06:56, Sean Paul wrote: On Thu, Mar 01, 2018 at 07:37:10PM -0500, Rob Clark wrote: On Thu, Mar 1, 2018 at 3:37 PM, wrote: > On 2018-03-01 07:27, Sean Paul wrote: >> >> On Wed, Feb 28, 2018 at 08:07:00PM -0800, jsa...@codeaurora.org wrote: >>> >>> On

Re: [Freedreno] [DPU PATCH 03/11] drm/msm: Refactor complete_commit() to look more the helpers

2018-03-08 Thread Jeykumar Sankaran
org> Reviewed-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/msm_atomic.c | 25 - 1 file changed, 16 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/msm/msm_atomic.c b/drivers/gpu/drm/msm/msm_atomic.c index f5794dce25dd..eb2ccda

Re: [Freedreno] [DPU PATCH 02/11] drm/msm: Don't duplicate modeset_enables atomic helper

2018-03-08 Thread Jeykumar Sankaran
On 2018-02-28 11:18, Sean Paul wrote: Instead, shuffle things around so we kickoff crtc after enabling encoder during modesets. Also moves the vblank wait to after the frame. Change-Id: I16c7b7f9390d04f6050aa20e17a5335fbf49eba3 Signed-off-by: Sean Paul ---

Re: [Freedreno] [PATCH RESEND 10/10] drm/msm: Restore msm_atomic_check hook

2018-03-08 Thread Jeykumar Sankaran
On 2018-02-21 07:18, Sean Paul wrote: Somehow this got lost, put it back. We might want to re-evaluate whether this is actually necessary, but for now let's actually use the thing. Signed-off-by: Sean Paul <seanp...@chromium.org> Reviewed-by: Jeykumar Sankaran <jsa...@codea

Re: [Freedreno] [PATCH RESEND 05/10] drm/msm: Defer probe if display component not found

2018-03-08 Thread Jeykumar Sankaran
On 2018-02-21 07:18, Sean Paul wrote: The downstream driver relies on Makefile ordering of files to ensure components probe in the right order. This took me entirely too long to sort out, so don't rely on that any longer. Signed-off-by: Sean Paul <seanp...@chromium.org> Reviewed-by: Je

Re: [Freedreno] [PATCH RESEND 04/10] drm/msm: Remove smmu driver init/cleanup from msm_drv

2018-03-08 Thread Jeykumar Sankaran
On 2018-02-21 07:18, Sean Paul wrote: Leftover bits from the initial removal of smmu. Signed-off-by: Sean Paul <seanp...@chromium.org> Reviewed-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/msm_drv.c | 6 -- 1 file changed, 6 deletions(-) diff --g

Re: [Freedreno] [PATCH RESEND 02/10] drm/msm: Include the dpu_dbg header in msm_drv.c

2018-03-08 Thread Jeykumar Sankaran
On 2018-02-21 07:18, Sean Paul wrote: We'll need to clean up these conditionals further, but at least fix the compilation error. Signed-off-by: Sean Paul <seanp...@chromium.org> Reviewed-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/msm_drv.c | 3 +++ 1

Re: [Freedreno] [PATCH RESEND 02/10] drm/msm: Include the dpu_dbg header in msm_drv.c

2018-03-08 Thread Jeykumar Sankaran
On 2018-02-21 07:18, Sean Paul wrote: We'll need to clean up these conditionals further, but at least fix the compilation error. Signed-off-by: Sean Paul <seanp...@chromium.org> Reviewed by Jeykumar Sankaran --- drivers/gpu/drm/msm/msm_drv.c | 3 +++ 1 file changed, 3 insertions(+)

Re: [Freedreno] [PATCH RESEND 01/10] drm/msm: Add displayport files to Makefile

2018-03-08 Thread Jeykumar Sankaran
On 2018-02-21 07:18, Sean Paul wrote: Still behind a config flag, and ymmv when trying to build them, but they're present at least. Signed-off-by: Sean Paul <seanp...@chromium.org> Reviewed-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/M

Re: [Freedreno] [DPU PATCH 05/11] drm/msm: Mark the crtc->state->event consumed

2018-03-05 Thread Jeykumar Sankaran
viewed-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 2 ++ drivers/gpu/drm/msm/disp/mdp4/mdp4_crtc.c | 1 + drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c | 1 + 3 files changed, 4 insertions(+) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c b

Re: [Freedreno] [DPU PATCH 11/11] drm/msm: Remove dpu input fences

2018-03-02 Thread Jeykumar Sankaran
anp...@chromium.org> Reviewed-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/Makefile | 1 - drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 84 - drivers/gpu/drm/msm/disp/dpu1/dpu_fence.c | 404 -- drivers/gpu/drm/msm/disp/dpu1

[Freedreno] [DPU PATCH v3 1/2] drm/msm/dsi-staging: remove support for partial update

2018-03-01 Thread Jeykumar Sankaran
: - cleanup partial update ROI in panel files Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/dsi-staging/dsi_ctrl.c| 20 --- drivers/gpu/drm/msm/dsi-staging/dsi_ctrl.h| 12 -- drivers/gpu/drm/msm/dsi-staging/dsi_defs.h| 2 - drivers/gpu/drm/msm/dsi-s

[Freedreno] [DPU PATCH v3 2/2] drm/msm: remove partial update support

2018-03-01 Thread Jeykumar Sankaran
header file definitions for partial update changes since v2: - remove compilation flag for DPU_AD4. Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/disp/dpu1/dpu_connector.c | 139 +-- drivers/gpu/drm/msm/disp/dpu1/dpu_connector.h | 7 +- drive

[Freedreno] [DPU PATCH v2 2/2] drm/msm: remove partial update support

2018-02-27 Thread Jeykumar Sankaran
Implementation of partial update in DPU DRM is heavily dependent on custom properties and dsi hooks. Removing the support for now. We may need to revisit the support in the future. Change-Id: Idd87272fe4d4c0a26fcb405154c0605af1edf1ba Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.

[Freedreno] [DPU PATCH 1/2] drm/msm/dsi-staging: compile out partial update path

2018-02-22 Thread Jeykumar Sankaran
compile out partial update related changes from dsi-staging since the DPU dependencies are getting removed. Change-Id: I02462f520cdf99c8445b18e60212ca46155f9710 Signed-off-by: Jeykumar Sankaran <jsa...@codeaurora.org> --- drivers/gpu/drm/msm/dsi-staging/dsi_display.c | 9 +++-- drive

<    1   2   3   4