[lkml]   [2012]   [Mar]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH v6 1/5] Extcon (external connector): import Android's switch class and modify.
    On Fri, Mar 30, 2012 at 3:07 AM, Mark Brown
    <> wrote:
    > On Thu, Mar 29, 2012 at 03:27:01PM -0700, Erik Gilling wrote:
    >> On Fri, Mar 9, 2012 at 4:41 AM, Mark Brown
    >> > This seems somewhat sad - if ANDROID is turned on the standard ABI
    >> > vanishes.  It'd be much nicer to do this with a symlink (or with
    >> > symlinks within the android directory if the driver core doesn't support
    >> > that).  That way userspace code can be written to the new ABI and will
    >> > work on Android systems without ifdefery.
    >> This won't work if userspace code is receiving uevents through netlink
    >> and comparing based on the device name which it does in android.  Why
    > That's not really the point here - the point is that the new ABI
    > vanishes as soon as you turn on the legacy Android ABI.  You're right
    > that the particular fix I suggested has issues but the overall problem
    > exists and should be dealt with more sensibly.

    I'm also not in favor of having functionality conditionally compiled
    based on CONFIG_ANDROID. If fact, going through the patch stack
    there's much more that changes the ABI from the switch driver than
    just the name. Android asumes that there is a single "switch" for
    each logical entity (called cable types in extcon) each with a binary
    state (0,1). Here things have changed to have a single extcon
    instance that can have a bitmask of states which are sent as strings
    in the uevent.

    As it stands, this patch does not solve the cases where we use switch
    today and we'll probably continue to carry the switch driver in the
    common android tree. If, instead, we got rid of the idea of multiple
    states and mutual exclusivity and relied on the driver that uses
    extcon to have multiple instances for each logical entity and deal
    with mutual exclusion itself, we'd have a driver that would be pretty
    easy to support in android.

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2012-03-30 19:33    [W:0.022 / U:40.320 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site