[Freedreno] [PATCH AUTOSEL 4.9 05/13] drm: msm: Fix return type of dsi_mgr_connector_mode_valid for kCFI

2020-03-02 Thread Sasha Levin
From: John Stultz [ Upstream commit 7fd2dfc3694922eb7ace4801b7208cf9f62ebc7d ] I was hitting kCFI crashes when building with clang, and after some digging finally narrowed it down to the dsi_mgr_connector_mode_valid() function being implemented as returning an int, instead of an enum

[Freedreno] [PATCH AUTOSEL 4.9 06/13] drm/msm/dsi: save pll state before dsi host is powered off

2020-03-02 Thread Sasha Levin
From: Harigovindan P [ Upstream commit a1028dcfd0dd97884072288d0c8ed7f30399b528 ] Save pll state before dsi host is powered off. Without this change some register values gets resetted. Signed-off-by: Harigovindan P Signed-off-by: Rob Clark Signed-off-by: Sasha Levin ---

[Freedreno] [PATCH AUTOSEL 4.4 05/11] drm/msm/dsi: save pll state before dsi host is powered off

2020-03-02 Thread Sasha Levin
From: Harigovindan P [ Upstream commit a1028dcfd0dd97884072288d0c8ed7f30399b528 ] Save pll state before dsi host is powered off. Without this change some register values gets resetted. Signed-off-by: Harigovindan P Signed-off-by: Rob Clark Signed-off-by: Sasha Levin ---

[Freedreno] [PATCH AUTOSEL 4.4 04/11] drm: msm: Fix return type of dsi_mgr_connector_mode_valid for kCFI

2020-03-02 Thread Sasha Levin
From: John Stultz [ Upstream commit 7fd2dfc3694922eb7ace4801b7208cf9f62ebc7d ] I was hitting kCFI crashes when building with clang, and after some digging finally narrowed it down to the dsi_mgr_connector_mode_valid() function being implemented as returning an int, instead of an enum

[Freedreno] [PATCH AUTOSEL 4.19 11/32] drm/msm/dsi: save pll state before dsi host is powered off

2020-03-02 Thread Sasha Levin
From: Harigovindan P [ Upstream commit a1028dcfd0dd97884072288d0c8ed7f30399b528 ] Save pll state before dsi host is powered off. Without this change some register values gets resetted. Signed-off-by: Harigovindan P Signed-off-by: Rob Clark Signed-off-by: Sasha Levin ---

[Freedreno] [PATCH AUTOSEL 4.14 08/22] drm: msm: Fix return type of dsi_mgr_connector_mode_valid for kCFI

2020-03-02 Thread Sasha Levin
From: John Stultz [ Upstream commit 7fd2dfc3694922eb7ace4801b7208cf9f62ebc7d ] I was hitting kCFI crashes when building with clang, and after some digging finally narrowed it down to the dsi_mgr_connector_mode_valid() function being implemented as returning an int, instead of an enum

[Freedreno] [PATCH AUTOSEL 4.14 09/22] drm/msm/dsi: save pll state before dsi host is powered off

2020-03-02 Thread Sasha Levin
From: Harigovindan P [ Upstream commit a1028dcfd0dd97884072288d0c8ed7f30399b528 ] Save pll state before dsi host is powered off. Without this change some register values gets resetted. Signed-off-by: Harigovindan P Signed-off-by: Rob Clark Signed-off-by: Sasha Levin ---

[Freedreno] [PATCH AUTOSEL 4.14 07/22] drm/msm/mdp5: rate limit pp done timeout warnings

2020-03-02 Thread Sasha Levin
From: Brian Masney [ Upstream commit ef8c9809acb0805c991bba8bdd4749fc46d44a98 ] Add rate limiting of the 'pp done time out' warnings since these warnings can quickly fill the dmesg buffer. Signed-off-by: Brian Masney Signed-off-by: Rob Clark Signed-off-by: Sasha Levin ---

[Freedreno] [PATCH AUTOSEL 4.19 12/32] drm/msm/dsi/pll: call vco set rate explicitly

2020-03-02 Thread Sasha Levin
From: Harigovindan P [ Upstream commit c6659785dfb3f8d75f1fe637e4222ff8178f5280 ] For a given byte clock, if VCO recalc value is exactly same as vco set rate value, vco_set_rate does not get called assuming VCO is already set to required value. But Due to GDSC toggle, VCO values are erased in

[Freedreno] [PATCH AUTOSEL 4.19 09/32] drm: msm: Fix return type of dsi_mgr_connector_mode_valid for kCFI

2020-03-02 Thread Sasha Levin
From: John Stultz [ Upstream commit 7fd2dfc3694922eb7ace4801b7208cf9f62ebc7d ] I was hitting kCFI crashes when building with clang, and after some digging finally narrowed it down to the dsi_mgr_connector_mode_valid() function being implemented as returning an int, instead of an enum

[Freedreno] [PATCH AUTOSEL 4.19 08/32] drm/msm/mdp5: rate limit pp done timeout warnings

2020-03-02 Thread Sasha Levin
From: Brian Masney [ Upstream commit ef8c9809acb0805c991bba8bdd4749fc46d44a98 ] Add rate limiting of the 'pp done time out' warnings since these warnings can quickly fill the dmesg buffer. Signed-off-by: Brian Masney Signed-off-by: Rob Clark Signed-off-by: Sasha Levin ---

[Freedreno] [PATCH AUTOSEL 5.4 16/58] drm/msm/dsi: save pll state before dsi host is powered off

2020-03-02 Thread Sasha Levin
From: Harigovindan P [ Upstream commit a1028dcfd0dd97884072288d0c8ed7f30399b528 ] Save pll state before dsi host is powered off. Without this change some register values gets resetted. Signed-off-by: Harigovindan P Signed-off-by: Rob Clark Signed-off-by: Sasha Levin ---

[Freedreno] [PATCH AUTOSEL 5.4 11/58] drm/msm/mdp5: rate limit pp done timeout warnings

2020-03-02 Thread Sasha Levin
From: Brian Masney [ Upstream commit ef8c9809acb0805c991bba8bdd4749fc46d44a98 ] Add rate limiting of the 'pp done time out' warnings since these warnings can quickly fill the dmesg buffer. Signed-off-by: Brian Masney Signed-off-by: Rob Clark Signed-off-by: Sasha Levin ---

[Freedreno] [PATCH AUTOSEL 5.4 17/58] drm/msm/dsi/pll: call vco set rate explicitly

2020-03-02 Thread Sasha Levin
From: Harigovindan P [ Upstream commit c6659785dfb3f8d75f1fe637e4222ff8178f5280 ] For a given byte clock, if VCO recalc value is exactly same as vco set rate value, vco_set_rate does not get called assuming VCO is already set to required value. But Due to GDSC toggle, VCO values are erased in

[Freedreno] [PATCH AUTOSEL 5.4 12/58] drm: msm: Fix return type of dsi_mgr_connector_mode_valid for kCFI

2020-03-02 Thread Sasha Levin
From: John Stultz [ Upstream commit 7fd2dfc3694922eb7ace4801b7208cf9f62ebc7d ] I was hitting kCFI crashes when building with clang, and after some digging finally narrowed it down to the dsi_mgr_connector_mode_valid() function being implemented as returning an int, instead of an enum

[Freedreno] [PATCH AUTOSEL 5.5 12/66] drm/msm/mdp5: rate limit pp done timeout warnings

2020-03-02 Thread Sasha Levin
From: Brian Masney [ Upstream commit ef8c9809acb0805c991bba8bdd4749fc46d44a98 ] Add rate limiting of the 'pp done time out' warnings since these warnings can quickly fill the dmesg buffer. Signed-off-by: Brian Masney Signed-off-by: Rob Clark Signed-off-by: Sasha Levin ---

[Freedreno] [PATCH AUTOSEL 5.5 13/66] drm: msm: Fix return type of dsi_mgr_connector_mode_valid for kCFI

2020-03-02 Thread Sasha Levin
From: John Stultz [ Upstream commit 7fd2dfc3694922eb7ace4801b7208cf9f62ebc7d ] I was hitting kCFI crashes when building with clang, and after some digging finally narrowed it down to the dsi_mgr_connector_mode_valid() function being implemented as returning an int, instead of an enum

[Freedreno] [PATCH AUTOSEL 5.5 18/66] drm/msm/dsi/pll: call vco set rate explicitly

2020-03-02 Thread Sasha Levin
From: Harigovindan P [ Upstream commit c6659785dfb3f8d75f1fe637e4222ff8178f5280 ] For a given byte clock, if VCO recalc value is exactly same as vco set rate value, vco_set_rate does not get called assuming VCO is already set to required value. But Due to GDSC toggle, VCO values are erased in

[Freedreno] [PATCH AUTOSEL 5.5 17/66] drm/msm/dsi: save pll state before dsi host is powered off

2020-03-02 Thread Sasha Levin
From: Harigovindan P [ Upstream commit a1028dcfd0dd97884072288d0c8ed7f30399b528 ] Save pll state before dsi host is powered off. Without this change some register values gets resetted. Signed-off-by: Harigovindan P Signed-off-by: Rob Clark Signed-off-by: Sasha Levin ---

Re: [Freedreno] [PATCH v4 1/2] dt-bindings: display: msm: Convert GMU bindings to YAML

2020-03-02 Thread Rob Herring
On Mon, 2 Mar 2020 13:47:46 -0700, Jordan Crouse wrote: > Convert display/msm/gmu.txt to display/msm/gmu.yaml and remove the old > text bindings. > > Signed-off-by: Jordan Crouse > --- > > .../devicetree/bindings/display/msm/gmu.txt| 116 --- >

Re: [Freedreno] [PATCH v3 2/2] drm/msm/a6xx: Use the DMA API for GMU memory objects

2020-03-02 Thread Ruhl, Michael J
>-Original Message- >From: dri-devel On Behalf Of >Jordan Crouse >Sent: Monday, March 2, 2020 1:24 PM >To: linux-arm-...@vger.kernel.org >Cc: David Airlie ; freedreno@lists.freedesktop.org; >smase...@codeaurora.org; linux-ker...@vger.kernel.org; dri- >de...@lists.freedesktop.org; Douglas

Re: [Freedreno] [PATCH v4 2/2] drm/msm/a6xx: Use the DMA API for GMU memory objects

2020-03-02 Thread Ruhl, Michael J
>-Original Message- >From: dri-devel On Behalf Of >Jordan Crouse >Sent: Monday, March 2, 2020 3:48 PM >To: linux-arm-...@vger.kernel.org >Cc: Douglas Anderson ; David Airlie >; freedreno@lists.freedesktop.org; >smase...@codeaurora.org; linux-ker...@vger.kernel.org; dri-

Re: [Freedreno] [PATCH v4 02/22] drm: Add get_scanout_position() to struct drm_crtc_helper_funcs

2020-03-02 Thread Daniel Vetter
On Thu, Jan 23, 2020 at 2:59 PM Thomas Zimmermann wrote: > > The new callback get_scanout_position() reads the current location > of the scanout process. The operation is currently located in struct > drm_driver, but really belongs to the CRTC. Drivers will be converted > in separate patches. > >

Re: [Freedreno] [PATCH v3 1/2] dt-bindings: display: msm: Convert GMU bindings to YAML

2020-03-02 Thread Sam Ravnborg
Hi Jordan. On Mon, Mar 02, 2020 at 11:23:43AM -0700, Jordan Crouse wrote: > Convert display/msm/gmu.txt to display/msm/gmu.yaml and remove the old > text bindings. > > Signed-off-by: Jordan Crouse > --- > > .../devicetree/bindings/display/msm/gmu.txt| 116 --- > - >

[Freedreno] [PATCH v4 1/2] dt-bindings: display: msm: Convert GMU bindings to YAML

2020-03-02 Thread Jordan Crouse
Convert display/msm/gmu.txt to display/msm/gmu.yaml and remove the old text bindings. Signed-off-by: Jordan Crouse --- .../devicetree/bindings/display/msm/gmu.txt| 116 --- .../devicetree/bindings/display/msm/gmu.yaml | 123 + 2 files changed,

[Freedreno] [PATCH v4 0/2] msm/gpu/a6xx: use the DMA-API for GMU memory allocations

2020-03-02 Thread Jordan Crouse
When CONFIG_INIT_ON_ALLOC_DEFAULT_ON the GMU memory allocator runs afoul of cache coherency issues because it is mapped as write-combine without clearing the cache after it was zeroed. Rather than duplicate the hacky workaround we use in the GEM allocator for the same reason it turns out that we

[Freedreno] [PATCH v4 2/2] drm/msm/a6xx: Use the DMA API for GMU memory objects

2020-03-02 Thread Jordan Crouse
The GMU has very few memory allocations and uses a flat memory space so there is no good reason to go out of our way to bypass the DMA APIs which were basically designed for this exact scenario. v4: Use dma_alloc_wc() v3: Set the dma mask correctly and use dma_addr_t for the iova type v2: Pass

Re: [Freedreno] [PATCH v3 2/2] drm/msm/a6xx: Use the DMA API for GMU memory objects

2020-03-02 Thread Jordan Crouse
On Mon, Mar 02, 2020 at 06:56:47PM +, Ruhl, Michael J wrote: > >-Original Message- > >From: dri-devel On Behalf Of > >Jordan Crouse > >Sent: Monday, March 2, 2020 1:24 PM > >To: linux-arm-...@vger.kernel.org > >Cc: David Airlie ; freedreno@lists.freedesktop.org; >

[Freedreno] [PATCH v3 1/2] dt-bindings: display: msm: Convert GMU bindings to YAML

2020-03-02 Thread Jordan Crouse
Convert display/msm/gmu.txt to display/msm/gmu.yaml and remove the old text bindings. Signed-off-by: Jordan Crouse --- .../devicetree/bindings/display/msm/gmu.txt| 116 --- .../devicetree/bindings/display/msm/gmu.yaml | 123 + 2 files changed,

[Freedreno] [PATCH v3 0/2] msm/gpu/a6xx: use the DMA-API for GMU memory allocations

2020-03-02 Thread Jordan Crouse
When CONFIG_INIT_ON_ALLOC_DEFAULT_ON the GMU memory allocator runs afoul of cache coherency issues because it is mapped as write-combine without clearing the cache after it was zeroed. Rather than duplicate the hacky workaround we use in the GEM allocator for the same reason it turns out that we

Re: [Freedreno] [PATCH] drm: msm: a6x: Disable interrupts before recovery

2020-03-02 Thread Jordan Crouse
On Mon, Mar 02, 2020 at 02:17:16PM +0530, Sharat Masetty wrote: > This patch disables interrupts in the GPU RBBM hang detect fault handler > before going to recovery. Okay, but why? What were you seeing? A little bit of extra description would make it easier to understand this change in the

[Freedreno] [PATCH] drm: msm: a6x: Disable interrupts before recovery

2020-03-02 Thread Sharat Masetty
This patch disables interrupts in the GPU RBBM hang detect fault handler before going to recovery. Signed-off-by: Sharat Masetty --- drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu.c