[PATCH 1/3] drm/amdgpu: move check from cpu address to bo

Christian König christian.koenig at amd.com
Tue Jul 4 07:19:17 UTC 2017


Actually amdgpu_bo_free_kernel is NULL save, so you should rather remove 
the "if"s here altogether.

Christian.

Am 04.07.2017 um 08:10 schrieb Huang Rui:
> Signed-off-by: Huang Rui <ray.huang at amd.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 4 ++--
>   drivers/gpu/drm/amd/amdgpu/psp_v3_1.c   | 2 +-
>   2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
> index 79db294..1469e6d 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
> @@ -435,10 +435,10 @@ static int psp_hw_fini(void *handle)
>   
>   	psp_ring_destroy(psp, PSP_RING_TYPE__KM);
>   
> -	if (psp->tmr_buf)
> +	if (psp->tmr_bo)
>   		amdgpu_bo_free_kernel(&psp->tmr_bo, &psp->tmr_mc_addr, &psp->tmr_buf);
>   
> -	if (psp->fw_pri_buf)
> +	if (psp->fw_pri_bo)
>   		amdgpu_bo_free_kernel(&psp->fw_pri_bo,
>   				      &psp->fw_pri_mc_addr, &psp->fw_pri_buf);
>   
> diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c b/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
> index 988ebd9..3d057d8 100644
> --- a/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
> +++ b/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
> @@ -341,7 +341,7 @@ int psp_v3_1_ring_destroy(struct psp_context *psp, enum psp_ring_type ring_type)
>   	ret = psp_wait_for(psp, SOC15_REG_OFFSET(MP0, 0, mmMP0_SMN_C2PMSG_64),
>   			   0x80000000, 0x80000000, false);
>   
> -	if (ring->ring_mem)
> +	if (adev->firmware.rbuf)
>   		amdgpu_bo_free_kernel(&adev->firmware.rbuf,
>   				      &ring->ring_mem_mc_addr,
>   				      (void **)&ring->ring_mem);




More information about the amd-gfx mailing list