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 zhuohao@chromium.org Signed-off-by: Max Chang changmax@chromium.org

I guess this is the v2

Acked-by: Pierre-Louis Bossart pierre-louis.bossart@linux.intel.com

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 = &ctx->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, &ctx->kabylake_headset);