diff options
author | Alexey Dobriyan <adobriyan@gmail.com> | 2008-04-29 04:01:44 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-29 11:06:18 -0400 |
commit | c74c120a21d87b0b6925ada5830d8cac21e852d9 (patch) | |
tree | 79558a29ecadc7b71eeb5bdf0945680f0560b2ed /drivers/s390 | |
parent | 928b4d8c8963e75bdb133f562b03b07f9aa4844a (diff) |
proc: remove proc_root from drivers
Remove proc_root export. Creation and removal works well if parent PDE is
supplied as NULL -- it worked always that way.
So, one useless export removed and consistency added, some drivers created
PDEs with &proc_root as parent but removed them as NULL and so on.
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/s390')
-rw-r--r-- | drivers/s390/block/dasd_proc.c | 6 | ||||
-rw-r--r-- | drivers/s390/char/tape_proc.c | 4 | ||||
-rw-r--r-- | drivers/s390/cio/blacklist.c | 2 | ||||
-rw-r--r-- | drivers/s390/cio/qdio.c | 4 |
4 files changed, 8 insertions, 8 deletions
diff --git a/drivers/s390/block/dasd_proc.c b/drivers/s390/block/dasd_proc.c index 556063e8f7a9..8ae9406b10ad 100644 --- a/drivers/s390/block/dasd_proc.c +++ b/drivers/s390/block/dasd_proc.c | |||
@@ -311,7 +311,7 @@ out_error: | |||
311 | int | 311 | int |
312 | dasd_proc_init(void) | 312 | dasd_proc_init(void) |
313 | { | 313 | { |
314 | dasd_proc_root_entry = proc_mkdir("dasd", &proc_root); | 314 | dasd_proc_root_entry = proc_mkdir("dasd", NULL); |
315 | if (!dasd_proc_root_entry) | 315 | if (!dasd_proc_root_entry) |
316 | goto out_nodasd; | 316 | goto out_nodasd; |
317 | dasd_proc_root_entry->owner = THIS_MODULE; | 317 | dasd_proc_root_entry->owner = THIS_MODULE; |
@@ -335,7 +335,7 @@ dasd_proc_init(void) | |||
335 | out_nostatistics: | 335 | out_nostatistics: |
336 | remove_proc_entry("devices", dasd_proc_root_entry); | 336 | remove_proc_entry("devices", dasd_proc_root_entry); |
337 | out_nodevices: | 337 | out_nodevices: |
338 | remove_proc_entry("dasd", &proc_root); | 338 | remove_proc_entry("dasd", NULL); |
339 | out_nodasd: | 339 | out_nodasd: |
340 | return -ENOENT; | 340 | return -ENOENT; |
341 | } | 341 | } |
@@ -345,5 +345,5 @@ dasd_proc_exit(void) | |||
345 | { | 345 | { |
346 | remove_proc_entry("devices", dasd_proc_root_entry); | 346 | remove_proc_entry("devices", dasd_proc_root_entry); |
347 | remove_proc_entry("statistics", dasd_proc_root_entry); | 347 | remove_proc_entry("statistics", dasd_proc_root_entry); |
348 | remove_proc_entry("dasd", &proc_root); | 348 | remove_proc_entry("dasd", NULL); |
349 | } | 349 | } |
diff --git a/drivers/s390/char/tape_proc.c b/drivers/s390/char/tape_proc.c index c9b96d51b28f..0c39636b2174 100644 --- a/drivers/s390/char/tape_proc.c +++ b/drivers/s390/char/tape_proc.c | |||
@@ -125,7 +125,7 @@ tape_proc_init(void) | |||
125 | { | 125 | { |
126 | tape_proc_devices = | 126 | tape_proc_devices = |
127 | create_proc_entry ("tapedevices", S_IFREG | S_IRUGO | S_IWUSR, | 127 | create_proc_entry ("tapedevices", S_IFREG | S_IRUGO | S_IWUSR, |
128 | &proc_root); | 128 | NULL); |
129 | if (tape_proc_devices == NULL) { | 129 | if (tape_proc_devices == NULL) { |
130 | PRINT_WARN("tape: Cannot register procfs entry tapedevices\n"); | 130 | PRINT_WARN("tape: Cannot register procfs entry tapedevices\n"); |
131 | return; | 131 | return; |
@@ -141,5 +141,5 @@ void | |||
141 | tape_proc_cleanup(void) | 141 | tape_proc_cleanup(void) |
142 | { | 142 | { |
143 | if (tape_proc_devices != NULL) | 143 | if (tape_proc_devices != NULL) |
144 | remove_proc_entry ("tapedevices", &proc_root); | 144 | remove_proc_entry ("tapedevices", NULL); |
145 | } | 145 | } |
diff --git a/drivers/s390/cio/blacklist.c b/drivers/s390/cio/blacklist.c index e8597ec92247..ef33d5df2229 100644 --- a/drivers/s390/cio/blacklist.c +++ b/drivers/s390/cio/blacklist.c | |||
@@ -375,7 +375,7 @@ cio_ignore_proc_init (void) | |||
375 | struct proc_dir_entry *entry; | 375 | struct proc_dir_entry *entry; |
376 | 376 | ||
377 | entry = create_proc_entry ("cio_ignore", S_IFREG | S_IRUGO | S_IWUSR, | 377 | entry = create_proc_entry ("cio_ignore", S_IFREG | S_IRUGO | S_IWUSR, |
378 | &proc_root); | 378 | NULL); |
379 | if (!entry) | 379 | if (!entry) |
380 | return -ENOENT; | 380 | return -ENOENT; |
381 | 381 | ||
diff --git a/drivers/s390/cio/qdio.c b/drivers/s390/cio/qdio.c index 10aa1e780801..43876e287370 100644 --- a/drivers/s390/cio/qdio.c +++ b/drivers/s390/cio/qdio.c | |||
@@ -3632,7 +3632,7 @@ qdio_add_procfs_entry(void) | |||
3632 | { | 3632 | { |
3633 | proc_perf_file_registration=0; | 3633 | proc_perf_file_registration=0; |
3634 | qdio_perf_proc_file=create_proc_entry(QDIO_PERF, | 3634 | qdio_perf_proc_file=create_proc_entry(QDIO_PERF, |
3635 | S_IFREG|0444,&proc_root); | 3635 | S_IFREG|0444,NULL); |
3636 | if (qdio_perf_proc_file) { | 3636 | if (qdio_perf_proc_file) { |
3637 | qdio_perf_proc_file->read_proc=&qdio_perf_procfile_read; | 3637 | qdio_perf_proc_file->read_proc=&qdio_perf_procfile_read; |
3638 | } else proc_perf_file_registration=-1; | 3638 | } else proc_perf_file_registration=-1; |
@@ -3647,7 +3647,7 @@ static void | |||
3647 | qdio_remove_procfs_entry(void) | 3647 | qdio_remove_procfs_entry(void) |
3648 | { | 3648 | { |
3649 | if (!proc_perf_file_registration) /* means if it went ok earlier */ | 3649 | if (!proc_perf_file_registration) /* means if it went ok earlier */ |
3650 | remove_proc_entry(QDIO_PERF,&proc_root); | 3650 | remove_proc_entry(QDIO_PERF,NULL); |
3651 | } | 3651 | } |
3652 | 3652 | ||
3653 | /** | 3653 | /** |