Intel/sof-essx8336: Fix HiFi.conf

1. Switch Headphone on/off in Headphones enable/disable sequence.  This
   should be obvious.
2. "Headset Switch" is not recognized and triggers errors running
   "alsaucm -c hw:0 set _verb Hifi".  Change it to "Headset Mic Switch".
   And in the disable sequence we should turn it off, not on.

Fixes: https://github.com/alsa-project/alsa-ucm-conf/pull/328
Signed-off-by: Xi Ruoyao <xry111@xry111.site>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>

diff --git a/ucm2/Intel/sof-essx8336/HiFi.conf b/ucm2/Intel/sof-essx8336/HiFi.conf
index 974a80aafb83e14b491f1dbb70cf1dccf894afe6..2be5aa55b140300752b2cd9ddd23d295b5da339c 100644 (file)
--- a/ucm2/Intel/sof-essx8336/HiFi.conf
+++ b/ucm2/Intel/sof-essx8336/HiFi.conf
@@ -82,6 +82,7 @@ SectionDevice."Speaker" {
        DisableSequence [
                cset "name='Speaker Switch' off"
        ]
+
        Value {
                PlaybackPriority 100
                PlaybackPCM "hw:${CardId}"
@@ -99,6 +100,14 @@ SectionDevice."Headphones" {
                "Speaker"
        ]
 
+       EnableSequence [
+               cset "name='Headphone Switch' on"
+       ]
+
+       DisableSequence [
+               cset "name='Headphone Switch' off"
+       ]
+
        Value {
                PlaybackPriority 300
                PlaybackPCM "hw:${CardId}"
@@ -123,12 +132,12 @@ SectionDevice."Headset" {
        }
 
        EnableSequence [
-               cset "name='Headset Switch' on"
+               cset "name='Headset Mic Switch' on"
                cset "name='Digital Mic Mux' 'dmic disable'"
        ]
 
        DisableSequence [
-               cset "name='Headset Switch' on"
+               cset "name='Headset Mic Switch' off"
        ]
 
        Value {