-----Original Message----- From: Pierre-Louis Bossart [mailto:pierre-louis.bossart@linux.intel.com] Sent: Thursday, April 30, 2015 8:40 AM To: Fang, Yang A; Koul, Vinod Cc: alsa-devel@alsa-project.org; Jain, Praveen K; Diwakar, Praveen; lgirdwood@gmail.com; Iriawan, Denny; broonie@kernel.org; kevin.strasser@linux.intel.com; dgreid@chromium.org Subject: Re: [alsa-devel] [PATCH 2/4] ASoC: Intel: Add Cherrytrail & Braswell machine driver cht_bsw_max98090
- ret = snd_soc_card_jack_new(runtime->card, "Headset Jack",
SND_JACK_HEADPHONE |SND_JACK_MICROPHONE |
SND_JACK_BTN_0 | SND_JACK_BTN_1 |SND_JACK_BTN_2 | SND_JACK_BTN_3, jack,NULL, 0);
Do you really support all those buttons?
TI driver supports 4 buttons sound/soc/codecs/ts3a227e.c google made 4 buttons headsets
Not every codec is capable of handling 4 buttons. That TI codec was made specifically to detect 4 different levels of impedance. Looking at the Max98090, I see support for a single button press (change in the status of JKSNS). what makes you think you can handle 4 buttons in the codec hardware?
This machine driver will also probe TI driver as an aux device. So Ti driver will handle Buttons event .
Thanks, Yang