diff options
author | Doug Thompson <dougthompson@xmission.com> | 2009-08-03 06:37:06 -0400 |
---|---|---|
committer | Borislav Petkov <borislav.petkov@amd.com> | 2009-08-03 10:54:20 -0400 |
commit | 126b67b8d26f6623d199aa59279f2e3243f2144c (patch) | |
tree | 034e8bab3512d4c5ff87400925a83754c2918937 /drivers/edac | |
parent | ed680c4ad478d0fee9740f7d029087f181346564 (diff) |
amd64_edac: fix ECC checking
On the good path of BIOS enabled ECC and no override, the value returned
is 1 by omission and thus is deemed failing by the probe-function.
Allow proper module initialization by clearing the retval explicitly.
Signed-off-by: Doug Thompson <dougthompson@xmission.com>
Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
Diffstat (limited to 'drivers/edac')
-rw-r--r-- | drivers/edac/amd64_edac.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c index 24964c1d0af9..5fa924d61b10 100644 --- a/drivers/edac/amd64_edac.c +++ b/drivers/edac/amd64_edac.c | |||
@@ -2977,6 +2977,9 @@ static int amd64_check_ecc_enabled(struct amd64_pvt *pvt) | |||
2977 | "ECC is enabled by BIOS, Proceeding " | 2977 | "ECC is enabled by BIOS, Proceeding " |
2978 | "with EDAC module initialization\n"); | 2978 | "with EDAC module initialization\n"); |
2979 | 2979 | ||
2980 | /* Signal good ECC status */ | ||
2981 | ret = 0; | ||
2982 | |||
2980 | /* CLEAR the override, since BIOS controlled it */ | 2983 | /* CLEAR the override, since BIOS controlled it */ |
2981 | ecc_enable_override = 0; | 2984 | ecc_enable_override = 0; |
2982 | } | 2985 | } |