Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1707959

Re: [PATCH v3 3/5] ASoC: codecs: msm8916-wcd-analog: add MBHC support

Show all headers | View raw

Hi Srinivas,

On Wed, Aug 09, 2017 at 06:49:25PM +0200, srinivas.kandagatla@linaro.org wrote:
> From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> 
> MBHC (MultiButton Headset Control) support is available in pm8921 in two
> blocks, one to detect mechanical headset insertion and removal and other
> block to support headset type detection and 5 button detection and othe
> features like impedance calculation.
> 
> This patch adds support to:
> 1> Support to NC and NO type of headset Jacks.
> 2> Mechanical insertion and detection of headset jack.
> 3> Detect a 3 pole Headphone and a 4 pole Headset.
> 4> Detect 5 buttons.
> 
> Tested it on DB410c with Audio Mezz board with 4 pole and 3 pole
> headset/headphones.

I have the same issue with this patchset, KEY_MEDIA is being reported
when unplugging a headset. I added a few traces and what I observe is
that the "button pressed" irq is fired when unplugging, but no "button
released" irq follows. To get a "button released" irq, I need to either
plug a headset, or plug-and-unplug headphones.

So basically, detection fails because we don't get the "button pressed"
irq prior to the mechanical switch irq. Do you know what could explain
the MBHC not firing the "button released" irq when unplugging headset?


Thanks,
-- 
Damien

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v3 0/5] ASoC: codecs: msm8916-wcd-analog: Add support to MBHC srinivas.kandagatla@linaro.org - 2017-08-09 18:50 +0200
  [PATCH v3 5/5] arm64: dts: apq8016-sbc: add mbhc buttons support srinivas.kandagatla@linaro.org - 2017-08-09 19:00 +0200
  [PATCH v3 3/5] ASoC: codecs: msm8916-wcd-analog: add MBHC support srinivas.kandagatla@linaro.org - 2017-08-09 19:00 +0200
    Re: [PATCH v3 3/5] ASoC: codecs: msm8916-wcd-analog: add MBHC support Damien Riegel <damien.riegel@savoirfairelinux.com> - 2017-08-09 23:20 +0200
      Re: [PATCH v3 3/5] ASoC: codecs: msm8916-wcd-analog: add MBHC support Srinivas Kandagatla <srinivas.kandagatla@linaro.org> - 2017-08-10 12:10 +0200
        Re: [PATCH v3 3/5] ASoC: codecs: msm8916-wcd-analog: add MBHC support Damien Riegel <damien.riegel@savoirfairelinux.com> - 2017-08-10 15:40 +0200
  [PATCH v3 2/5] ASoC: codecs: msm8916-wcd-analog: get micbias voltage from dt srinivas.kandagatla@linaro.org - 2017-08-09 19:00 +0200
    Re: [PATCH v3 2/5] ASoC: codecs: msm8916-wcd-analog: get micbias  voltage from dt Mark Brown <broonie@kernel.org> - 2017-08-10 17:30 +0200
      Re: [PATCH v3 2/5] ASoC: codecs: msm8916-wcd-analog: get micbias  voltage from dt Srinivas Kandagatla <srinivas.kandagatla@linaro.org> - 2017-08-11 17:20 +0200

csiph-web