Hi, Patch prefix should have been "PATCH alsa-ucm-conf", I have fixed this in my git config now, sorry about that. Regards, Hans On 12/6/20 1:47 PM, Hans de Goede wrote: > Fix the Speaker define / var being changed from "Speaker" to "MonoSpeaker" > not having any effect because the variable gets expanded before the if > making the change is interpreted. > > Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> > --- > ucm2/chtnau8824/HiFi.conf | 15 +++++++++++---- > 1 file changed, 11 insertions(+), 4 deletions(-) > > diff --git a/ucm2/chtnau8824/HiFi.conf b/ucm2/chtnau8824/HiFi.conf > index 3ccd6b1..d7a5f63 100644 > --- a/ucm2/chtnau8824/HiFi.conf > +++ b/ucm2/chtnau8824/HiFi.conf > @@ -34,8 +34,15 @@ SectionVerb { > } > } > > -Include.spk.File "/codecs/nau8824/${var:Speaker}.conf" > -Include.hp.File "/codecs/nau8824/HeadPhones.conf" > +# The includes using $vars in there path must be conditional otherwise the > +# $var gets expanded before the other If-s above can change the vars. > +If.cfg-includes { > + Condition { Type String Empty "" } > + True { > + Include.spk.File "/codecs/nau8824/${var:Speaker}.conf" > + Include.hp.File "/codecs/nau8824/HeadPhones.conf" > > -Include.mic.File "/codecs/nau8824/InternalMic.conf" > -Include.hsmic.File "/codecs/nau8824/HeadsetMic.conf" > + Include.mic.File "/codecs/nau8824/InternalMic.conf" > + Include.hsmic.File "/codecs/nau8824/HeadsetMic.conf" > + } > +} >