[Freedreno] [PATCH 2/2] drm/msm/dpu: Wire up needs_dirtyfb

2021-05-08 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c index 5a74f93e29da..868ee6136438 100644 ---

[Freedreno] [PATCH 0/2] drm: Fix atomic helper dirtyfb stalls

2021-05-08 Thread Rob Clark
From: Rob Clark Someone on IRC once asked an innocent enough sounding question: Why with xf86-video-modesetting is es2gears limited at 120fps. So I broke out the perfetto tracing mesa MR and took a look. It turns out the problem was drm_atomic_helper_dirtyfb(), which would end up waiting for

Re: [Freedreno] [PATCH 1/1] drm/msm/dpu: Fix error return code in dpu_mdss_init()

2021-05-08 Thread Leizhen (ThunderTown)
On 2021/5/8 15:58, Stephen Boyd wrote: > Quoting Leizhen (ThunderTown) (2021-05-08 00:55:04) >> >> >> On 2021/5/8 14:09, Stephen Boyd wrote: >>> Quoting Zhen Lei (2021-05-07 19:42:54) Fix to return a negative error code from the error handling case instead of 0, as done elsewhere in

Re: [Freedreno] [PATCH 1/1] drm/msm/dpu: Fix error return code in dpu_mdss_init()

2021-05-08 Thread Stephen Boyd
Quoting Leizhen (ThunderTown) (2021-05-08 00:55:04) > > > On 2021/5/8 14:09, Stephen Boyd wrote: > > Quoting Zhen Lei (2021-05-07 19:42:54) > >> Fix to return a negative error code from the error handling case instead > >> of 0, as done elsewhere in this function. > >> > >> Fixes: 070e64dc1bbc

Re: [Freedreno] [PATCH 1/1] drm/msm/dpu: Fix error return code in dpu_mdss_init()

2021-05-08 Thread Leizhen (ThunderTown)
On 2021/5/8 14:09, Stephen Boyd wrote: > Quoting Zhen Lei (2021-05-07 19:42:54) >> Fix to return a negative error code from the error handling case instead >> of 0, as done elsewhere in this function. >> >> Fixes: 070e64dc1bbc ("drm/msm/dpu: Convert to a chained irq chip") >> Reported-by: Hulk

Re: [Freedreno] [PATCH 1/1] drm/msm/dpu: Fix error return code in dpu_mdss_init()

2021-05-08 Thread Stephen Boyd
Quoting Zhen Lei (2021-05-07 19:42:54) > Fix to return a negative error code from the error handling case instead > of 0, as done elsewhere in this function. > > Fixes: 070e64dc1bbc ("drm/msm/dpu: Convert to a chained irq chip") > Reported-by: Hulk Robot > Signed-off-by: Zhen Lei > --- >