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

[Sibi Sankar <sibis-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>]

Tested-by: Sibi Sankar <sibis@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@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.
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno