ALSA: i2c: Clean up with new procfs helpers
authorTakashi Iwai <tiwai@suse.de>
Mon, 4 Feb 2019 15:00:54 +0000 (16:00 +0100)
committerTakashi Iwai <tiwai@suse.de>
Wed, 6 Feb 2019 17:11:56 +0000 (18:11 +0100)
Simplify the proc fs creation code with new helper functions,
snd_card_ro_proc_new() and snd_card_rw_proc_new().
Just a code refactoring and no functional changes.

Reviewed-by: Jaroslav Kysela <perex@perex.cz>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/i2c/other/ak4113.c
sound/i2c/other/ak4114.c
sound/i2c/other/ak4xxx-adda.c

index 4099e60..573599d 100644 (file)
@@ -492,9 +492,8 @@ static void snd_ak4113_proc_regs_read(struct snd_info_entry *entry,
 
 static void snd_ak4113_proc_init(struct ak4113 *ak4113)
 {
-       struct snd_info_entry *entry;
-       if (!snd_card_proc_new(ak4113->card, "ak4113", &entry))
-               snd_info_set_text_ops(entry, ak4113, snd_ak4113_proc_regs_read);
+       snd_card_ro_proc_new(ak4113->card, "ak4113", ak4113,
+                            snd_ak4113_proc_regs_read);
 }
 
 int snd_ak4113_build(struct ak4113 *ak4113,
index 7fb1aeb..76afb97 100644 (file)
@@ -465,9 +465,8 @@ static void snd_ak4114_proc_regs_read(struct snd_info_entry *entry,
 
 static void snd_ak4114_proc_init(struct ak4114 *ak4114)
 {
-       struct snd_info_entry *entry;
-       if (!snd_card_proc_new(ak4114->card, "ak4114", &entry))
-               snd_info_set_text_ops(entry, ak4114, snd_ak4114_proc_regs_read);
+       snd_card_ro_proc_new(ak4114->card, "ak4114", ak4114,
+                            snd_ak4114_proc_regs_read);
 }
 
 int snd_ak4114_build(struct ak4114 *ak4114,
index 7f2761a..62a6c5f 100644 (file)
@@ -875,13 +875,7 @@ static void proc_regs_read(struct snd_info_entry *entry,
 
 static int proc_init(struct snd_akm4xxx *ak)
 {
-       struct snd_info_entry *entry;
-       int err;
-       err = snd_card_proc_new(ak->card, ak->name, &entry);
-       if (err < 0)
-               return err;
-       snd_info_set_text_ops(entry, ak, proc_regs_read);
-       return 0;
+       return snd_card_ro_proc_new(ak->card, ak->name, ak, proc_regs_read);
 }
 
 int snd_akm4xxx_build_controls(struct snd_akm4xxx *ak)