[Freedreno] [PATCH] msm/gpu/a6xx: Force of_dma_configure to setup DMA for GMU

Sibi Sankar sibis at codeaurora.org
Fri Sep 14 16:18:05 UTC 2018


Tested-by: Sibi Sankar <sibis at codeaurora.org>

On 2018-09-14 20:33, Jordan Crouse wrote:
> The point of the 'force_dma' parameter for of_dma_configure
> is to force the device to be set up even if DMA capability is
> not described by the firmware which is exactly the use case
>  we have for GMU - we need SMMU to get set up but we have no
> other dma capabilities since memory is managed by the GPU
> driver. Currently we pass false so of_dma_configure() fails
> and subsequently GMU and GPU probe does as well.
> 
> Fixes: 4b565ca5a2c ("drm/msm: Add A6XX device support")
> Signed-off-by: Jordan Crouse <jcrouse at codeaurora.org>
> ---
>  drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gmu.c
> b/drivers/gpu/drm/msm/adreno/a6xx_gmu.c
> index bbb8126ec5c5..cbaca366fc20 100644
> --- a/drivers/gpu/drm/msm/adreno/a6xx_gmu.c
> +++ b/drivers/gpu/drm/msm/adreno/a6xx_gmu.c
> @@ -1140,7 +1140,7 @@ int a6xx_gmu_probe(struct a6xx_gpu *a6xx_gpu,
> struct device_node *node)
> 
>  	gmu->dev = &pdev->dev;
> 
> -	of_dma_configure(gmu->dev, node, false);
> +	of_dma_configure(gmu->dev, node, true);
> 
>  	/* Fow now, don't do anything fancy until we get our feet under us */
>  	gmu->idle_level = GMU_IDLE_STATE_ACTIVE;

-- 
-- Sibi Sankar --
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project.


More information about the Freedreno mailing list