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_device.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_device.c')
-rw-r--r-- | drivers/edac/edac_device.c | 31 |
1 files changed, 0 insertions, 31 deletions
diff --git a/drivers/edac/edac_device.c b/drivers/edac/edac_device.c index c559bf543422..63372fa7ecfe 100644 --- a/drivers/edac/edac_device.c +++ b/drivers/edac/edac_device.c | |||
@@ -375,37 +375,6 @@ static void del_edac_device_from_global_list(struct edac_device_ctl_info | |||
375 | wait_for_completion(&edac_device->removal_complete); | 375 | wait_for_completion(&edac_device->removal_complete); |
376 | } | 376 | } |
377 | 377 | ||
378 | /** | ||
379 | * edac_device_find | ||
380 | * Search for a edac_device_ctl_info structure whose index is 'idx'. | ||
381 | * | ||
382 | * If found, return a pointer to the structure. | ||
383 | * Else return NULL. | ||
384 | * | ||
385 | * Caller must hold device_ctls_mutex. | ||
386 | */ | ||
387 | struct edac_device_ctl_info *edac_device_find(int idx) | ||
388 | { | ||
389 | struct list_head *item; | ||
390 | struct edac_device_ctl_info *edac_dev; | ||
391 | |||
392 | /* Iterate over list, looking for exact match of ID */ | ||
393 | list_for_each(item, &edac_device_list) { | ||
394 | edac_dev = list_entry(item, struct edac_device_ctl_info, link); | ||
395 | |||
396 | if (edac_dev->dev_idx >= idx) { | ||
397 | if (edac_dev->dev_idx == idx) | ||
398 | return edac_dev; | ||
399 | |||
400 | /* not on list, so terminate early */ | ||
401 | break; | ||
402 | } | ||
403 | } | ||
404 | |||
405 | return NULL; | ||
406 | } | ||
407 | EXPORT_SYMBOL_GPL(edac_device_find); | ||
408 | |||
409 | /* | 378 | /* |
410 | * edac_device_workq_function | 379 | * edac_device_workq_function |
411 | * performs the operation scheduled by a workq request | 380 | * performs the operation scheduled by a workq request |