diff options
| author | Doug Thompson <dougthompson@xmission.com> | 2009-08-03 12:37:06 +0200 |
|---|---|---|
| committer | Borislav Petkov <borislav.petkov@amd.com> | 2009-08-03 16:54:20 +0200 |
| commit | 126b67b8d26f6623d199aa59279f2e3243f2144c (patch) | |
| tree | 034e8bab3512d4c5ff87400925a83754c2918937 /drivers/edac | |
| parent | Linux 2.6.31-rc5 (diff) | |
| download | linux-126b67b8d26f6623d199aa59279f2e3243f2144c.tar.xz linux-126b67b8d26f6623d199aa59279f2e3243f2144c.zip | |
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) "ECC is enabled by BIOS, Proceeding " "with EDAC module initialization\n"); + /* Signal good ECC status */ + ret = 0; + /* CLEAR the override, since BIOS controlled it */ ecc_enable_override = 0; } |