diff options
| author | Borislav Petkov <borislav.petkov@amd.com> | 2009-06-22 17:42:24 (GMT) |
|---|---|---|
| committer | Borislav Petkov <borislav.petkov@amd.com> | 2009-06-26 11:06:41 (GMT) |
| commit | 30c875cbc1836a03a1acc6c998fa8a04f29f8f73 (patch) | |
| tree | 311168363af0e39dfd81d5150d18f1aa2d32df68 | |
| parent | 584fcff428bde3b9985ba21498764e9dba2fd3ce (diff) | |
| download | linux-30c875cbc1836a03a1acc6c998fa8a04f29f8f73.tar.xz | |
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>
| -rw-r--r-- | drivers/edac/amd64_edac.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c index 3b76605..8497963 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 " |