lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 29 Mar 2012 15:27:01 -0700 From: Erik Gilling <konkers@...gle.com> To: Mark Brown <broonie@...nsource.wolfsonmicro.com> Cc: MyungJoo Ham <myungjoo.ham@...sung.com>, Arnd Bergmann <arnd@...db.de>, NeilBrown <neilb@...e.de>, gregkh@...uxfoundation.org, Linus Walleij <linus.walleij@...aro.org>, Dmitry Torokhov <dmitry.torokhov@...il.com>, Arve Hjønnevag <arve@...roid.com>, linux-kernel@...r.kernel.org, Randy Dunlap <rdunlap@...otime.net>, myungjoo.ham@...il.com, John Stultz <john.stultz@...aro.org>, linux-arm-kernel@...ts.infradead.org, Joerg Roedel <joerg.roedel@....com>, Kyungmin Park <kyungmin.park@...sung.com>, Morten CHRISTIANSEN <morten.christiansen@...ricsson.com>, Mike Lockwood <lockwood@...roid.com> Subject: Re: [PATCH v6 1/5] Extcon (external connector): import Android's switch class and modify. On Fri, Mar 9, 2012 at 4:41 AM, Mark Brown <broonie@...nsource.wolfsonmicro.com> wrote: > On Mon, Feb 27, 2012 at 09:15:35PM +0900, MyungJoo Ham wrote: > >> +#ifdef CONFIG_ANDROID >> + extcon_class = class_create(THIS_MODULE, "switch"); >> +#else /* !CONFIG_ANDROID */ >> + extcon_class = class_create(THIS_MODULE, "extcon"); >> +#endif /* CONFIG_ANDROID */ > > 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 change the name at all? Extcon is much more specific than "switch." A switch can can be used for more than just external connections such as availability of HMDI audio (which is only available once a HDMI mode is chosen and the interface is enabled.) -Erik -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists