ALSA: rme32: Use snd_ctl_enum_info()
authorTakashi Iwai <tiwai@suse.de>
Mon, 20 Oct 2014 16:19:12 +0000 (18:19 +0200)
committerTakashi Iwai <tiwai@suse.de>
Tue, 21 Oct 2014 07:05:24 +0000 (09:05 +0200)
... and reduce the open codes.  Also add missing const to text arrays.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/rme32.c

index 4afd3ca..6c60dcd 100644 (file)
@@ -1608,30 +1608,24 @@ snd_rme32_info_inputtype_control(struct snd_kcontrol *kcontrol,
                                 struct snd_ctl_elem_info *uinfo)
 {
        struct rme32 *rme32 = snd_kcontrol_chip(kcontrol);
-       static char *texts[4] = { "Optical", "Coaxial", "Internal", "XLR" };
+       static const char * const texts[4] = {
+               "Optical", "Coaxial", "Internal", "XLR"
+       };
+       int num_items;
 
-       uinfo->type = SNDRV_CTL_ELEM_TYPE_ENUMERATED;
-       uinfo->count = 1;
        switch (rme32->pci->device) {
        case PCI_DEVICE_ID_RME_DIGI32:
        case PCI_DEVICE_ID_RME_DIGI32_8:
-               uinfo->value.enumerated.items = 3;
+               num_items = 3;
                break;
        case PCI_DEVICE_ID_RME_DIGI32_PRO:
-               uinfo->value.enumerated.items = 4;
+               num_items = 4;
                break;
        default:
                snd_BUG();
-               break;
-       }
-       if (uinfo->value.enumerated.item >
-           uinfo->value.enumerated.items - 1) {
-               uinfo->value.enumerated.item =
-                   uinfo->value.enumerated.items - 1;
+               return -EINVAL;
        }
-       strcpy(uinfo->value.enumerated.name,
-              texts[uinfo->value.enumerated.item]);
-       return 0;
+       return snd_ctl_enum_info(uinfo, 1, num_items, texts);
 }
 static int
 snd_rme32_get_inputtype_control(struct snd_kcontrol *kcontrol,
@@ -1695,20 +1689,12 @@ static int
 snd_rme32_info_clockmode_control(struct snd_kcontrol *kcontrol,
                                 struct snd_ctl_elem_info *uinfo)
 {
-       static char *texts[4] = { "AutoSync", 
+       static const char * const texts[4] = { "AutoSync",
                                  "Internal 32.0kHz", 
                                  "Internal 44.1kHz", 
                                  "Internal 48.0kHz" };
 
-       uinfo->type = SNDRV_CTL_ELEM_TYPE_ENUMERATED;
-       uinfo->count = 1;
-       uinfo->value.enumerated.items = 4;
-       if (uinfo->value.enumerated.item > 3) {
-               uinfo->value.enumerated.item = 3;
-       }
-       strcpy(uinfo->value.enumerated.name,
-              texts[uinfo->value.enumerated.item]);
-       return 0;
+       return snd_ctl_enum_info(uinfo, 1, 4, texts);
 }
 static int
 snd_rme32_get_clockmode_control(struct snd_kcontrol *kcontrol,