[Freedreno] [DPU PATCH 3/3] drm/msm/dsi-staging: Gate bus scale code

2018-03-27 Thread Rajesh Yadav
DSI driver relies on downstream bus scaling driver (msm_bus) for bus bandwidth voting. Gate the bus bandwidth voting code under CONFIG_QCOM_BUS_SCALING. Signed-off-by: Rajesh Yadav --- drivers/gpu/drm/msm/dsi-staging/dsi_clk_manager.c | 8 drivers/gpu/drm/msm/dsi-staging/dsi_ctrl.c

[Freedreno] [DPU PATCH 2/3] drm/msm: Fix return type mismatch for dpu_kms_init

2018-03-27 Thread Rajesh Yadav
dpu_kms_init returns pointer to struct msm_kms but incase of platform_get_irq() failure, int was returned. Fix the return type to avoid compilation error. Signed-off-by: Rajesh Yadav --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/d

[Freedreno] [DPU PATCH 1/3] drm/msm: Remove unused variables

2018-03-27 Thread Rajesh Yadav
Fix compilation errors due to unused variables. Signed-off-by: Rajesh Yadav --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c| 2 +- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c | 1 - drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 1 - drivers/gpu/drm/msm/disp/dpu1/dpu_plan

[Freedreno] [DPU PATCH 0/3] Minor fixes in MSM DPU driver

2018-03-27 Thread Rajesh Yadav
This patch series includes few minor fixes in MSM DPU and dsi-staging stagging driver which are identified while pulling in the patches to downstream tree. DPU driver is currently hosted at https://gitlab.freedesktop.org/seanpaul/dpu-staging Rajesh Yadav (3): drm/msm: Remove unused variables

[Freedreno] [PATCH 2/4] drm/msm: Mark the crtc->state->event consumed

2018-03-27 Thread Sean Paul
Don't leave the event != NULL once it's consumed, this is used a signal to the atomic helpers that the event will be handled by the driver. Cc: Jeykumar Sankaran Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/mdp4/mdp4_crtc.c | 1 + drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c | 1 + 2 file

[Freedreno] [PATCH 0/4] drm/msm: Switch to atomic helpers

2018-03-27 Thread Sean Paul
I originally sent these patches targetted against the msm dpu code, but I've rebased them on msm-next since they're _mostly_ the same. The set is based on 'drm/msm: Use drm_private_obj/state instead of subclassing' which I sent up earlier. The set has been tested on mdp5 db410c. Sean Sean Paul (

[Freedreno] [PATCH 3/4] drm/msm: Remove msm_commit/worker, use atomic helper commit

2018-03-27 Thread Sean Paul
Moving further towards switching fully to the the atomic helpers, this patch removes the hand-rolled worker nonblock commit code and uses the atomic helpers commit_work model. Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/msm_atomic.c | 145 +-- drivers/gpu/drm/msm

[Freedreno] [PATCH 4/4] drm/msm: Switch to atomic_helper_commit()

2018-03-27 Thread Sean Paul
Now that all of the msm-specific goo is tucked safely away we can switch over to using the atomic helper commit directly. \o/ Cc: Abhinav Kumar Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/msm_atomic.c | 120 +-- drivers/gpu/drm/msm/msm_drv.c| 7 +- drivers

[Freedreno] [PATCH 1/4] drm/msm: Refactor complete_commit() to look more the helpers

2018-03-27 Thread Sean Paul
Factor out the commit_tail() portions of complete_commit() into a separate function to facilitate moving to the atomic helpers in future patches. Cc: Jeykumar Sankaran Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/msm_atomic.c | 23 ++- 1 file changed, 14 insertions(+), 9

[Freedreno] [PATCH v4] drm/msm: Use drm_private_obj/state instead of subclassing

2018-03-27 Thread Sean Paul
Now that we have private state handled by the core, we can use those instead of rolling our own swap_state for private data. Originally posted here: https://patchwork.freedesktop.org/patch/211361/ Changes in v2: - Use state->state in disp duplicate_state callback (Jeykumar) Changes in v3: - Upd

Re: [Freedreno] [PATCH v3 0/8] freedreno: a2xx improvements

2018-03-27 Thread Rob Clark
On Tue, Mar 27, 2018 at 5:53 AM, Wladimir J. van der Laan wrote: > >> Patchset looks reasonable to me, from a quick look. Testing a22x >> isn't so easy without resurrecting an ancient downstream kernel for >> old snadragon devices, so I think it is ok to ignore that. If the > > Do you mean: > >

Re: [Freedreno] [PATCH v3 0/8] freedreno: a2xx improvements

2018-03-27 Thread Wladimir J. van der Laan
> Patchset looks reasonable to me, from a quick look. Testing a22x > isn't so easy without resurrecting an ancient downstream kernel for > old snadragon devices, so I think it is ok to ignore that. If the Do you mean: - try to keep supporting a22x on best-effort basis by providing fallback b

Re: [Freedreno] [PATCH 00/23] drm: Eliminate plane->fb/crtc usage for atomic drivers

2018-03-27 Thread Daniel Vetter
On Thu, Mar 22, 2018 at 05:22:50PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > I really just wanted to fix i915 to re-enable its planes afer load > detection (a two line patch). This is what I actually ended up with > after I ran into a framebuffer refcount leak with said two line patch