[AMD Official Use Only - General]
This patch is,
Reviewed-by: Tim Huang <tim.hu...@amd.com>
> -----Original Message-----
> From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of Jesse
> Zhang
> Sent: Friday, May 10, 2024 10:51 AM
> To: amd-gfx@lists.freedesktop.org
> Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Koenig, Christian
> <christian.koe...@amd.com>; Huang, Tim <tim.hu...@amd.com>; Zhang,
> Jesse(Jie) <jesse.zh...@amd.com>; Zhang, Jesse(Jie)
> <jesse.zh...@amd.com>
> Subject: [PATCH 16/22] drm/amd/pm: fix enum type compared against 0
>
> This less-than-zero comparison of an unsigned value is never true. type < 0U
>
> Signed-off-by: Jesse Zhang <jesse.zh...@amd.com>
> ---
> drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c
> b/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c
> index d439b95bfb79..602aa6941231 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c
> +++ b/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c
> @@ -56,7 +56,7 @@ static const char * const __smu_message_names[] =
> { static const char *smu_get_message_name(struct smu_context *smu,
> enum smu_message_type type)
> {
> - if (type < 0 || type >= SMU_MSG_MAX_COUNT)
> + if (type >= SMU_MSG_MAX_COUNT)
> return "unknown smu message";
>
> return __smu_message_names[type];
> --
> 2.25.1
