RE: [PATCH 12/20] x86, edac: use Intel family name macros for edac driver

From	"Luck, Tony" <tony.luck@intel.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH 12/20] x86, edac: use Intel family name macros for edac driver
Date	2016-06-02 19:30 +0200
Message-ID	<rFEBr-wH-5@gated-at.bofh.it> (permalink)
References	<rFowF-7d9-5@gated-at.bofh.it> <rFowG-7d9-31@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

> 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@intel.com>

I'll work on fixing the other mess independently

-Tony

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 12/20] x86, edac: use Intel family name macros for edac driver Dave Hansen <dave@sr71.net> - 2016-06-02 02:20 +0200
  RE: [PATCH 12/20] x86, edac: use Intel family name macros for edac  driver "Luck, Tony" <tony.luck@intel.com> - 2016-06-02 18:20 +0200
  RE: [PATCH 12/20] x86, edac: use Intel family name macros for edac  driver "Luck, Tony" <tony.luck@intel.com> - 2016-06-02 19:30 +0200

csiph-web