Re: [PATCH 0/5] Teach EDAC about non-volatile DIMMs and add partial support to skx_edac

[Dan Williams <dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>]

On Wed, Feb 28, 2018 at 5:04 AM, Borislav Petkov <bp-Gina5bIWoIWzQB+pC5nmwQ@public.gmane.org> wrote:
> On Thu, Feb 22, 2018 at 11:58:06AM -0800, Tony Luck wrote:
>> This series was posted as RFC and got some review back in December:
>>
>>    https://marc.info/?l=linux-edac&m=151257213825419&w=2
>>
>> It couldn't go upstream back then because I was waiting for some updates
>> to the ACPICA headers for NFIT support.  Those patches are complete now,
>> so here is the series again.
>>
>> The "partial support" part is that the changes here only do the detection of
>> the non-volatile DIMMs. Coming later will be another patch/series to teach
>> skx_edac how to decode system addresses to NVDIMM addresses. But this part
>> stands on its own and is a useful first step.
>>
>> Tony Luck (5):
>>   EDAC: Drop duplicated array of strings for memory type names
>>   edac: Add new memory type for non-volatile DIMMs
>>   acpi, nfit: Add function to look up nvdimm device and provide SMBIOS
>>     handle
>>   firmware: dmi: Add function to look up a handle and return DIMM size
>>   EDAC, skx_edac: Detect non-volatile DIMMs
>>
>>  drivers/acpi/nfit/core.c     | 27 ++++++++++++++++++
>>  drivers/edac/Kconfig         |  5 +++-
>>  drivers/edac/edac_mc.c       | 41 +++++++++++++-------------
>>  drivers/edac/edac_mc_sysfs.c | 26 ++---------------
>>  drivers/edac/skx_edac.c      | 68 ++++++++++++++++++++++++++++++++++++++++----
>>  drivers/firmware/dmi_scan.c  | 31 ++++++++++++++++++++
>>  include/acpi/nfit.h          | 26 +++++++++++++++++
>>  include/linux/dmi.h          |  2 ++
>>  include/linux/edac.h         |  3 ++
>>  9 files changed, 178 insertions(+), 51 deletions(-)
>>  create mode 100644 include/acpi/nfit.h
>
> Series looks ok to me.
>
> Dan, I could route patch 3 through the EDAC tree if you'd prefer.

Yes, please, and you can add:

Acked-by: Dan Williams <dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>