diff options
| author |   Borislav Petkov <borislav.petkov@amd.com> | 2010-05-25 18:21:07 +0200 |
|---|---|---|
| committer | Borislav Petkov <borislav.petkov@amd.com> | 2010-08-03 16:14:06 +0200 |
| commit | eba042a81edd6baaff44831b2d719b14a6d21e58 (patch) | |
| tree | a1e768aadf18708114e2d6c4d4a54b3af49e42e9 /drivers/edac/edac_core.h | |
| parent | amd64_edac: Correct scrub rate setting (diff) | |
| download | linux-dev-eba042a81edd6baaff44831b2d719b14a6d21e58.tar.xz linux-dev-eba042a81edd6baaff44831b2d719b14a6d21e58.zip | |
edac, mc: Improve scrub rate handling
Fortify the interface to not accept negative values, remove
memctrl_int_store() as a result. Also, sanitize bandwidth setting by
making the argument a simple u32 instead of strange u32 pointer being
passed around for no obvious reason. Then, fix error handling and teach
it to return proper error values. Finally, make code more readable,
simplify debug messages.
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: Arthur Jones <ajones@riverbed.com>
Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
Acked-by: Doug Thompson <dougthompson@xmission.com>
Diffstat (limited to 'drivers/edac/edac_core.h')
| -rw-r--r-- | drivers/edac/edac_core.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/edac/edac_core.h b/drivers/edac/edac_core.h index ade4f1d70539..ce7146677e9b 100644 --- a/drivers/edac/edac_core.h +++ b/drivers/edac/edac_core.h @@ -378,7 +378,7 @@ struct mem_ctl_info { internal representation and configures whatever else needs to be configured. */ - int (*set_sdram_scrub_rate) (struct mem_ctl_info * mci, u32 * bw); + int (*set_sdram_scrub_rate) (struct mem_ctl_info * mci, u32 bw); /* Get the current sdram memory scrub rate from the internal representation and converts it to the closest matching |