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 36e1486..d0c372e 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;
        }