* Re: [alsa-devel] [PATCH v1 5/6] ASoC: codecs: msm8916-wcd-analog: add MBHC support
@ 2017-07-26 19:35 Damien Riegel
2017-07-27 10:03 ` Srinivas Kandagatla
0 siblings, 1 reply; 2+ messages in thread
From: Damien Riegel @ 2017-07-26 19:35 UTC (permalink / raw)
To: Srinivas Kandagatla; +Cc: alsa-devel, linux-kernel
[Sorry for breaking the thread, but I don't have the initial message to
reply to.]
I tried your patchset and faced some issues regarding removal detection:
the driver detects headphone and microphone removal as soon as I press a
button on the headset.
evtest logs:
[Headset insertion]
Event: time 10207.841157, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 1
Event: time 10207.841157, -------------- SYN_REPORT ------------
Event: time 10208.039105, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 0
Event: time 10208.039105, -------------- SYN_REPORT ------------
Event: time 10208.075519, type 5 (EV_SW), code 2 (SW_HEADPHONE_INSERT), value 1
Event: time 10208.075519, type 5 (EV_SW), code 4 (SW_MICROPHONE_INSERT), value 1
Event: time 10208.075519, -------------- SYN_REPORT ------------
[Pressing Volume up Button]
Event: time 10210.590403, type 1 (EV_KEY), code 115 (KEY_VOLUMEUP), value 1
Event: time 10210.590403, type 5 (EV_SW), code 2 (SW_HEADPHONE_INSERT), value 0
Event: time 10210.590403, type 5 (EV_SW), code 4 (SW_MICROPHONE_INSERT), value 0
Event: time 10210.590403, -------------- SYN_REPORT ------------
Event: time 10210.719560, type 1 (EV_KEY), code 115 (KEY_VOLUMEUP), value 0
Event: time 10210.719560, -------------- SYN_REPORT ------------
[Unplugging]
Event: time 10221.926156, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 1
Event: time 10221.926156, -------------- SYN_REPORT ------------
Event: time 10222.026466, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 0
Event: time 10222.026466, -------------- SYN_REPORT ------------
Also, the KEY_MEDIA is wrongly reported but I don't see that every time,
I think it depends on how fast I plug in the headset. Anyway, I think
that can easily be filtered out by testing the presence of the
microphone.
Regards,
--
Damien
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [alsa-devel] [PATCH v1 5/6] ASoC: codecs: msm8916-wcd-analog: add MBHC support
2017-07-26 19:35 [alsa-devel] [PATCH v1 5/6] ASoC: codecs: msm8916-wcd-analog: add MBHC support Damien Riegel
@ 2017-07-27 10:03 ` Srinivas Kandagatla
0 siblings, 0 replies; 2+ messages in thread
From: Srinivas Kandagatla @ 2017-07-27 10:03 UTC (permalink / raw)
To: Damien Riegel, Srinivas Kandagatla; +Cc: alsa-devel, linux-kernel
Thanks Damien, for testing the patchset.
On 26/07/17 20:35, Damien Riegel wrote:
> [Sorry for breaking the thread, but I don't have the initial message to
> reply to.]
>
> I tried your patchset and faced some issues regarding removal detection:
> the driver detects headphone and microphone removal as soon as I press a
> button on the headset.
>
I will try to reproduce this on my end and hopefully fix it in next version.
--srini
> evtest logs:
>
> [Headset insertion]
> Event: time 10207.841157, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 1
> Event: time 10207.841157, -------------- SYN_REPORT ------------
> Event: time 10208.039105, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 0
> Event: time 10208.039105, -------------- SYN_REPORT ------------
> Event: time 10208.075519, type 5 (EV_SW), code 2 (SW_HEADPHONE_INSERT), value 1
> Event: time 10208.075519, type 5 (EV_SW), code 4 (SW_MICROPHONE_INSERT), value 1
> Event: time 10208.075519, -------------- SYN_REPORT ------------
> [Pressing Volume up Button]
> Event: time 10210.590403, type 1 (EV_KEY), code 115 (KEY_VOLUMEUP), value 1
> Event: time 10210.590403, type 5 (EV_SW), code 2 (SW_HEADPHONE_INSERT), value 0
> Event: time 10210.590403, type 5 (EV_SW), code 4 (SW_MICROPHONE_INSERT), value 0
> Event: time 10210.590403, -------------- SYN_REPORT ------------
> Event: time 10210.719560, type 1 (EV_KEY), code 115 (KEY_VOLUMEUP), value 0
> Event: time 10210.719560, -------------- SYN_REPORT ------------
> [Unplugging]
> Event: time 10221.926156, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 1
> Event: time 10221.926156, -------------- SYN_REPORT ------------
> Event: time 10222.026466, type 1 (EV_KEY), code 226 (KEY_MEDIA), value 0
> Event: time 10222.026466, -------------- SYN_REPORT ------------
>
>
> Also, the KEY_MEDIA is wrongly reported but I don't see that every time,
> I think it depends on how fast I plug in the headset. Anyway, I think
> that can easily be filtered out by testing the presence of the
> microphone.
>
>
> Regards,
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-07-27 10:03 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-26 19:35 [alsa-devel] [PATCH v1 5/6] ASoC: codecs: msm8916-wcd-analog: add MBHC support Damien Riegel
2017-07-27 10:03 ` Srinivas Kandagatla
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).