diff options
author | Borislav Petkov <borislav.petkov@amd.com> | 2010-10-01 16:38:19 (GMT) |
---|---|---|
committer | Borislav Petkov <borislav.petkov@amd.com> | 2011-01-07 10:33:50 (GMT) |
commit | 395ae783b384e5243804b07fba3e3f8379ddf1d6 (patch) | |
tree | d5b841a23508eed59caf67d7a4ee411e808b2860 /drivers | |
parent | 9f56da0e3c3269abe0b2745a54f1b082c3c14433 (diff) | |
download | linux-395ae783b384e5243804b07fba3e3f8379ddf1d6.tar.xz |
amd64_edac: Add per-family init function
Run a per-family init function which does all the settings based on
the family this driver instance is running on. Move the scrubrate
calculation in it and simplify code.
Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/edac/amd64_edac.c | 44 | ||||
-rw-r--r-- | drivers/edac/amd64_edac.h | 3 |
2 files changed, 28 insertions, 19 deletions
diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c index 0cbcf6e..de36721 100644 --- a/drivers/edac/amd64_edac.c +++ b/drivers/edac/amd64_edac.c @@ -117,8 +117,7 @@ struct scrubrate scrubrates[] = { * scan the scrub rate mapping table for a close or matching bandwidth value to * issue. If requested is too big, then use last maximum value found. */ -static int amd64_search_set_scrub_rate(struct pci_dev *ctl, u32 new_bw, - u32 min_scrubrate) +static int __amd64_set_scrub_rate(struct pci_dev *ctl, u32 new_bw, u32 min_rate) { u32 scrubval; int i; @@ -134,7 +133,7 @@ static int amd64_search_set_scrub_rate(struct pci_dev *ctl, u32 new_bw, * skip scrub rates which aren't recommended * (see F10 BKDG, F3x58) */ - if (scrubrates[i].scrubval < min_scrubrate) + if (scrubrates[i].scrubval < min_rate) continue; if (scrubrates[i].bandwidth <= new_bw) @@ -160,25 +159,11 @@ static int amd64_search_set_scrub_rate(struct pci_dev *ctl, u32 new_bw, return 0; } -static int amd64_set_scrub_rate(struct mem_ctl_info *mci, u32 bandwidth) +static int amd64_set_scrub_rate(struct mem_ctl_info *mci, u32 bw) { struct amd64_pvt *pvt = mci->pvt_info; - u32 min_scrubrate = 0x0; - switch (boot_cpu_data.x86) { - case 0xf: - min_scrubrate = K8_MIN_SCRUB_RATE_BITS; - break; - case 0x10: - min_scrubrate = F10_MIN_SCRUB_RATE_BITS; - break; - - default: - amd64_printk(KERN_ERR, "Unsupported family!\n"); - return -EINVAL; - } - return amd64_search_set_scrub_rate(pvt->misc_f3_ctl, bandwidth, - min_scrubrate); + return __amd64_set_scrub_rate(pvt->misc_f3_ctl, bw, pvt->min_scrubrate); } static int amd64_get_scrub_rate(struct mem_ctl_info *mci, u32 *bw) @@ -2607,6 +2592,23 @@ static void amd64_setup_mci_misc_attributes(struct mem_ctl_info *mci) mci->get_sdram_scrub_rate = amd64_get_scrub_rate; } +static int amd64_per_family_init(struct amd64_pvt *pvt) +{ + switch (boot_cpu_data.x86) { + case 0xf: + pvt->min_scrubrate = K8_MIN_SCRUB_RATE_BITS; + break; + case 0x10: + pvt->min_scrubrate = F10_MIN_SCRUB_RATE_BITS; + break; + + default: + amd64_printk(KERN_ERR, "Unsupported family!\n"); + return -EINVAL; + } + return 0; +} + /* * Init stuff for this DRAM Controller device. * @@ -2637,6 +2639,10 @@ static int amd64_probe_one_instance(struct pci_dev *dram_f2_ctl, pvt->mc_type_index = mc_type_index; pvt->ops = family_ops(mc_type_index); + ret = -EINVAL; + if (amd64_per_family_init(pvt)) + goto err_free; + /* * We have the dram_f2_ctl device as an argument, now go reserve its * sibling devices from the PCI system. diff --git a/drivers/edac/amd64_edac.h b/drivers/edac/amd64_edac.h index c8f2734..e5204fe 100644 --- a/drivers/edac/amd64_edac.h +++ b/drivers/edac/amd64_edac.h @@ -461,6 +461,9 @@ struct amd64_pvt { /* MC Type Index value: socket F vs Family 10h */ u32 mc_type_index; + /* DCT per-family scrubrate setting */ + u32 min_scrubrate; + /* misc settings */ struct flags { unsigned long cf8_extcfg:1; |