RE: [PATCH 12/20] x86, edac: use Intel family name macros for edac driver
From: Luck, Tony
Date: Thu Jun 02 2016 - 13:27:21 EST
> This needs to add an entry for BDX-DE (use the same table initializer). Probably as
> a separate patch before/after this.
Oops ... a bit worse than that. I assumed that index into the array matches the
enum ... (with a comment!) ... having two entries for the same "type" would break
that. I'll have to add a type field to something.
If your patch is on a fast-track, then go ahead with it:
Acked-by: Tony Luck <tony.luck@xxxxxxxxx>
I'll work on fixing the other mess independently
-Tony