Dne 17. 01. 21 v 17:52 Jaroslav Kysela napsal(a): > Dne 17. 01. 21 v 17:09 Hans de Goede napsal(a): >> Hi, >> >> On 12/16/20 4:38 PM, Hans de Goede wrote: >>> Move DAC Channel Source selection to Speaker/Headphones EnableSeq. >>> >>> The main reason for doing this is to make it easier to allow using >>> variables, as the Speaker/Headphones conf files are evaluated after >>> the main HiFi.conf had a chance to set them. >>> >>> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> >> >> ping? It would be nice to get this series merged for proper SOF >> support. >> >> Maybe this got confused with my other UCM chtnau8824 series which >> I send 10 days before t > > Hi, > > Thanks for the reminder. I though that this one was resolved. I don't think > that we need the first change - the evaluation for If and Include sequences is > executed before SectionVerb {} parsing. So the If...Define.RightOutputChannel > should be valid for SectionVerb...Include...File > "/codecs/nau8824/EnableSeq.conf". The 'DACR Channel Source' control seems to > require a static settings so keeping it in the verb init sequence has sense. > > I applied the second (SST variable) patch for now. It's nice cleanup. Thanks. Hans, could you test https://github.com/alsa-project/alsa-ucm-conf/pull/74 ? Thank you, Jaroslav -- Jaroslav Kysela <perex@xxxxxxxx> Linux Sound Maintainer; ALSA Project; Red Hat, Inc.