[ALSA] oxygen: use an array of snd_kcontrol pointers
authorClemens Ladisch <clemens@ladisch.de>
Mon, 14 Jan 2008 07:56:01 +0000 (08:56 +0100)
committerJaroslav Kysela <perex@perex.cz>
Thu, 31 Jan 2008 16:29:57 +0000 (17:29 +0100)
Use an array for the pointers to known controls so that it is easier to
add more.

Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
sound/pci/oxygen/oxygen.h
sound/pci/oxygen/oxygen_lib.c
sound/pci/oxygen/oxygen_mixer.c
sound/pci/oxygen/oxygen_pcm.c

index 248f7ed22fda009d02fbc55e09ac3606362b9f12..a10c37788059b11db071eaf6a2f3698d155e30bc 100644 (file)
 #define PCM_AC97       5
 #define PCM_COUNT      6
 
+enum {
+       CONTROL_SPDIF_PCM,
+       CONTROL_SPDIF_INPUT_BITS,
+       CONTROL_COUNT
+};
+
 #define OXYGEN_PCI_SUBID(sv, sd) \
        .vendor = PCI_VENDOR_ID_CMEDIA, \
        .device = 0x8788, \
@@ -50,8 +56,7 @@ struct oxygen {
        u32 spdif_bits;
        u32 spdif_pcm_bits;
        struct snd_pcm_substream *streams[PCM_COUNT];
-       struct snd_kcontrol *spdif_pcm_ctl;
-       struct snd_kcontrol *spdif_input_bits_ctl;
+       struct snd_kcontrol *controls[CONTROL_COUNT];
        struct work_struct spdif_input_bits_work;
 };
 
index 9cd4be2593ab158e0b94fa3a61608b758e05a859..5b77c9439c36dfc9e14a1ffe00f962d32c213e95 100644 (file)
@@ -114,7 +114,7 @@ static void oxygen_spdif_input_bits_changed(struct work_struct *work)
                }
        }
 
-       if (chip->spdif_input_bits_ctl) {
+       if (chip->controls[CONTROL_SPDIF_INPUT_BITS]) {
                spin_lock_irq(&chip->reg_lock);
                chip->interrupt_mask |= OXYGEN_INT_SPDIF_IN_CHANGE;
                oxygen_write16(chip, OXYGEN_INTERRUPT_MASK,
@@ -122,7 +122,7 @@ static void oxygen_spdif_input_bits_changed(struct work_struct *work)
                spin_unlock_irq(&chip->reg_lock);
 
                snd_ctl_notify(chip->card, SNDRV_CTL_EVENT_MASK_VALUE,
-                              &chip->spdif_input_bits_ctl->id);
+                              &chip->controls[CONTROL_SPDIF_INPUT_BITS]->id);
        }
 }
 
index ca72799bea270db9a8865b83166613005c16fd79..f7350faada1a3058b52da074cd35189c92738ede 100644 (file)
@@ -586,15 +586,22 @@ static const struct snd_kcontrol_new controls[] = {
 static void oxygen_any_ctl_free(struct snd_kcontrol *ctl)
 {
        struct oxygen *chip = ctl->private_data;
+       unsigned int i;
 
        /* I'm too lazy to write a function for each control :-) */
-       chip->spdif_pcm_ctl = NULL;
-       chip->spdif_input_bits_ctl = NULL;
+       for (i = 0; i < ARRAY_SIZE(chip->controls); ++i)
+               chip->controls[i] = NULL;
 }
 
 int oxygen_mixer_init(struct oxygen *chip)
 {
-       unsigned int i;
+       static const char *const known_ctl_names[CONTROL_COUNT] = {
+               [CONTROL_SPDIF_PCM] =
+                       SNDRV_CTL_NAME_IEC958("", PLAYBACK, PCM_STREAM),
+               [CONTROL_SPDIF_INPUT_BITS] =
+                       SNDRV_CTL_NAME_IEC958("", CAPTURE, DEFAULT),
+       };
+       unsigned int i, j;
        struct snd_kcontrol *ctl;
        int err;
 
@@ -610,15 +617,11 @@ int oxygen_mixer_init(struct oxygen *chip)
                err = snd_ctl_add(chip->card, ctl);
                if (err < 0)
                        return err;
-               if (!strcmp(ctl->id.name,
-                           SNDRV_CTL_NAME_IEC958("", PLAYBACK, PCM_STREAM))) {
-                       chip->spdif_pcm_ctl = ctl;
-                       ctl->private_free = oxygen_any_ctl_free;
-               } else if (!strcmp(ctl->id.name,
-                                SNDRV_CTL_NAME_IEC958("", CAPTURE, DEFAULT))) {
-                       chip->spdif_input_bits_ctl = ctl;
-                       ctl->private_free = oxygen_any_ctl_free;
-               }
+               for (j = 0; j < CONTROL_COUNT; ++j)
+                       if (!strcmp(ctl->id.name, known_ctl_names[j])) {
+                               chip->controls[j] = ctl;
+                               ctl->private_free = oxygen_any_ctl_free;
+                       }
        }
        return chip->model->mixer_init ? chip->model->mixer_init(chip) : 0;
 }
index d4a1d73718e360f2624fd6a373c9f20000555b40..5f67a799a034763ebf94435313cd7ee8f6b1b072 100644 (file)
@@ -209,11 +209,11 @@ static int oxygen_open(struct snd_pcm_substream *substream,
        chip->pcm_active |= 1 << channel;
        if (channel == PCM_SPDIF) {
                chip->spdif_pcm_bits = chip->spdif_bits;
-               chip->spdif_pcm_ctl->vd[0].access &=
+               chip->controls[CONTROL_SPDIF_PCM]->vd[0].access &=
                        ~SNDRV_CTL_ELEM_ACCESS_INACTIVE;
                snd_ctl_notify(chip->card, SNDRV_CTL_EVENT_MASK_VALUE |
                               SNDRV_CTL_EVENT_MASK_INFO,
-                              &chip->spdif_pcm_ctl->id);
+                              &chip->controls[CONTROL_SPDIF_PCM]->id);
        }
        mutex_unlock(&chip->mutex);
 
@@ -258,11 +258,11 @@ static int oxygen_close(struct snd_pcm_substream *substream)
        mutex_lock(&chip->mutex);
        chip->pcm_active &= ~(1 << channel);
        if (channel == PCM_SPDIF) {
-               chip->spdif_pcm_ctl->vd[0].access |=
+               chip->controls[CONTROL_SPDIF_PCM]->vd[0].access |=
                        SNDRV_CTL_ELEM_ACCESS_INACTIVE;
                snd_ctl_notify(chip->card, SNDRV_CTL_EVENT_MASK_VALUE |
                               SNDRV_CTL_EVENT_MASK_INFO,
-                              &chip->spdif_pcm_ctl->id);
+                              &chip->controls[CONTROL_SPDIF_PCM]->id);
        }
        if (channel == PCM_SPDIF || channel == PCM_MULTICH)
                oxygen_update_spdif_source(chip);