diff options
author | Arjan van de Ven <arjan@linux.intel.com> | 2006-03-11 06:27:15 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-11 12:19:34 -0500 |
commit | 4136cabff33d6d73b8daf2f2612670cc0296f844 (patch) | |
tree | 85fe83bd7ab21edcda6b3b17ac1b96e435f3b562 | |
parent | cd6ef84e6ac9454080707f2f338360f5d7e556fc (diff) |
[PATCH] edac: disable a few sysfs files to avoid them becoming an ABI
Disable (via ugly #if 0's) the 3 sysfs files that I think by now we all
agree are very much wrong. These files shouldn't become part of the ABI by
the 2.6.16 release, so I rather have this minimal patch merged to disable
them for now, the real fix can then come during the 2.6.17 devel window.
Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | drivers/edac/edac_mc.c | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/drivers/edac/edac_mc.c b/drivers/edac/edac_mc.c index b10ee4698b1d..262e44544dc8 100644 --- a/drivers/edac/edac_mc.c +++ b/drivers/edac/edac_mc.c | |||
@@ -132,11 +132,13 @@ static struct kobject edac_pci_kobj; | |||
132 | * /sys/devices/system/edac/mc; | 132 | * /sys/devices/system/edac/mc; |
133 | * data structures and methods | 133 | * data structures and methods |
134 | */ | 134 | */ |
135 | #if 0 | ||
135 | static ssize_t memctrl_string_show(void *ptr, char *buffer) | 136 | static ssize_t memctrl_string_show(void *ptr, char *buffer) |
136 | { | 137 | { |
137 | char *value = (char*) ptr; | 138 | char *value = (char*) ptr; |
138 | return sprintf(buffer, "%s\n", value); | 139 | return sprintf(buffer, "%s\n", value); |
139 | } | 140 | } |
141 | #endif | ||
140 | 142 | ||
141 | static ssize_t memctrl_int_show(void *ptr, char *buffer) | 143 | static ssize_t memctrl_int_show(void *ptr, char *buffer) |
142 | { | 144 | { |
@@ -207,7 +209,9 @@ struct memctrl_dev_attribute attr_##_name = { \ | |||
207 | }; | 209 | }; |
208 | 210 | ||
209 | /* cwrow<id> attribute f*/ | 211 | /* cwrow<id> attribute f*/ |
212 | #if 0 | ||
210 | MEMCTRL_STRING_ATTR(mc_version,EDAC_MC_VERSION,S_IRUGO,memctrl_string_show,NULL); | 213 | MEMCTRL_STRING_ATTR(mc_version,EDAC_MC_VERSION,S_IRUGO,memctrl_string_show,NULL); |
214 | #endif | ||
211 | 215 | ||
212 | /* csrow<id> control files */ | 216 | /* csrow<id> control files */ |
213 | MEMCTRL_ATTR(panic_on_ue,S_IRUGO|S_IWUSR,memctrl_int_show,memctrl_int_store); | 217 | MEMCTRL_ATTR(panic_on_ue,S_IRUGO|S_IWUSR,memctrl_int_show,memctrl_int_store); |
@@ -222,7 +226,6 @@ static struct memctrl_dev_attribute *memctrl_attr[] = { | |||
222 | &attr_log_ue, | 226 | &attr_log_ue, |
223 | &attr_log_ce, | 227 | &attr_log_ce, |
224 | &attr_poll_msec, | 228 | &attr_poll_msec, |
225 | &attr_mc_version, | ||
226 | NULL, | 229 | NULL, |
227 | }; | 230 | }; |
228 | 231 | ||
@@ -309,6 +312,8 @@ struct list_control { | |||
309 | int *count; | 312 | int *count; |
310 | }; | 313 | }; |
311 | 314 | ||
315 | |||
316 | #if 0 | ||
312 | /* Output the list as: vendor_id:device:id<,vendor_id:device_id> */ | 317 | /* Output the list as: vendor_id:device:id<,vendor_id:device_id> */ |
313 | static ssize_t edac_pci_list_string_show(void *ptr, char *buffer) | 318 | static ssize_t edac_pci_list_string_show(void *ptr, char *buffer) |
314 | { | 319 | { |
@@ -430,6 +435,7 @@ static ssize_t edac_pci_list_string_store(void *ptr, const char *buffer, | |||
430 | return count; | 435 | return count; |
431 | } | 436 | } |
432 | 437 | ||
438 | #endif | ||
433 | static ssize_t edac_pci_int_show(void *ptr, char *buffer) | 439 | static ssize_t edac_pci_int_show(void *ptr, char *buffer) |
434 | { | 440 | { |
435 | int *value = ptr; | 441 | int *value = ptr; |
@@ -498,6 +504,7 @@ struct edac_pci_dev_attribute edac_pci_attr_##_name = { \ | |||
498 | .store = _store, \ | 504 | .store = _store, \ |
499 | }; | 505 | }; |
500 | 506 | ||
507 | #if 0 | ||
501 | static struct list_control pci_whitelist_control = { | 508 | static struct list_control pci_whitelist_control = { |
502 | .list = pci_whitelist, | 509 | .list = pci_whitelist, |
503 | .count = &pci_whitelist_count | 510 | .count = &pci_whitelist_count |
@@ -520,6 +527,7 @@ EDAC_PCI_STRING_ATTR(pci_parity_blacklist, | |||
520 | S_IRUGO|S_IWUSR, | 527 | S_IRUGO|S_IWUSR, |
521 | edac_pci_list_string_show, | 528 | edac_pci_list_string_show, |
522 | edac_pci_list_string_store); | 529 | edac_pci_list_string_store); |
530 | #endif | ||
523 | 531 | ||
524 | /* PCI Parity control files */ | 532 | /* PCI Parity control files */ |
525 | EDAC_PCI_ATTR(check_pci_parity,S_IRUGO|S_IWUSR,edac_pci_int_show,edac_pci_int_store); | 533 | EDAC_PCI_ATTR(check_pci_parity,S_IRUGO|S_IWUSR,edac_pci_int_show,edac_pci_int_store); |
@@ -531,8 +539,6 @@ static struct edac_pci_dev_attribute *edac_pci_attr[] = { | |||
531 | &edac_pci_attr_check_pci_parity, | 539 | &edac_pci_attr_check_pci_parity, |
532 | &edac_pci_attr_panic_on_pci_parity, | 540 | &edac_pci_attr_panic_on_pci_parity, |
533 | &edac_pci_attr_pci_parity_count, | 541 | &edac_pci_attr_pci_parity_count, |
534 | &edac_pci_attr_pci_parity_whitelist, | ||
535 | &edac_pci_attr_pci_parity_blacklist, | ||
536 | NULL, | 542 | NULL, |
537 | }; | 543 | }; |
538 | 544 | ||