Re: [alsa-devel] [PATCH] ASoC: Intel: kbl_da7219_max98927: fix the audio jack button remapping

2018-12-10 Thread Pierre-Louis Bossart



On 12/9/18 10:32 PM, Zhuohao Lee wrote:

 From the da7219 spec, the button A, B, C and D are remapped to
0, 1, 2 and 3 respectively where button A is KEY_PLAYPAUSE,
B is KEY_VOLUMEUP, C is KEY_VOLUMEDOWN and D is KEY_VOICECOMMAND.

Signed-off-by: Zhuohao Lee 
Signed-off-by: Max Chang  


I guess this is the v2

Acked-by: Pierre-Louis Bossart 

As discussed in the v1 thread, we'll need to align other DA7219-based 
devices to use the same mapping.




---
  sound/soc/intel/boards/kbl_da7219_max98927.c | 6 +++---
  1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/sound/soc/intel/boards/kbl_da7219_max98927.c 
b/sound/soc/intel/boards/kbl_da7219_max98927.c
index 3fa1c3ca6d37..d3531f0eb357 100644
--- a/sound/soc/intel/boards/kbl_da7219_max98927.c
+++ b/sound/soc/intel/boards/kbl_da7219_max98927.c
@@ -262,9 +262,9 @@ static int kabylake_da7219_codec_init(struct 
snd_soc_pcm_runtime *rtd)
  
  	jack = >kabylake_headset;

snd_jack_set_key(jack->jack, SND_JACK_BTN_0, KEY_PLAYPAUSE);
-   snd_jack_set_key(jack->jack, SND_JACK_BTN_1, KEY_VOICECOMMAND);
-   snd_jack_set_key(jack->jack, SND_JACK_BTN_2, KEY_VOLUMEUP);
-   snd_jack_set_key(jack->jack, SND_JACK_BTN_3, KEY_VOLUMEDOWN);
+   snd_jack_set_key(jack->jack, SND_JACK_BTN_1, KEY_VOLUMEUP);
+   snd_jack_set_key(jack->jack, SND_JACK_BTN_2, KEY_VOLUMEDOWN);
+   snd_jack_set_key(jack->jack, SND_JACK_BTN_3, KEY_VOICECOMMAND);
  
  	da7219_aad_jack_det(component, >kabylake_headset);
  


[PATCH] ASoC: Intel: kbl_da7219_max98927: fix the audio jack button remapping

2018-12-09 Thread Zhuohao Lee
>From the da7219 spec, the button A, B, C and D are remapped to
0, 1, 2 and 3 respectively where button A is KEY_PLAYPAUSE,
B is KEY_VOLUMEUP, C is KEY_VOLUMEDOWN and D is KEY_VOICECOMMAND.

Signed-off-by: Zhuohao Lee 
Signed-off-by: Max Chang  
---
 sound/soc/intel/boards/kbl_da7219_max98927.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/sound/soc/intel/boards/kbl_da7219_max98927.c 
b/sound/soc/intel/boards/kbl_da7219_max98927.c
index 3fa1c3ca6d37..d3531f0eb357 100644
--- a/sound/soc/intel/boards/kbl_da7219_max98927.c
+++ b/sound/soc/intel/boards/kbl_da7219_max98927.c
@@ -262,9 +262,9 @@ static int kabylake_da7219_codec_init(struct 
snd_soc_pcm_runtime *rtd)
 
jack = >kabylake_headset;
snd_jack_set_key(jack->jack, SND_JACK_BTN_0, KEY_PLAYPAUSE);
-   snd_jack_set_key(jack->jack, SND_JACK_BTN_1, KEY_VOICECOMMAND);
-   snd_jack_set_key(jack->jack, SND_JACK_BTN_2, KEY_VOLUMEUP);
-   snd_jack_set_key(jack->jack, SND_JACK_BTN_3, KEY_VOLUMEDOWN);
+   snd_jack_set_key(jack->jack, SND_JACK_BTN_1, KEY_VOLUMEUP);
+   snd_jack_set_key(jack->jack, SND_JACK_BTN_2, KEY_VOLUMEDOWN);
+   snd_jack_set_key(jack->jack, SND_JACK_BTN_3, KEY_VOICECOMMAND);
 
da7219_aad_jack_det(component, >kabylake_headset);
 
-- 
2.20.0.rc2.403.gdbc3b29805-goog