Thread (18 messages) flat view 18 messages, 6 authors, 2012-04-02

[PATCH v6 1/5] Extcon (external connector): import Android's switch class and modify.

From: Mark Brown <hidden>
Date: 2012-03-09 12:41:28
Also in: lkml

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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120309/a0180f35/attachment-0001.sig>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help