From: Xi Ruoyao <xry111@xry111.site>
Date: Thu, 22 Jun 2023 21:14:30 +0000 (+0800)
Subject: Intel/sof-essx8336: Fix HiFi.conf
X-Git-Tag: v1.2.10~13
X-Git-Url: https://git.alsa-project.org/?a=commitdiff_plain;h=226c9b7b01b5f84a094ebe3f44a64e00b245942a;p=alsa-ucm-conf.git

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 974a80a..2be5aa5 100644
--- 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 {