lkml.org 
[lkml]   [2013]   [Jul]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH 1/4] ALSA: Added jack detection kcontrol support
    From
    Mark,

    On Fri, Jul 26, 2013 at 3:48 PM, Mark Brown <broonie@kernel.org> wrote:
    > On Fri, Jul 26, 2013 at 12:10:27PM -0700, Felipe Tonello wrote:
    >> On Fri, Jul 26, 2013 at 11:54 AM, Mark Brown <broonie@kernel.org> wrote:
    >
    >> > This isn't ideal for multi-function jacks like headsets - it will report
    >> > a single boolean value for the jack regardless of what's plugged in
    >> > meaning userpace can't do things like figure out if a headset or
    >> > headphone is present. It's probably OK for any realistic input button
    >> > since you're not going to get an input button without other things being
    >> > present.
    >
    >> The KControl for Jack is boolean anyway. You can check it with "amixer
    >> contents". user-space can figure out based on the control name. At
    >> least PulseAudio does that way.
    >
    > No, it can't do that for headset jacks - these will be created with a
    > single jack reporting multiple states, there's a state for headphone and
    > a state for microphone. The system can generally distinguish between
    > having a headset or just plain headphones inserted and act accordingly
    > (for example, recording from the built in microphone on a phone when
    > used with normal headpones).
    >
    >> > What I'd expect to happen here is that for multi function jacks we
    >> > create a control per function if the controls are valid.

    Ok, so the idea is just to change the control to type integer instead
    of boolean, right?
    Because as you say, the user will be able to check the type of jack
    based on the status value, right?

    >
    >> Do you mean based on snd_jack_types?
    >
    > Yes. If there's only one function supported the current code is fine
    > but for multiple functions it's going to discard useful information.

    So, what do you suggest to do that? I'm not sure if I understand what
    you are saying.
    When you mean function, do you mean the SND_JACK_BTN_n or the the jack
    types, such as SND_JACK_HEADPHONE, and so on?

    If a codec creates a jack type SND_JACK_HEADSET (= SND_JACK_HEADPHONE
    | SND_JACK_MICROPHONE). It should be created two controls, name +
    "Headphone Jack" and name + "Microphone Jack"? If so, what about the
    status to report? How to know which control to report?

    Felipe Tonello


    \
     
     \ /
      Last update: 2013-07-27 01:21    [W:3.117 / U:1.096 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site