diff options
| author | Borislav Petkov <borislav.petkov@amd.com> | 2009-06-22 19:42:24 +0200 |
|---|---|---|
| committer | Borislav Petkov <borislav.petkov@amd.com> | 2009-06-26 13:06:41 +0200 |
| commit | 30c875cbc1836a03a1acc6c998fa8a04f29f8f73 (patch) | |
| tree | 311168363af0e39dfd81d5150d18f1aa2d32df68 /drivers/edac | |
| parent | 584fcff428bde3b9985ba21498764e9dba2fd3ce (diff) | |
| download | lwn-30c875cbc1836a03a1acc6c998fa8a04f29f8f73.tar.gz lwn-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.c | 7 |
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 " |