3 files changed, 28 insertions, 32 deletions

diff --git a/sound/core/info.c b/sound/core/info.c
index 5bb97e7d325a..3c9bd6b10a96 100644
--- a/sound/core/info.c
+++ b/sound/core/info.c
@@ -153,13 +153,6 @@ EXPORT_SYMBOL(snd_seq_root);
 struct snd_info_entry *snd_oss_root;
 #endif
 
-static void snd_remove_proc_entry(struct proc_dir_entry *parent,
-                                  struct proc_dir_entry *de)
-{
-        if (de)
-                remove_proc_entry(de->name, parent);
-}
-
 static loff_t snd_info_entry_llseek(struct file *file, loff_t offset, int orig)
 {
         struct snd_info_private_data *data;
@@ -310,12 +303,10 @@ static int snd_info_entry_open(struct inode *inode, struct file *file)
         struct snd_info_entry *entry;
         struct snd_info_private_data *data;
         struct snd_info_buffer *buffer;
-        struct proc_dir_entry *p;
         int mode, err;
 
         mutex_lock(&info_mutex);
-        p = PDE(inode);
-        entry = p == NULL ? NULL : (struct snd_info_entry *)p->data;
+        entry = PDE_DATA(inode);
         if (entry == NULL || ! entry->p) {
                 mutex_unlock(&info_mutex);
                 return -ENODEV;
@@ -582,7 +573,7 @@ int __exit snd_info_done(void)
 #ifdef CONFIG_SND_OSSEMUL
                 snd_info_free_entry(snd_oss_root);
 #endif
-                snd_remove_proc_entry(NULL, snd_proc_root);
+                proc_remove(snd_proc_root);
         }
         return 0;
 }
@@ -644,7 +635,7 @@ void snd_info_card_id_change(struct snd_card *card)
 {
         mutex_lock(&info_mutex);
         if (card->proc_root_link) {
-                snd_remove_proc_entry(snd_proc_root, card->proc_root_link);
+                proc_remove(card->proc_root_link);
                 card->proc_root_link = NULL;
         }
         if (strcmp(card->id, card->proc_root->name))
@@ -663,10 +654,8 @@ void snd_info_card_disconnect(struct snd_card *card)
         if (!card)
                 return;
         mutex_lock(&info_mutex);
-        if (card->proc_root_link) {
-                snd_remove_proc_entry(snd_proc_root, card->proc_root_link);
-                card->proc_root_link = NULL;
-        }
+        proc_remove(card->proc_root_link);
+        card->proc_root_link = NULL;
         if (card->proc_root)
                 snd_info_disconnect(card->proc_root);
         mutex_unlock(&info_mutex);
@@ -858,7 +847,7 @@ static void snd_info_disconnect(struct snd_info_entry *entry)
         list_del_init(&entry->list);
         root = entry->parent == NULL ? snd_proc_root : entry->parent->p;
         snd_BUG_ON(!root);
-        snd_remove_proc_entry(root, entry->p);
+        proc_remove(entry->p);
         entry->p = NULL;
 }
 
@@ -959,15 +948,21 @@ int snd_info_register(struct snd_info_entry * entry)
                 return -ENXIO;
         root = entry->parent == NULL ? snd_proc_root : entry->parent->p;
         mutex_lock(&info_mutex);
-        p = create_proc_entry(entry->name, entry->mode, root);
-        if (!p) {
-                mutex_unlock(&info_mutex);
-                return -ENOMEM;
+        if (S_ISDIR(entry->mode)) {
+                p = proc_mkdir_mode(entry->name, entry->mode, root);
+                if (!p) {
+                        mutex_unlock(&info_mutex);
+                        return -ENOMEM;
+                }
+        } else {
+                p = proc_create_data(entry->name, entry->mode, root,
+                                        &snd_info_entry_operations, entry);
+                if (!p) {
+                        mutex_unlock(&info_mutex);
+                        return -ENOMEM;
+                }
+                proc_set_size(p, entry->size);
         }
-        if (!S_ISDIR(entry->mode))
-                p->proc_fops = &snd_info_entry_operations;
-        p->size = entry->size;
-        p->data = entry;
         entry->p = p;
         if (entry->parent)
                 list_add_tail(&entry->list, &entry->parent->children);
diff --git a/sound/oss/dmasound/dmasound_core.c b/sound/oss/dmasound/dmasound_core.c
index c918313c2206..bac43b5b6e95 100644
--- a/sound/oss/dmasound/dmasound_core.c
+++ b/sound/oss/dmasound/dmasound_core.c
@@ -835,7 +835,7 @@ static void sq_reset(void)
         shared_resources_initialised = 0 ;
 }
 
-static int sq_fsync(struct file *filp, struct dentry *dentry)
+static int sq_fsync(void)
 {
         int rc = 0;
         int timeout = 5;
@@ -874,7 +874,7 @@ static int sq_release(struct inode *inode, struct file *file)
 
         if (file->f_mode & FMODE_WRITE) {
                 if (write_sq.busy)
-                        rc = sq_fsync(file, file->f_path.dentry);
+                        rc = sq_fsync();
 
                 sq_reset_output() ; /* make sure dma is stopped and all is quiet */
                 write_sq_release_buffers();
@@ -1025,7 +1025,7 @@ static int sq_ioctl(struct file *file, u_int cmd, u_long arg)
                 */
                 result = 0 ;
                 if (file->f_mode & FMODE_WRITE) {
-                        result = sq_fsync(file, file->f_path.dentry);
+                        result = sq_fsync();
                         sq_reset_output() ;
                 }
                 /* if we are the shared resource owner then release them */
diff --git a/sound/sound_firmware.c b/sound/sound_firmware.c
index e14903468051..b155137ee312 100644
--- a/sound/sound_firmware.c
+++ b/sound/sound_firmware.c
@@ -1,6 +1,7 @@
 #include <linux/vmalloc.h>
 #include <linux/module.h>
 #include <linux/fs.h>
+#include <linux/file.h>
 #include <linux/mm.h>
 #include <linux/sched.h>
 #include <asm/uaccess.h>
@@ -23,14 +24,14 @@ static int do_mod_firmware_load(const char *fn, char **fp)
         if (l <= 0 || l > 131072)
         {
                 printk(KERN_INFO "Invalid firmware '%s'\n", fn);
-                filp_close(filp, NULL);
+                fput(filp);
                 return 0;
         }
         dp = vmalloc(l);
         if (dp == NULL)
         {
                 printk(KERN_INFO "Out of memory loading '%s'.\n", fn);
-                filp_close(filp, NULL);
+                fput(filp);
                 return 0;
         }
         pos = 0;
@@ -38,10 +39,10 @@ static int do_mod_firmware_load(const char *fn, char **fp)
         {
                 printk(KERN_INFO "Failed to read '%s'.\n", fn);
                 vfree(dp);
-                filp_close(filp, NULL);
+                fput(filp);
                 return 0;
         }
-        filp_close(filp, NULL);
+        fput(filp);
         *fp = dp;
         return (int) l;
 }