the-modem-distro / pinephone_modem_sdk

Pinephone Modem SDK: Tools to build your own bootloader, kernel and rootfs
GNU General Public License v3.0
591 stars 63 forks source link

alsa configure file name and location may vary: user should be warned #211

Closed broukema closed 8 months ago

broukema commented 11 months ago

The recommendation at https://github.com/the-modem-distro/pinephone_modem_sdk/blob/kirkstone/docs/SETTINGS.md#pinephone to modify VoiceCall.conf for the Pinephone requires some guesswork to interpret in Mobian systems with these sorts of ALSA configure files from the package alsa-ucm-conf, such as Mobian/trixie.

(i) location: Currently there are three .conf files in https://salsa.debian.org/alsa-team/alsa-ucm-conf/-/tree/debian/master/ucm2/Allwinner/A64/PinePhone/ that are placed in /usr/share/alsa/ucm2/Allwinner/A64/PinePhone/.

(ii) file: The file that has AIF2 DAC Playback Volume and AIF2 ADC Capture Volume is PinePhone.conf, not VoiceCall.conf.

Side comment: It's not yet clear to me if making those two changes is enough. It seems that

SUGGESTED FIX: In any case, https://github.com/the-modem-distro/pinephone_modem_sdk/blob/kirkstone/docs/SETTINGS.md#pinephone should be updated to (i) warn the user that the specific location of the file might be different within the /usr/share/alsa/ucm2/ tree, (ii) the name of the file might differ, and (iii) the user might have to reboot for the change to take effect.

More info: https://tracker.debian.org/pkg/alsa-ucm-conf

broukema commented 11 months ago

The file location may be a recent change from Mobian/bookworm to Mobian/trixie.

broukema commented 8 months ago

Closed by merge #226 :).