On 14/05/2024 18:04, Marc Gonzalez wrote:
Right now, GPU init fails:
[ 2.756363] [drm:adreno_bind] Found GPU: 5.4.0.1
[ 2.767183] [drm:a5xx_gpu_init]
[ 2.767422] [drm:adreno_gpu_init] fast_rate=710000097, slow_rate=27000000
[ 3.003869] [drm:msm_gpu_init] ebi1_clk: fffffffffffffffe
[ 3.004002] adreno 5000000.gpu: supply vdd not found, using dummy regulator
[ 3.008463] [drm:msm_gpu_init] gpu_reg: ffff0000819e4000
[ 3.015105] adreno 5000000.gpu: supply vddcx not found, using dummy regulator
[ 3.020702] [drm:msm_gpu_init] gpu_cx: ffff0000819e4180
[ 3.028173] [drm:adreno_iommu_create_address_space]
[ 3.054552] [drm:msm_gpu_init] gpu->aspace=ffffffffffffffed
[ 3.058112] [drm:a5xx_destroy] 5.4.0.1
[ 3.065922] [drm:msm_gpu_cleanup] 5.4.0.1
[ 3.074237] msm_dpu c901000.display-controller: failed to load adreno gpu
[ 3.082412] msm_dpu c901000.display-controller: failed to bind 5000000.gpu (ops a3xx_ops): -19
[ 3.088342] msm_dpu c901000.display-controller: [drm:drm_managed_release] drmres release begin
...
[ 3.197694] [drm:drm_managed_release] drmres release end
[ 3.204009] msm_dpu c901000.display-controller: adev bind failed: -19
I agree with Marjin the log of the failure isn't required.
How about this as a commit log ?
"The GPU currently fails to initialise because the adreno_smmu node is
missing. We require the adreno_smmu node to translate device virtual
addresses properly.
Enable it now."
[ 3.220381] msm_dpu c901000.display-controller: bound 5000000.gpu (ops a3xx_ops)
[ 3.235503] [drm:dpu_kms_hw_init:1053] dpu hardware revision:0x30000000
I'd drop that, we know the fix works.
Fixes: 87cd46d68aeac8 ("Configure Adreno GPU and related IOMMU")
Retain this though, its a legitimate fix the GPU won't work without this
change.
Then add
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>
---
bod