[PATCH] drm/amdgpu: use helper macro HW_ERR instead of Hardware error string

[Zhang, Hawking]

[AMD Official Use Only - General]

Reviewed-by: Hawking Zhang <Hawking.Zhang at amd.com>

Regards,
Hawking
-----Original Message-----
From: Wang, Yang(Kevin) <KevinYang.Wang at amd.com>
Sent: Monday, January 29, 2024 17:10
To: amd-gfx at lists.freedesktop.org
Cc: Zhang, Hawking <Hawking.Zhang at amd.com>; Zhou1, Tao <Tao.Zhou1 at amd.com>; Wang, Yang(Kevin) <KevinYang.Wang at amd.com>
Subject: [PATCH] drm/amdgpu: use helper macro HW_ERR instead of Hardware error string

use helper macro HW_ERR to instead of Hardwareare error string.

Signed-off-by: Yang Wang <kevinyang.wang at amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c |  4 ++--  drivers/gpu/drm/amd/amdgpu/amdgpu_mca.c | 12 ++++++------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c
index d7d2ec3ce399..be525cf3a182 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_aca.c
@@ -120,10 +120,10 @@ static void aca_smu_bank_dump(struct amdgpu_device *adev, int idx, int total, st  {
        int i;

-       dev_info(adev->dev, "[Hardware error] Accelerator Check Architecture events logged\n");
+       dev_info(adev->dev, HW_ERR "Accelerator Check Architecture events
+logged\n");
        /* plus 1 for output format, e.g: ACA[08/08]: xxxx */
        for (i = 0; i < ARRAY_SIZE(aca_regs); i++)
-               dev_info(adev->dev, "[Hardware error] ACA[%02d/%02d].%s=0x%016llx\n",
+               dev_info(adev->dev, HW_ERR "ACA[%02d/%02d].%s=0x%016llx\n",
                         idx + 1, total, aca_regs[i].name, bank->regs[aca_regs[i].reg_idx]);  }

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_mca.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_mca.c
index 6452c09f22c6..24ad4b97177b 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_mca.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_mca.c
@@ -212,16 +212,16 @@ int amdgpu_mca_smu_set_debug_mode(struct amdgpu_device *adev, bool enable)

 static void amdgpu_mca_smu_mca_bank_dump(struct amdgpu_device *adev, int idx, struct mca_bank_entry *entry)  {
-       dev_info(adev->dev, "[Hardware error] Accelerator Check Architecture events logged\n");
-       dev_info(adev->dev, "[Hardware error] aca entry[%02d].STATUS=0x%016llx\n",
+       dev_info(adev->dev, HW_ERR "Accelerator Check Architecture events logged\n");
+       dev_info(adev->dev, HW_ERR "aca entry[%02d].STATUS=0x%016llx\n",
                 idx, entry->regs[MCA_REG_IDX_STATUS]);
-       dev_info(adev->dev, "[Hardware error] aca entry[%02d].ADDR=0x%016llx\n",
+       dev_info(adev->dev, HW_ERR "aca entry[%02d].ADDR=0x%016llx\n",
                 idx, entry->regs[MCA_REG_IDX_ADDR]);
-       dev_info(adev->dev, "[Hardware error] aca entry[%02d].MISC0=0x%016llx\n",
+       dev_info(adev->dev, HW_ERR "aca entry[%02d].MISC0=0x%016llx\n",
                 idx, entry->regs[MCA_REG_IDX_MISC0]);
-       dev_info(adev->dev, "[Hardware error] aca entry[%02d].IPID=0x%016llx\n",
+       dev_info(adev->dev, HW_ERR "aca entry[%02d].IPID=0x%016llx\n",
                 idx, entry->regs[MCA_REG_IDX_IPID]);
-       dev_info(adev->dev, "[Hardware error] aca entry[%02d].SYND=0x%016llx\n",
+       dev_info(adev->dev, HW_ERR "aca entry[%02d].SYND=0x%016llx\n",
                 idx, entry->regs[MCA_REG_IDX_SYND]);  }

--
2.34.1