sof-hda-dsp: add workaround for speaker LED on MSI laptops

Fixes: Master Playback Switch doesn't get attached to the Mute LED
       on these two platforms. This change adresses this issue.

Also fixes 0915b27a which accidentally broke dell workaround.

Closes: https://github.com/alsa-project/alsa-ucm-conf/pull/716
Fixes: 0915b27 ("ucm2: use new SetLED macro to hide the implementation details")
Signed-off-by: Federico Lombardo <lombardo.federico.lf@gmail.com>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>

diff --git a/ucm2/Intel/sof-hda-dsp/sof-hda-dsp.conf b/ucm2/Intel/sof-hda-dsp/sof-hda-dsp.conf
index 892894ba739061f49b560451c82a78976a6ba005..c1a019a857ff07809606e8a96d91d3baf443dd0e 100644 (file)
--- a/ucm2/Intel/sof-hda-dsp/sof-hda-dsp.conf
+++ b/ucm2/Intel/sof-hda-dsp/sof-hda-dsp.conf
@@ -46,13 +46,13 @@ SectionUseCase."HiFi" {
 
 Include.hda-init.File "/HDA/init.conf"
 
-If.DellMuteLed {
+If.PlatformMuteLed {
        Condition {
-               Type String
-               Haystack "$${sys:class/leds/platform::mute/device}"
-               Needle "dell-laptop"
+               Type RegexMatch
+               Regex "(dell-laptop|msi-ec)"
+               String "$${sys:class/leds/platform::mute/device}"
        }
-       True.Macro [{ SetLED { LED="speaker" Action="detach" CtlId="Master Playback Switch" } }]
+       True.Macro [{ SetLED { LED="speaker" Action="attach" CtlId="Master Playback Switch" } }]
 }
 
 If.dmic {