[PATCH] proc: remove useless (and buggy) ->nlink settings

Bug: pnx8550 code creates directory but resets ->nlink to 1.

create_proc_entry() et al will correctly set ->nlink for you.

Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Jeff Dike <jdike@addtoit.com>
Cc: Corey Minyard <minyard@acm.org>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Kyle McMartin <kyle@mcmartin.ca>
Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
Cc: Greg KH <greg@kroah.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

7 files changed, 0 insertions, 9 deletions

diff --git a/drivers/char/ipmi/ipmi_msghandler.c b/drivers/char/ipmi/ipmi_msghandler.c
index 230064ede08d..3aff5e99b674 100644
--- a/drivers/char/ipmi/ipmi_msghandler.c
+++ b/drivers/char/ipmi/ipmi_msghandler.c
@@ -1887,7 +1887,6 @@ int ipmi_smi_add_proc_entry(ipmi_smi_t smi, char *name,
                 kfree(entry);
                 rv = -ENOMEM;
         } else {
-                file->nlink = 1;
                 file->data = data;
                 file->read_proc = read_proc;
                 file->write_proc = write_proc;
diff --git a/drivers/ide/ide-proc.c b/drivers/ide/ide-proc.c
index aa049dab3d95..ad49bd823ebd 100644
--- a/drivers/ide/ide-proc.c
+++ b/drivers/ide/ide-proc.c
@@ -413,7 +413,6 @@ void ide_add_proc_entries(struct proc_dir_entry *dir, ide_proc_entry_t *p, void
         while (p->name != NULL) {
                 ent = create_proc_entry(p->name, p->mode, dir);
                 if (!ent) return;
-                ent->nlink = 1;
                 ent->data = data;
                 ent->read_proc = p->read_proc;
                 ent->write_proc = p->write_proc;
diff --git a/drivers/macintosh/via-pmu.c b/drivers/macintosh/via-pmu.c
index 8ca75e52f637..eb6653f69ce4 100644
--- a/drivers/macintosh/via-pmu.c
+++ b/drivers/macintosh/via-pmu.c
@@ -516,7 +516,6 @@ static int __init via_pmu_dev_init(void)
                                         proc_get_irqstats, NULL);
                 proc_pmu_options = create_proc_entry("options", 0600, proc_pmu_root);
                 if (proc_pmu_options) {
-                        proc_pmu_options->nlink = 1;
                         proc_pmu_options->read_proc = proc_read_options;
                         proc_pmu_options->write_proc = proc_write_options;
                 }
diff --git a/drivers/misc/hdpuftrs/hdpu_nexus.c b/drivers/misc/hdpuftrs/hdpu_nexus.c
index ea9d5f233c83..6a51e99a8079 100644
--- a/drivers/misc/hdpuftrs/hdpu_nexus.c
+++ b/drivers/misc/hdpuftrs/hdpu_nexus.c
@@ -72,11 +72,9 @@ static int hdpu_nexus_probe(struct platform_device *pdev)
                 printk("Could not map slot id\n");
         hdpu_slot_id = create_proc_entry("sky_slot_id", 0666, &proc_root);
         hdpu_slot_id->read_proc = hdpu_slot_id_read;
-        hdpu_slot_id->nlink = 1;
 
         hdpu_chassis_id = create_proc_entry("sky_chassis_id", 0666, &proc_root);
         hdpu_chassis_id->read_proc = hdpu_chassis_id_read;
-        hdpu_chassis_id->nlink = 1;
         return 0;
 }
 
diff --git a/drivers/parisc/led.c b/drivers/parisc/led.c
index 8dac2ba82bb9..9a731c101d10 100644
--- a/drivers/parisc/led.c
+++ b/drivers/parisc/led.c
@@ -252,7 +252,6 @@ static int __init led_create_procfs(void)
         proc_pdc_root->owner = THIS_MODULE;
         ent = create_proc_entry("led", S_IFREG|S_IRUGO|S_IWUSR, proc_pdc_root);
         if (!ent) return -1;
-        ent->nlink = 1;
         ent->data = (void *)LED_NOLCD; /* LED */
         ent->read_proc = led_proc_read;
         ent->write_proc = led_proc_write;
@@ -262,7 +261,6 @@ static int __init led_create_procfs(void)
         {
                 ent = create_proc_entry("lcd", S_IFREG|S_IRUGO|S_IWUSR, proc_pdc_root);
                 if (!ent) return -1;
-                ent->nlink = 1;
                 ent->data = (void *)LED_HASLCD; /* LCD */
                 ent->read_proc = led_proc_read;
                 ent->write_proc = led_proc_write;
diff --git a/drivers/s390/crypto/zcrypt_api.c b/drivers/s390/crypto/zcrypt_api.c
index b9e59bc9435a..2c785148d21e 100644
--- a/drivers/s390/crypto/zcrypt_api.c
+++ b/drivers/s390/crypto/zcrypt_api.c
@@ -1063,7 +1063,6 @@ int __init zcrypt_api_init(void)
                 rc = -ENOMEM;
                 goto out_misc;
         }
-        zcrypt_entry->nlink = 1;
         zcrypt_entry->data = NULL;
         zcrypt_entry->read_proc = zcrypt_status_read;
         zcrypt_entry->write_proc = zcrypt_status_write;
diff --git a/drivers/usb/gadget/rndis.c b/drivers/usb/gadget/rndis.c
index 408c3380d602..6ec8cf1a3ccb 100644
--- a/drivers/usb/gadget/rndis.c
+++ b/drivers/usb/gadget/rndis.c
@@ -1419,7 +1419,6 @@ int __devinit rndis_init (void)
                         return -EIO;
                 }
 
-                rndis_connect_state [i]->nlink = 1;
                 rndis_connect_state [i]->write_proc = rndis_proc_write;
                 rndis_connect_state [i]->read_proc = rndis_proc_read;
                 rndis_connect_state [i]->data = (void *)