diff options
author | Adrian Bunk <bunk@kernel.org> | 2008-04-29 04:03:18 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-29 11:06:26 -0400 |
commit | 1a45027d1afd7e85254b5ef8535e93ce3d588cf4 (patch) | |
tree | 20fae852bbb0f352ed787ed283581e5311a66319 /drivers/edac/edac_pci.c | |
parent | ff6ac2a616c85d1215899ffda815e29b699cbd3a (diff) |
edac: remove unneeded functions and add static accessor
Collection of patches, merged into one, from Adrian that do the following:
1) This patch makes the following needlessly global functions static:
- edac_pci_get_log_pe()
- edac_pci_get_log_npe()
- edac_pci_get_panic_on_pe()
- edac_pci_unregister_sysfs_instance_kobj()
- edac_pci_main_kobj_setup()
2) Remove unneeded function edac_device_find()
3) Added #if 0 around function edac_pci_find()
4) make the needlessly global edac_pci_generic_check() static
5) Removed function edac_check_mc_devices()
Doug Thompson modified Adrian's patches, to bettern represent
the direction of EDAC, and make them one patch.
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Signed-off-by: Adrian Bunk <bunk@kernel.org>
Signed-off-by: Doug Thompson <dougthompson@xmission.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/edac/edac_pci.c')
-rw-r--r-- | drivers/edac/edac_pci.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/edac/edac_pci.c b/drivers/edac/edac_pci.c index 2d4176c3133e..9b24340b52e1 100644 --- a/drivers/edac/edac_pci.c +++ b/drivers/edac/edac_pci.c | |||
@@ -189,6 +189,9 @@ static void del_edac_pci_from_global_list(struct edac_pci_ctl_info *pci) | |||
189 | wait_for_completion(&pci->complete); | 189 | wait_for_completion(&pci->complete); |
190 | } | 190 | } |
191 | 191 | ||
192 | #if 0 | ||
193 | /* Older code, but might use in the future */ | ||
194 | |||
192 | /* | 195 | /* |
193 | * edac_pci_find() | 196 | * edac_pci_find() |
194 | * Search for an edac_pci_ctl_info structure whose index is 'idx' | 197 | * Search for an edac_pci_ctl_info structure whose index is 'idx' |
@@ -219,6 +222,7 @@ struct edac_pci_ctl_info *edac_pci_find(int idx) | |||
219 | return NULL; | 222 | return NULL; |
220 | } | 223 | } |
221 | EXPORT_SYMBOL_GPL(edac_pci_find); | 224 | EXPORT_SYMBOL_GPL(edac_pci_find); |
225 | #endif | ||
222 | 226 | ||
223 | /* | 227 | /* |
224 | * edac_pci_workq_function() | 228 | * edac_pci_workq_function() |
@@ -422,7 +426,7 @@ EXPORT_SYMBOL_GPL(edac_pci_del_device); | |||
422 | * | 426 | * |
423 | * a Generic parity check API | 427 | * a Generic parity check API |
424 | */ | 428 | */ |
425 | void edac_pci_generic_check(struct edac_pci_ctl_info *pci) | 429 | static void edac_pci_generic_check(struct edac_pci_ctl_info *pci) |
426 | { | 430 | { |
427 | debugf4("%s()\n", __func__); | 431 | debugf4("%s()\n", __func__); |
428 | edac_pci_do_parity_check(); | 432 | edac_pci_do_parity_check(); |