EDAC, AMD: carve out MCi_STATUS decoding

The MCi_STATUS registers have most field definitions in common so decode
them in the general path. Do not pass ecc_type along and compute it in
__amd64_decode_bus_error instead.

Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>

1 files changed, 4 insertions, 4 deletions

diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c
index 2080b1e2e8a2..c81ca2cf8dc7 100644
--- a/drivers/edac/amd64_edac.c
+++ b/drivers/edac/amd64_edac.c
@@ -2283,10 +2283,11 @@ static void amd64_handle_ue(struct mem_ctl_info *mci,
 }
 
 static inline void __amd64_decode_bus_error(struct mem_ctl_info *mci,
-                                            struct err_regs *info, int ecc_type)
+                                            struct err_regs *info)
 {
         u32 ec  = ERROR_CODE(info->nbsl);
         u32 xec = EXT_ERROR_CODE(info->nbsl);
+        int ecc_type = info->nbsh & (0x3 << 13);
 
         pr_emerg(" Transaction type: %s(%s), %s, Cache Level: %s, %s\n",
                  RRRR_MSG(ec), II_MSG(ec), TO_MSG(ec), LL_MSG(ec), PP_MSG(ec));
@@ -2316,12 +2317,11 @@ static inline void __amd64_decode_bus_error(struct mem_ctl_info *mci,
                 edac_mc_handle_ce_no_info(mci, EDAC_MOD_STR "Error Overflow");
 }
 
-void amd64_decode_bus_error(int node_id, struct err_regs *regs,
-                                   int ecc_type)
+void amd64_decode_bus_error(int node_id, struct err_regs *regs)
 {
         struct mem_ctl_info *mci = mci_lookup[node_id];
 
-        __amd64_decode_bus_error(mci, regs, ecc_type);
+        __amd64_decode_bus_error(mci, regs);
 
         /*
          * Check the UE bit of the NB status high register, if set generate some