• Mark A. Grondona's avatar
    EDAC: Fix incorrect edac mode reporting in sb_edac · c6e13b52
    Mark A. Grondona authored
    The edac driver for Sandy Bridge was found to be reporting "FPM"
    for edac_mode, which clearly doesn't make sense. It was found that
    sb_edac.c:get_dimm_config was reusing a variable for both mem_type
    and edac_type, and thus was overwriting the value after setting
    it correctly. This patch fixes that issue.
    
    Before the patch:
    /sys/devices/system/edac/mc/mc0/csrow0/edac_mode:FPM
    /sys/devices/system/edac/mc/mc0/csrow1/edac_mode:FPM
    /sys/devices/system/edac/mc/mc0/csrow2/edac_mode:FPM
    /sys/devices/system/edac/mc/mc0/csrow3/edac_mode:FPM
    
    After:
    /sys/devices/system/edac/mc/mc0/csrow0/edac_mode:S4ECD4ED
    /sys/devices/system/edac/mc/mc0/csrow1/edac_mode:S4ECD4ED
    /sys/devices/system/edac/mc/mc0/csrow2/edac_mode:S4ECD4ED
    /sys/devices/system/edac/mc/mc0/csrow3/edac_mode:S4ECD4ED
    Signed-off-by: default avatarMark A. Grondona <mgrondona@llnl.gov>
    Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
    c6e13b52
sb_edac.c 46 KB