[PATCH] drm/amdgpu: Use str_on_off() helper

[Deucher, Alexander]

[Public]

> -----Original Message-----
> From: long.yunjian at zte.com.cn <long.yunjian at zte.com.cn>
> Sent: Friday, June 20, 2025 11:26 PM
> To: Deucher, Alexander <Alexander.Deucher at amd.com>
> Cc: Koenig, Christian <Christian.Koenig at amd.com>; airlied at gmail.com;
> simona at ffwll.ch; Zhang, Hawking <Hawking.Zhang at amd.com>; Wang,
> Yang(Kevin) <KevinYang.Wang at amd.com>; Liu, Xiang(Dean)
> <Xiang.Liu at amd.com>; Zhou1, Tao <Tao.Zhou1 at amd.com>; yanzhen at vivo.com;
> fang.yumeng at zte.com.cn; amd-gfx at lists.freedesktop.org; dri-
> devel at lists.freedesktop.org; linux-kernel at vger.kernel.org; xu.lifeng1 at zte.com.cn;
> ouyang.maochun at zte.com.cn; mou.yi at zte.com.cn
> Subject: [PATCH] drm/amdgpu: Use str_on_off() helper
>
> From: Yumeng Fang <fang.yumeng at zte.com.cn>
>
> Remove hard-coded strings by using the str_on_off() helper.

I personally find that these helpers hurt readability.

Alex

>
> Signed-off-by: Yumeng Fang <fang.yumeng at zte.com.cn>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c | 3 ++-
> drivers/gpu/drm/amd/amdgpu/amdgpu_mca.c | 3 ++-
>  2 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c
> index cbc40cad581b..c3c88d8e1ce2 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c
> @@ -22,6 +22,7 @@
>   */
>
>  #include <linux/list.h>
> +#include <linux/string_choices.h>
>  #include "amdgpu.h"
>  #include "amdgpu_aca.h"
>  #include "amdgpu_ras.h"
> @@ -873,7 +874,7 @@ static int amdgpu_aca_smu_debug_mode_set(void *data,
> u64 val)
>       if (ret)
>               return ret;
>
> -     dev_info(adev->dev, "amdgpu set smu aca debug mode %s success\n", val
> ? "on" : "off");
> +     dev_info(adev->dev, "amdgpu set smu aca debug mode %s success\n",
> +str_on_off(val));
>
>       return 0;
>  }
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_mca.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_mca.c
> index 3ca03b5e0f91..81dcc7ebac3b 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_mca.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_mca.c
> @@ -20,6 +20,7 @@
>   * OTHER DEALINGS IN THE SOFTWARE.
>   *
>   */
> +#include <linux/string_choices.h>
>  #include "amdgpu_ras.h"
>  #include "amdgpu.h"
>  #include "amdgpu_mca.h"
> @@ -519,7 +520,7 @@ static int amdgpu_mca_smu_debug_mode_set(void *data,
> u64 val)
>       if (ret)
>               return ret;
>
> -     dev_info(adev->dev, "amdgpu set smu mca debug mode %s success\n", val
> ? "on" : "off");
> +     dev_info(adev->dev, "amdgpu set smu mca debug mode %s success\n",
> +str_on_off(val));
>
>       return 0;
>  }
> --
> 2.25.1