[PATCH] drm/amd/display: Remove redundant braces in 'amdgpu_dm_crtc_notify_ta_to_read()'

Rodrigo Siqueira Jordao Rodrigo.Siqueira at amd.com
Thu Jun 22 15:05:15 UTC 2023



On 6/22/23 03:38, Srinivasan Shanmugam wrote:
> Adhere to Linux kernel coding style.
> 
> Reported by checkpatch:
> 
> WARNING: braces {} are not necessary for single statement blocks
> 
> Cc: Rodrigo Siqueira <Rodrigo.Siqueira at amd.com>
> Cc: Aurabindo Pillai <aurabindo.pillai at amd.com>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam at amd.com>
> ---
>   drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c | 6 ++----
>   1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c
> index 0802f8e8fac5..52ecfa746b54 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c
> @@ -123,9 +123,8 @@ static void amdgpu_dm_crtc_notify_ta_to_read(struct work_struct *work)
>   	secure_display_ctx = container_of(work, struct secure_display_context, notify_ta_work);
>   	crtc = secure_display_ctx->crtc;
>   
> -	if (!crtc) {
> +	if (!crtc)
>   		return;
> -	}
>   
>   	psp = &drm_to_adev(crtc->dev)->psp;
>   
> @@ -151,9 +150,8 @@ static void amdgpu_dm_crtc_notify_ta_to_read(struct work_struct *work)
>   	ret = psp_securedisplay_invoke(psp, TA_SECUREDISPLAY_COMMAND__SEND_ROI_CRC);
>   
>   	if (!ret) {
> -		if (securedisplay_cmd->status != TA_SECUREDISPLAY_STATUS__SUCCESS) {
> +		if (securedisplay_cmd->status != TA_SECUREDISPLAY_STATUS__SUCCESS)
>   			psp_securedisplay_parse_resp_status(psp, securedisplay_cmd->status);
> -		}
>   	}
>   
>   	mutex_unlock(&psp->securedisplay_context.mutex);

Reviewed-by: Rodrigo Siqueira <Rodrigo.Siqueira at amd.com>


More information about the amd-gfx mailing list