[PATCH Review 1/1] drm/amdgpu/pm: adjust EccInfo_t struct

[Zhang, Hawking]

[AMD Official Use Only - General]

For the structure itself, the change is okay to me. But you'll have to apply pmfw version check in the implementation to make data matches with fw structure

The patch is

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

Regards,
Hawking

-----Original Message-----
From: Stanley.Yang <Stanley.Yang at amd.com>
Sent: Thursday, June 16, 2022 11:22
To: amd-gfx at lists.freedesktop.org; Zhang, Hawking <Hawking.Zhang at amd.com>; Zhou1, Tao <Tao.Zhou1 at amd.com>; Li, Candice <Candice.Li at amd.com>; Quan, Evan <Evan.Quan at amd.com>
Cc: Yang, Stanley <Stanley.Yang at amd.com>
Subject: [PATCH Review 1/1] drm/amdgpu/pm: adjust EccInfo_t struct

The EccInfo_t struct in driver_if.h is as below in official release verion 68.55.0  typedef struct {
   uint64_t mca_umc_status;
   uint64_t mca_umc_addr;

   uint16_t ce_count_lo_chip;
   uint16_t ce_count_hi_chip;

   uint32_t eccPadding;

   uint64_t mca_ceumc_addr;
 } EccInfo_t;
It's different from the debug version druing develop print correctable error address, so adjust EccInfo_t struct.

Signed-off-by: Stanley.Yang <Stanley.Yang at amd.com>
---
 .../drm/amd/pm/swsmu/inc/pmfw_if/smu13_driver_if_aldebaran.h   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu13_driver_if_aldebaran.h b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu13_driver_if_aldebaran.h
index 6f92038470ec..7a6075daa7b2 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu13_driver_if_aldebaran.h
+++ b/drivers/gpu/drm/amd/pm/swsmu/inc/pmfw_if/smu13_driver_if_aldebaran
+++ .h
@@ -521,12 +521,13 @@ typedef struct {
 typedef struct {
        uint64_t mca_umc_status;
        uint64_t mca_umc_addr;
-       uint64_t mca_ceumc_addr;

        uint16_t ce_count_lo_chip;
        uint16_t ce_count_hi_chip;

        uint32_t eccPadding;
+
+       uint64_t mca_ceumc_addr;
 } EccInfo_V2_t;

 typedef struct {
--
2.17.1