• Tony Luck's avatar
    EDAC: Drop duplicated array of strings for memory type names · d6dd77eb
    Tony Luck authored
    Somehow we ended up with two separate arrays of strings to describe the
    "enum mem_type" values.
    
    In edac_mc.c we have an exported list edac_mem_types[] that is used
    by a couple of drivers in debug messaged.
    
    In edac_mc_sysfs.c we have a private list that is used to display
    values in:
      /sys/devices/system/edac/mc/mc*/dimm*/dimm_mem_type
      /sys/devices/system/edac/mc/mc*/csrow*/mem_type
    
    This list was missing a value for MEM_LRDDR3.
    
    The string values in the two lists were different :-(
    
    Combining the lists, I kept the values so that the sysfs output
    will be unchanged as some scripts may depend on that.
    Reported-by: default avatarBorislav Petkov <bp@suse.de>
    Acked-by: default avatarBorislav Petkov <bp@suse.de>
    Signed-off-by: default avatarTony Luck <tony.luck@intel.com>
    Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
    Cc: Aristeu Rozanski <aris@redhat.com>
    Cc: Dan Williams <dan.j.williams@intel.com>
    Cc: Jean Delvare <jdelvare@suse.com>
    Cc: Len Brown <lenb@kernel.org>
    Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
    Cc: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
    Cc: linux-acpi@vger.kernel.org
    Cc: linux-edac <linux-edac@vger.kernel.org>
    Cc: linux-nvdimm@lists.01.org
    Link: http://lkml.kernel.org/r/20180312182430.10335-2-tony.luck@intel.comSigned-off-by: default avatarBorislav Petkov <bp@suse.de>
    d6dd77eb
edac_mc_sysfs.c 27.4 KB