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