MCE, AMD: Drop too granulary family model checks
authorBorislav Petkov <borislav.petkov@amd.com>
Wed, 4 Apr 2012 12:21:02 +0000 (14:21 +0200)
committerBorislav Petkov <borislav.petkov@amd.com>
Wed, 4 Apr 2012 13:50:11 +0000 (15:50 +0200)
MCA details seldom change inbetween the models of a family so don't
be too conservative and enable decoding on everything starting from
K8 onwards. Minor adjustments can come in later but most importantly,
we have some decoding infrastructure in place for upcoming models by
default.

Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
drivers/edac/mce_amd.c

index 36e1486eb9aa0578583f60697e647ace8c8631b4..d0c372e30de41766cac54568c12d2994738cbf38 100644 (file)
@@ -754,9 +754,7 @@ static int __init mce_amd_init(void)
        if (c->x86_vendor != X86_VENDOR_AMD)
                return 0;
 
-       if ((c->x86 < 0xf || c->x86 > 0x12) &&
-           (c->x86 != 0x14 || c->x86_model > 0xf) &&
-           (c->x86 != 0x15 || c->x86_model > 0xf))
+       if (c->x86 < 0xf || c->x86 > 0x15)
                return 0;
 
        fam_ops = kzalloc(sizeof(struct amd_decoder_ops), GFP_KERNEL);
@@ -797,7 +795,7 @@ static int __init mce_amd_init(void)
                break;
 
        default:
-               printk(KERN_WARNING "Huh? What family is that: %d?!\n", c->x86);
+               printk(KERN_WARNING "Huh? What family is it: 0x%x?!\n", c->x86);
                kfree(fam_ops);
                return -EINVAL;
        }