lkml.org 
[lkml]   [2021]   [Jan]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH 01/14] mfd: arizona: Add jack pointer to struct arizona
    From
    Date
    Hi,

    On 12/30/20 2:38 PM, Mark Brown wrote:
    > On Tue, Dec 29, 2020 at 04:33:09PM +0100, Hans de Goede wrote:
    >> On 12/29/20 4:08 PM, Mark Brown wrote:

    <snip>

    >>> The whole purpose of creating sound/core/jack.c is to abstract away the
    >>> userspace interfaces from the drivers, most audio devices shouldn't be
    >>> working with extcon directly just as they shouldn't be creating input
    >>> devices or ALSA controls directly. The missing bit there is to add
    >>> extcon into jack.c.
    >
    >> So what you are suggesting is making sound/core/jack.c register the
    >> extcon device and then have arizona-extcon.c talk to sound/core/jack.c
    >> and no longer do any extcon stuff itself.
    >
    > Yes.

    Ok, so this seems to be the same solution as which the opensource.cirrus.com
    folks want in that both you and the opensource.cirrus.com people want to
    change the arizona-extcon.c driver to be changed to stop reporting extcon
    info directly itself and instead do all the reporting through sound/core/jack.c.

    Where the thoughts on this seem to differ is that the opensource.cirrus.com
    folks seem to be fine with dropping extcon support, where as you suggest
    to extend sound/core/jack.c to register an extcon device and have it report
    extcon events.

    I'm with the opensource.cirrus.com folks here that ATM it seems best to just
    drop extcon support since the only user is Android, which also supports the
    input_dev API.

    If the need arises we can always add extcon-support to sound/core/jack.c
    later.

    So I'll start on reworking this patch-series to change the arizona-extcon.c
    driver to stop it reporting extcon info directly itself and instead do all
    the reporting through sound/core/jack.c.

    Regards,

    Hans

    \
     
     \ /
      Last update: 2021-01-01 14:27    [W:2.860 / U:0.300 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site