summaryrefslogtreecommitdiffstats
path: root/drivers/edac
diff options
context:
space:
mode:
authorBorislav Petkov <borislav.petkov@amd.com>2009-06-22 19:42:24 +0200
committerBorislav Petkov <borislav.petkov@amd.com>2009-06-26 13:06:41 +0200
commit30c875cbc1836a03a1acc6c998fa8a04f29f8f73 (patch)
tree311168363af0e39dfd81d5150d18f1aa2d32df68 /drivers/edac
parentamd64_edac: check only ECC bit in amd64_determine_edac_cap (diff)
downloadlinux-30c875cbc1836a03a1acc6c998fa8a04f29f8f73.tar.xz
linux-30c875cbc1836a03a1acc6c998fa8a04f29f8f73.zip
amd64_edac: fix ecc_enable_override handling
amd64_check_ecc_enabled() returns non-zero status when ECC checking/correcting is disabled and this fails further loading of the driver even when 'ecc_enable_override' boot param is used. Fix that by clearing return status in that case. Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
Diffstat (limited to 'drivers/edac')
-rw-r--r--drivers/edac/amd64_edac.c7
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c
index 3b76605330e9..8497963a61f6 100644
--- a/drivers/edac/amd64_edac.c
+++ b/drivers/edac/amd64_edac.c
@@ -2966,7 +2966,12 @@ static int amd64_check_ecc_enabled(struct amd64_pvt *pvt)
" Use of the override can cause "
"unknown side effects.\n");
ret = -ENODEV;
- }
+ } else
+ /*
+ * enable further driver loading if ECC enable is
+ * overridden.
+ */
+ ret = 0;
} else {
amd64_printk(KERN_INFO,
"ECC is enabled by BIOS, Proceeding "