A new update was posted to me through email a few hours ago. > This seems relatively safe from regressions: the only thing that is now > called for all cards is snd_use_case_mgr_open. It would be > preferable if this was proactively selected, e g through a udev rule or > through a module parameter passed on through module- > udev-detect to the alsa card module.
This has not changed in the version posted, comment remains. > - It would be great if all the new ucm functionality was in a separate pair > of .c and .h files. That would make it easier for us to pick > up patches to the alsa-card.c file from upstream. I also believe upstream > would appreciate that separation. This has improved. > - pa_log is used extensively, should be pa_log_debug, pa_log_info to avoid annoying syslog messages. This is ok now although it seems like every pa_log has just changed to pa_log_info. > For the jack detection code, that's currently not finished. The current code doesn't add anything of value to the end user, It is unclear to me whether the new version adds value or not. Some things might have improved, but there is still no card matching, and there will still be calls to ucm functions even for non ucm cards. > that part is more invasive to current code and so if we decide to merge it I > strongly recommend to keep it disabled by default and > make a module parameter to module-udev-detect to enable jack detection > explicitly. This has not been considered, recommend not to take the jack detection part for now. In addition, the new version has three new patches about "namereg_fail", without any explanation as of what they would solve. Summary: UCM profile patches (OK): 0001-alsa-make-visible-profile-functions.patch 0002-alsa-add-proplist-ucm-data.patch 0003-alsa-add-ucm-data-structures-and-calls.patch 0004-alsa-add-support-for-ucm-profiles.patch Namereg patches (Not OK - No reason given why this is needed or what it would improve): 0001-alsa-sink-source-Use-the-namereg_fail-module-argumen.patch 0001-module-alsa-card-New-argument-namereg_fail.patch 0001-module-udev-detect-When-loading-module-alsa-card-use.patch Jack detection patches (Not OK - still not finished and too invasive at this part of the cycle): 0005-alsa-add-jack-detection-support.patch 0006-alsa-add-hook-calls-for-jack-insertion-and-removal.patch 0007-alsa-load-jack-detection-module.patch Might be needed for compilation on our current version of PulseAudio (OK if needed): UCM-decibel_fixes-compilation-error-on-PA-0-9-22.patch UCM-undefined-references-pa_xxx.patch -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/746023 Title: No sound on omap4 -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
