[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