On Fri, Dec 06, 2013 at 03:03:19PM +0000, Mark Brown wrote: > On Fri, Dec 06, 2013 at 10:38:17PM +0800, Nicolin Chen wrote: > > On Fri, Dec 06, 2013 at 02:42:56PM +0000, Mark Brown wrote: > > > > What's this uevent trying to do and why is it open coded in the driver? > > > That's a requirement from Android team of Freescale. By checking this > > event they can determine whether current output is speaker or headphone > > so as to place an icon on the notification area. > > > Sir, if you think this patch is not common enough or non-community style, > > I can try to cut it down to an common one or entirely drop it. > > This isn't mainline stuff, this is the Android standard switch class. > Someone should really contribute something to Android which backs it > onto the back of ALSA jacks. The mainline equivalent is extcon which > does have switch class emulation but it's a bit broken in its generation > of uevents and is also not yet integrated into the ALSA jack stuff. > > The thing to do is ideally integrate extcon with the ALSA jack code and > update the Android headset observer to work with extcon (or write a > separate extcon based observer). Thank you for the detailed explain. It looks more complicated to me with current code base. I think I should just extract the mainline-style part so it'll make things easy if there's gonna be some plausible integrations afterward. Thank you indeed, Nicolin Chen -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html