[PATCH] drm/amdgpu: remove redundant semicolons in RAS_EVENT_LOG

Christian König ckoenig.leichtzumerken at gmail.com
Thu Jul 4 13:03:59 UTC 2024


Am 04.07.24 um 07:52 schrieb Yang Wang:
> remove redundant semicolons in RAS_EVENT_LOG to avoid
> code format check warning.
>
> Fixes: 951c09c88fca ("drm/amdgpu: fix compiler 'side-effect' check issue for RAS_EVENT_LOG()")
>
> Signed-off-by: Yang Wang <kevinyang.wang at amd.com>

Ah, yes one of the most common mistakes of all times :)

Reviewed-by: Christian König <christian.koenig at amd.com>

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h
> index 9224fc6418e4..518b10f190ec 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h
> @@ -72,7 +72,7 @@ struct amdgpu_iv_entry;
>   #define RAS_EVENT_ID_IS_VALID(x)	(!((x) & BIT_ULL(63)))
>   
>   #define RAS_EVENT_LOG(adev, id, fmt, ...)	\
> -	amdgpu_ras_event_log_print((adev), (id), (fmt), ##__VA_ARGS__);
> +	amdgpu_ras_event_log_print((adev), (id), (fmt), ##__VA_ARGS__)
>   
>   #define amdgpu_ras_mark_ras_event(adev, type)	\
>   	(amdgpu_ras_mark_ras_event_caller((adev), (type), __builtin_return_address(0)))



More information about the amd-gfx mailing list