volumio/volumio3-os

USB Card Ordering does not seem to be honored

Closed this issue · 4 comments

Here we are writing the alsa ordering for USB
https://github.com/volumio/volumio3-os/blob/master/scripts/volumio/volumioconfig.sh#L374

However, on my PI on the OS, there's no trace of the text we are echoing in /etc/modprobe.d/alsa-base.conf

@ashthespy

I would guess that something is overwriting said configuration during package installation. So it would seem it can't be done at the base rootfs level, but needs to be done at the device image creation level.

Ideally we should collect such things and run another volumioconfig at the device level..

Yeah I spotted the mistake: we were overwriting it from configure:
b60d77e

But since some devices require adding some rules to it, we shall do either:

  • Collect all the mods in the same files
  • echo after configure has been launched...

But since some devices require adding some rules to it, we shall do either:

* Collect all the mods in the same files

It would be great if we stream line this -- I had an issue open in Build about the different places/methods configurations are added. It has grown, and we should tame it ;-)

EDIT: volumio/Build#429

Yeah I spotted the mistake: we were overwriting it from configure:
b60d77e

Could we switch to the other option - write the file in volumioconfig.sh

## Set USB card ordering
#https://alsa.opensrc.org/Usb-audio
cat <<-EOF >>/etc/modprobe.d/alsa-base.conf
#Tuning USB devices for minimal latencies
options snd-usb-audio nrpacks=1
# USB DACs will have device number 5 in whole Volumio device range
options snd-usb-audio index=5
EOF

My aim would be to slowly reduce the number of files copied in from the old rootfs dump that is in /volumio and gradually replace it with a "single source of truth" -- which are these scripts that write the configurations. Makes it much more traceable IMO..

What do you think?

PS: Should check if the rest of the options in that file are current for Buster - base packages may be adding other options that are getting overwritten by this (Jessie) version?