@@ -100,16 +100,18 @@ If.mono {
If.hp {
Condition { Type String Empty "" }
- True.Include.hs.File "/codecs/rt5640/HeadPhones.conf"
+ True {
+ Include.hs.File "/codecs/rt5640/HeadPhones.conf"
- SectionDevice."Headphones" {
- EnableSequence [
- cset "name='Headphone Switch' on"
- ]
+ SectionDevice."Headphones" {
+ EnableSequence [
+ cset "name='Headphone Switch' on"
+ ]
- DisableSequence [
- cset "name='Headphone Switch' off"
- ]
+ DisableSequence [
+ cset "name='Headphone Switch' off"
+ ]
+ }
}
}
@@ -178,15 +180,17 @@ If.in3 {
If.hsmic {
Condition { Type String Empty "" }
- True.Include.hsmic.File "/codecs/rt5640/HeadsetMic.conf"
+ True {
+ Include.hsmic.File "/codecs/rt5640/HeadsetMic.conf"
- SectionDevice."Headset" {
- EnableSequence [
- cset "name='Headset Mic Switch' on"
- ]
+ SectionDevice."Headset" {
+ EnableSequence [
+ cset "name='Headset Mic Switch' on"
+ ]
- DisableSequence [
- cset "name='Headset Mic Switch' off"
- ]
+ DisableSequence [
+ cset "name='Headset Mic Switch' off"
+ ]
+ }
}
}
Commit 446d0a71b47f ("codecs/rt5640: Move out BayTrail-specific pin switches") moved the setting of the 'Headphone Switch' / 'Headset Mic Switch' into "If.hp" resp "If.hsmic" blocks. But instead of making them part of the True {} block inside that If, the statements where added add the top-level of the If {} block where they are no-ops. Move them to inside the True blocks so that these Switches again get properly turned on/off. This fixes the Headphones and Headset-mic no longer working on bytcr-rt5640 devices. Cc: Dmitry Osipenko <digetx@gmail.com> Fixes: 446d0a71b47f ("codecs/rt5640: Move out BayTrail-specific pin switches") Signed-off-by: Hans de Goede <hdegoede@redhat.com> --- ucm2/bytcr-rt5640/HiFi.conf | 36 ++++++++++++++++++++---------------- 1 file changed, 20 insertions(+), 16 deletions(-)