[PATCH 2/4] alsa-ucm: Don't probe UCM

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



In case of UCM, it usually targets a very specific hardware. The
person writing the UCM file for that hardware knows what (s)he is
doing, hopefully.

Does it harm to probe all verbs at startup? Yes, because it can
cause longer startup time, clicks, or other side effects.

(This is a difference to the normal probing mechanism, where we try
to make one profile set file fit many different sound cards, hence
we need probing there. But not here.)

Signed-off-by: David Henningsson <david.henningsson at canonical.com>
---
 src/modules/alsa/alsa-ucm.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/modules/alsa/alsa-ucm.c b/src/modules/alsa/alsa-ucm.c
index 81d0aeb..ab23259 100644
--- a/src/modules/alsa/alsa-ucm.c
+++ b/src/modules/alsa/alsa-ucm.c
@@ -1367,6 +1367,7 @@ static int ucm_create_profile(
     return 0;
 }
 
+#if 0
 static snd_pcm_t* mapping_open_pcm(pa_alsa_ucm_config *ucm, pa_alsa_mapping *m, int mode) {
     snd_pcm_t* pcm;
     pa_sample_spec try_ss = ucm->core->default_sample_spec;
@@ -1514,6 +1515,7 @@ static void ucm_probe_profile_set(pa_alsa_ucm_config *ucm, pa_alsa_profile_set *
 
     pa_alsa_profile_set_drop_unsupported(ps);
 }
+#endif
 
 pa_alsa_profile_set* pa_alsa_ucm_add_profile_set(pa_alsa_ucm_config *ucm, pa_channel_map *default_channel_map) {
     pa_alsa_ucm_verb *verb;
@@ -1539,7 +1541,7 @@ pa_alsa_profile_set* pa_alsa_ucm_add_profile_set(pa_alsa_ucm_config *ucm, pa_cha
         ucm_create_profile(ucm, ps, verb, verb_name, verb_desc);
     }
 
-    ucm_probe_profile_set(ucm, ps);
+/*    ucm_probe_profile_set(ucm, ps); */
     ps->probed = true;
 
     return ps;
-- 
1.7.9.5



[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux