Thread (27 messages) 27 messages, 3 authors, 2015-03-09

[PATCH v5 3/8] pinctrl: cygnus: add initial IOMUX driver support

From: Linus Walleij <hidden>
Date: 2015-03-09 16:28:33
Also in: linux-devicetree, linux-gpio, lkml

On Thu, Mar 5, 2015 at 9:03 AM, Paul Bolle [off-list ref] wrote:
quoted
[...]
+MODULE_AUTHOR("Ray Jui [off-list ref]");
+MODULE_DESCRIPTION("Broadcom Cygnus IOMUX driver");
+MODULE_LICENSE("GPL v2");
These three macros will be preprocessed away. (And I guess you could
also drop the linux/module.h include, but I haven't actually tested that
guess.)
I think you're right. Or I fear you're right.

But this problem is present in so many drivers that a generic
fixup needs to be done with a script and across an entire subsystem
at once, and besides I'm not sure of these macros disturb so much.

They are documentation in a sense, albeit a kind of documentation
we used before we had git to record the actual authors of the
code.

Yours,
Linus Walleij
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help