[PATCH v2 1/3] drm/amdgpu: remove superfluous check

Zhang, Jerry (Junwei) Jerry.Zhang at amd.com
Wed Jul 5 05:56:28 UTC 2017


the series is
Reviewed-by: Junwei Zhang <Jerry.Zhang at amd.com>

Jerry

On 07/05/2017 01:32 PM, Huang Rui wrote:
> Signed-off-by: Huang Rui <ray.huang at amd.com>
> ---
>
> Changes from V1 -> V2
> - amdgpu_bo_free_kernel is safe even the bo is NULL, so remove the "if" check.
>
> Thanks,
> Ray
>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 15 +++++----------
>   drivers/gpu/drm/amd/amdgpu/psp_v3_1.c   |  8 ++++----
>   2 files changed, 9 insertions(+), 14 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
> index 79db294..d0104c7 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
> @@ -435,16 +435,11 @@ static int psp_hw_fini(void *handle)
>
>   	psp_ring_destroy(psp, PSP_RING_TYPE__KM);
>
> -	if (psp->tmr_buf)
> -		amdgpu_bo_free_kernel(&psp->tmr_bo, &psp->tmr_mc_addr, &psp->tmr_buf);
> -
> -	if (psp->fw_pri_buf)
> -		amdgpu_bo_free_kernel(&psp->fw_pri_bo,
> -				      &psp->fw_pri_mc_addr, &psp->fw_pri_buf);
> -
> -	if (psp->fence_buf_bo)
> -		amdgpu_bo_free_kernel(&psp->fence_buf_bo,
> -				      &psp->fence_buf_mc_addr, &psp->fence_buf);
> +	amdgpu_bo_free_kernel(&psp->tmr_bo, &psp->tmr_mc_addr, &psp->tmr_buf);
> +	amdgpu_bo_free_kernel(&psp->fw_pri_bo,
> +			      &psp->fw_pri_mc_addr, &psp->fw_pri_buf);
> +	amdgpu_bo_free_kernel(&psp->fence_buf_bo,
> +			      &psp->fence_buf_mc_addr, &psp->fence_buf);
>
>   	kfree(psp->cmd);
>   	psp->cmd = NULL;
> diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c b/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
> index 988ebd9..2718e86 100644
> --- a/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
> +++ b/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
> @@ -341,10 +341,10 @@ 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)
> -		amdgpu_bo_free_kernel(&adev->firmware.rbuf,
> -				      &ring->ring_mem_mc_addr,
> -				      (void **)&ring->ring_mem);
> +	amdgpu_bo_free_kernel(&adev->firmware.rbuf,
> +			      &ring->ring_mem_mc_addr,
> +			      (void **)&ring->ring_mem);
> +
>   	return ret;
>   }
>
>


More information about the amd-gfx mailing list