lparcfg: don't bother saving pointer to proc_dir_entry
authorAl Viro <viro@zeniv.linux.org.uk>
Mon, 1 Apr 2013 03:21:50 +0000 (23:21 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Tue, 9 Apr 2013 18:13:34 +0000 (14:13 -0400)
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
arch/powerpc/kernel/lparcfg.c

index f5725bce9ed29c55583b2805749a542628977698..801a757c363000fc7f5c4bc44bfcd51b3f30a239 100644 (file)
@@ -41,8 +41,6 @@
 
 /* #define LPARCFG_DEBUG */
 
-static struct proc_dir_entry *proc_ppc64_lparcfg;
-
 /*
  * Track sum of all purrs across all processors. This is used to further
  * calculate usage values by different applications
@@ -688,27 +686,22 @@ static const struct file_operations lparcfg_fops = {
 
 static int __init lparcfg_init(void)
 {
-       struct proc_dir_entry *ent;
        umode_t mode = S_IRUSR | S_IRGRP | S_IROTH;
 
        /* Allow writing if we have FW_FEATURE_SPLPAR */
        if (firmware_has_feature(FW_FEATURE_SPLPAR))
                mode |= S_IWUSR;
 
-       ent = proc_create("powerpc/lparcfg", mode, NULL, &lparcfg_fops);
-       if (!ent) {
+       if (!proc_create("powerpc/lparcfg", mode, NULL, &lparcfg_fops)) {
                printk(KERN_ERR "Failed to create powerpc/lparcfg\n");
                return -EIO;
        }
-
-       proc_ppc64_lparcfg = ent;
        return 0;
 }
 
 static void __exit lparcfg_cleanup(void)
 {
-       if (proc_ppc64_lparcfg)
-               remove_proc_entry("lparcfg", proc_ppc64_lparcfg->parent);
+       remove_proc_subtree("powerpc/lparcfg", NULL);
 }
 
 module_init(lparcfg_init);