ALSA: ice1724 - Support for multiple external clock types

* Support for customization of the external clock names
* Adding hooks to playback_pro_open and capture_pro_open, allowing e.g.
  limiting available stream rates to a single value when the external
  clock rate is detected

Signed-off-by: Pavel Hofman <pavel.hofman@ivitera.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>

1 files changed, 45 insertions, 13 deletions

diff --git a/sound/pci/ice1712/ice1724.c b/sound/pci/ice1712/ice1724.c
index 514e15385f7a..3f11195b2631 100644
--- a/sound/pci/ice1712/ice1724.c
+++ b/sound/pci/ice1712/ice1724.c
@@ -104,6 +104,8 @@ static int PRO_RATE_LOCKED;
 static int PRO_RATE_RESET = 1;
 static unsigned int PRO_RATE_DEFAULT = 44100;
 
+static char *ext_clock_names[1] = { "IEC958 In" };
+
 /*
  *  Basic I/O
  */
@@ -1042,6 +1044,8 @@ static int snd_vt1724_playback_pro_open(struct snd_pcm_substream *substream)
                                    VT1724_BUFFER_ALIGN);
         snd_pcm_hw_constraint_step(runtime, 0, SNDRV_PCM_HW_PARAM_BUFFER_BYTES,
                                    VT1724_BUFFER_ALIGN);
+        if (ice->pro_open)
+                ice->pro_open(ice, substream);
         return 0;
 }
 
@@ -1060,6 +1064,8 @@ static int snd_vt1724_capture_pro_open(struct snd_pcm_substream *substream)
                                    VT1724_BUFFER_ALIGN);
         snd_pcm_hw_constraint_step(runtime, 0, SNDRV_PCM_HW_PARAM_BUFFER_BYTES,
                                    VT1724_BUFFER_ALIGN);
+        if (ice->pro_open)
+                ice->pro_open(ice, substream);
         return 0;
 }
 
@@ -1813,15 +1819,21 @@ static int snd_vt1724_pro_internal_clock_info(struct snd_kcontrol *kcontrol,
                                               struct snd_ctl_elem_info *uinfo)
 {
         struct snd_ice1712 *ice = snd_kcontrol_chip(kcontrol);
-
+        int hw_rates_count = ice->hw_rates->count;
         uinfo->type = SNDRV_CTL_ELEM_TYPE_ENUMERATED;
         uinfo->count = 1;
-        uinfo->value.enumerated.items = ice->hw_rates->count + 1;
+
+        uinfo->value.enumerated.items = hw_rates_count + ice->ext_clock_count;
+        /* upper limit - keep at top */
         if (uinfo->value.enumerated.item >= uinfo->value.enumerated.items)
                 uinfo->value.enumerated.item = uinfo->value.enumerated.items - 1;
-        if (uinfo->value.enumerated.item == uinfo->value.enumerated.items - 1)
-                strcpy(uinfo->value.enumerated.name, "IEC958 Input");
+        if (uinfo->value.enumerated.item >= hw_rates_count)
+                /* ext_clock items */
+                strcpy(uinfo->value.enumerated.name,
+                                ice->ext_clock_names[
+                                uinfo->value.enumerated.item - hw_rates_count]);
         else
+                /* int clock items */
                 sprintf(uinfo->value.enumerated.name, "%d",
                         ice->hw_rates->list[uinfo->value.enumerated.item]);
         return 0;
@@ -1835,7 +1847,8 @@ static int snd_vt1724_pro_internal_clock_get(struct snd_kcontrol *kcontrol,
 
         spin_lock_irq(&ice->reg_lock);
         if (ice->is_spdif_master(ice)) {
-                ucontrol->value.enumerated.item[0] = ice->hw_rates->count;
+                ucontrol->value.enumerated.item[0] = ice->hw_rates->count +
+                        ice->get_spdif_master_type(ice);
         } else {
                 rate = ice->get_rate(ice);
                 ucontrol->value.enumerated.item[0] = 0;
@@ -1850,8 +1863,14 @@ static int snd_vt1724_pro_internal_clock_get(struct snd_kcontrol *kcontrol,
         return 0;
 }
 
+static int stdclock_get_spdif_master_type(struct snd_ice1712 *ice)
+{
+        /* standard external clock - only single type - SPDIF IN */
+        return 0;
+}
+
 /* setting clock to external - SPDIF */
-static void stdclock_set_spdif_clock(struct snd_ice1712 *ice)
+static int stdclock_set_spdif_clock(struct snd_ice1712 *ice, int type)
 {
         unsigned char oval;
         unsigned char i2s_oval;
@@ -1860,27 +1879,30 @@ static void stdclock_set_spdif_clock(struct snd_ice1712 *ice)
         /* setting 256fs */
         i2s_oval = inb(ICEMT1724(ice, I2S_FORMAT));
         outb(i2s_oval & ~VT1724_MT_I2S_MCLK_128X, ICEMT1724(ice, I2S_FORMAT));
+        return 0;
 }
 
+
 static int snd_vt1724_pro_internal_clock_put(struct snd_kcontrol *kcontrol,
                                              struct snd_ctl_elem_value *ucontrol)
 {
         struct snd_ice1712 *ice = snd_kcontrol_chip(kcontrol);
         unsigned int old_rate, new_rate;
         unsigned int item = ucontrol->value.enumerated.item[0];
-        unsigned int spdif = ice->hw_rates->count;
+        unsigned int first_ext_clock = ice->hw_rates->count;
 
-        if (item > spdif)
+        if (item >  first_ext_clock + ice->ext_clock_count - 1)
                 return -EINVAL;
 
+        /* if rate = 0 => external clock */
         spin_lock_irq(&ice->reg_lock);
         if (ice->is_spdif_master(ice))
                 old_rate = 0;
         else
                 old_rate = ice->get_rate(ice);
-        if (item == spdif) {
-                /* switching to external clock via SPDIF */
-                ice->set_spdif_clock(ice);
+        if (item >= first_ext_clock) {
+                /* switching to external clock */
+                ice->set_spdif_clock(ice, item - first_ext_clock);
                 new_rate = 0;
         } else {
                 /* internal on-card clock */
@@ -1892,7 +1914,7 @@ static int snd_vt1724_pro_internal_clock_put(struct snd_kcontrol *kcontrol,
         }
         spin_unlock_irq(&ice->reg_lock);
 
-        /* the first reset to the SPDIF master mode? */
+        /* the first switch to the ext. clock mode? */
         if (old_rate != new_rate && !new_rate) {
                 /* notify akm chips as well */
                 unsigned int i;
@@ -2550,6 +2572,9 @@ static int __devinit snd_vt1724_probe(struct pci_dev *pci,
                 return err;
         }
 
+        /* field init before calling chip_init */
+        ice->ext_clock_count = 0;
+
         for (tbl = card_tables; *tbl; tbl++) {
                 for (c = *tbl; c->subvendor; c++) {
                         if (c->subvendor == ice->eeprom.subvendor) {
@@ -2588,6 +2613,13 @@ __found:
                 ice->set_mclk = stdclock_set_mclk;
         if (!ice->set_spdif_clock)
                 ice->set_spdif_clock = stdclock_set_spdif_clock;
+        if (!ice->get_spdif_master_type)
+                ice->get_spdif_master_type = stdclock_get_spdif_master_type;
+        if (!ice->ext_clock_names)
+                ice->ext_clock_names = ext_clock_names;
+        if (!ice->ext_clock_count)
+                ice->ext_clock_count = ARRAY_SIZE(ext_clock_names);
+
         if (!ice->hw_rates)
                 set_std_hw_rates(ice);
 
@@ -2747,7 +2779,7 @@ static int snd_vt1724_resume(struct pci_dev *pci)
 
         if (ice->pm_saved_is_spdif_master) {
                 /* switching to external clock via SPDIF */
-                ice->set_spdif_clock(ice);
+                ice->set_spdif_clock(ice, 0);
         } else {
                 /* internal on-card clock */
                 snd_vt1724_set_pro_rate(ice, ice->pro_rate_default, 1);