On Wed, Aug 19, 2020 at 09:35:44AM -0500, Alex Kluver wrote: > Updates to the UEFI 2.8 Memory Error Record allow splitting the bank field > into bank address and bank group, and using the last 3 bits of the extended > field as a chip identifier. > > When needed, print correct version of bank field, bank group, and chip > identification > > Based on UEFI 2.8 Table 299. Memory Error Record Whoever commits this - those last two sentences need fullstops. > Reviewed-by: Steve Wahl <steve.wahl@xxxxxxx> > Reviewed-by: Kyle Meyer <kyle.meyer@xxxxxxx> > Reviewed-by: Russ Anderson <russ.anderson@xxxxxxx> > Signed-off-by: Alex Kluver <alex.kluver@xxxxxxx> > --- > > v1 -> v2: > * Add static inline cper_get_mem_extension() to make it > more readable, as suggested by Borislav Petkov. > > * Add second patch for bank field, bank group, and chip id. > > --- > drivers/edac/ghes_edac.c | 9 +++++++++ > drivers/firmware/efi/cper.c | 9 +++++++++ > include/linux/cper.h | 8 ++++++++ > 3 files changed, 26 insertions(+) For the EDAC bits: Acked-by: Borislav Petkov <bp@xxxxxxx> Thx. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette