[Freedreno] [PATCH] drm/msm/dp: remove duplicate include statement

2020-11-10 Thread Tian Tao
linux/rational.h is included more than once, Remove the one that isn't necessary. Signed-off-by: Tian Tao --- drivers/gpu/drm/msm/dp/dp_catalog.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/msm/dp/dp_catalog.c b/drivers/gpu/drm/msm/dp/dp_catalog.c index b15b4ce..105fa65

Re: [Freedreno] [PATCHv7 2/7] iommu/arm-smmu: Add domain attribute for system cache

2020-11-10 Thread Sai Prakash Ranjan
On 2020-11-10 17:48, Will Deacon wrote: On Fri, Oct 30, 2020 at 02:53:09PM +0530, Sai Prakash Ranjan wrote: Add iommu domain attribute for using system cache aka last level cache by client drivers like GPU to set right attributes for caching the hardware pagetables into the system cache.

Re: [Freedreno] [PATCHv7 1/7] iommu/io-pgtable-arm: Add support to use system cache

2020-11-10 Thread Sai Prakash Ranjan
On 2020-11-10 17:48, Will Deacon wrote: On Fri, Oct 30, 2020 at 02:53:08PM +0530, Sai Prakash Ranjan wrote: Add a quirk IO_PGTABLE_QUIRK_SYS_CACHE to override the attributes set in TCR for the page table walker when using system cache. Signed-off-by: Sai Prakash Ranjan ---

Re: [Freedreno] [PATCH v5 2/3] arm64: dts: qcom: sc7180: Add gpu cooling support

2020-11-10 Thread Bjorn Andersson
On Fri 30 Oct 05:47 CDT 2020, Akhil P Oommen wrote: > Add cooling-cells property and the cooling maps for the gpu tzones > to support GPU cooling. > > Signed-off-by: Akhil P Oommen > Reviewed-by: Matthias Kaehlcke > --- > arch/arm64/boot/dts/qcom/sc7180.dtsi | 30

Re: [Freedreno] [PATCH 2/2] drm/msm/a5xx: Clear shadow on suspend

2020-11-10 Thread Jordan Crouse
On Tue, Nov 10, 2020 at 10:44:00AM -0800, Rob Clark wrote: > From: Rob Clark > > Similar to the previous patch, clear shadow on suspend to avoid timeouts > waiting for ringbuffer space. > > Fixes: 8907afb476ac ("drm/msm: Allow a5xx to mark the RPTR shadow as > privileged") > Signed-off-by: Rob

Re: [Freedreno] [PATCH 1/2] drm/msm/a6xx: Clear shadow on suspend

2020-11-10 Thread Jordan Crouse
On Tue, Nov 10, 2020 at 10:43:59AM -0800, Rob Clark wrote: > From: Rob Clark > > Clear the shadow rptr on suspend. Otherwise, when we resume, we can > have a stale value until CP_WHERE_AM_I executes. If we suspend near > the ringbuffer wraparound point, this can lead to a chicken/egg >

[Freedreno] [PATCH 1/2] drm/msm/a6xx: Clear shadow on suspend

2020-11-10 Thread Rob Clark
From: Rob Clark Clear the shadow rptr on suspend. Otherwise, when we resume, we can have a stale value until CP_WHERE_AM_I executes. If we suspend near the ringbuffer wraparound point, this can lead to a chicken/egg situation where we are waiting for ringbuffer space to write the CP_WHERE_AM_I

[Freedreno] [PATCH 2/2] drm/msm/a5xx: Clear shadow on suspend

2020-11-10 Thread Rob Clark
From: Rob Clark Similar to the previous patch, clear shadow on suspend to avoid timeouts waiting for ringbuffer space. Fixes: 8907afb476ac ("drm/msm: Allow a5xx to mark the RPTR shadow as privileged") Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/adreno/a5xx_gpu.c | 12 +++- 1

Re: [Freedreno] [PATCHv7 1/7] iommu/io-pgtable-arm: Add support to use system cache

2020-11-10 Thread Will Deacon
On Fri, Oct 30, 2020 at 02:53:08PM +0530, Sai Prakash Ranjan wrote: > Add a quirk IO_PGTABLE_QUIRK_SYS_CACHE to override the > attributes set in TCR for the page table walker when > using system cache. > > Signed-off-by: Sai Prakash Ranjan > --- > drivers/iommu/io-pgtable-arm.c | 7 ++- >

Re: [Freedreno] [PATCHv7 2/7] iommu/arm-smmu: Add domain attribute for system cache

2020-11-10 Thread Will Deacon
On Fri, Oct 30, 2020 at 02:53:09PM +0530, Sai Prakash Ranjan wrote: > Add iommu domain attribute for using system cache aka last level > cache by client drivers like GPU to set right attributes for caching > the hardware pagetables into the system cache. > > Signed-off-by: Sai Prakash Ranjan >

Re: [Freedreno] [PATCHv7 7/7] iommu: arm-smmu-impl: Add a space before open parenthesis

2020-11-10 Thread Will Deacon
On Fri, Oct 30, 2020 at 02:53:14PM +0530, Sai Prakash Ranjan wrote: > Fix the checkpatch warning for space required before the open > parenthesis. > > Signed-off-by: Sai Prakash Ranjan > --- > drivers/iommu/arm/arm-smmu/arm-smmu-impl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

Re: [Freedreno] [PATCHv7 6/7] iommu: arm-smmu-impl: Use table to list QCOM implementations

2020-11-10 Thread Will Deacon
On Fri, Oct 30, 2020 at 02:53:13PM +0530, Sai Prakash Ranjan wrote: > Use table and of_match_node() to match qcom implementation > instead of multiple of_device_compatible() calls for each > QCOM SMMU implementation. > > Signed-off-by: Sai Prakash Ranjan > --- >

Re: [Freedreno] [PATCH 0/2] drm: Build fixes for msm and mediatek

2020-11-10 Thread Daniel Vetter
On Mon, Nov 09, 2020 at 11:32:40AM +0100, Thomas Zimmermann wrote: > Commit 49a3f51dfeee ("drm/gem: Use struct dma_buf_map in GEM vmap ops and > convert GEM backends") changed DRM's internal GEM vmap callbacks. Msm and > mediatek were forgotten during the conversion. > > Thomas Zimmermann (2): >

[Freedreno] [Resend][PATCH] drm/msm: delete conversion from bool value to bool return

2020-11-10 Thread Bernard
Delete the conversion from bool value to bool function return. Signed-off-by: Bernard Zhao --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c

[Freedreno] [Resend][PATCH] drm/msm: delete conversion from bool value to bool variable

2020-11-10 Thread Bernard
Remove the conversion from bool value to bool variable. Signed-off-by: Bernard Zhao --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c