Thread (20 messages) 20 messages, 6 authors, 2011-07-14

reply: [PATCH v5 1/3] ARM: mxs: add GPMI-NFC support for imx23/imx28

From: Huang Shijie <hidden>
Date: 2011-07-07 08:56:01

Hi:
On Friday 01 July 2011 10:57:40 Huang Shijie wrote:
quoted
quoted
quoted
It's annoying, but it really saves some lines.
It would save more lines if you introduce the macros globally and
convert all existing resource definitions ;-)
The origin code did not use any macros.
Some one suggested me to use macros.
So i used the macros.

Do i have to drop the macros?
Can you find out who made the suggestion? Let's first reach consensus
about how we want to do these things in the future.
http://www.spinics.net/lists/arm-kernel/msg121384.html

Thanks to your comments.


Best Regards
Huang Shijie
My opinion is that we should not have macros like these in any platform
specific code: If the macros are a good idea, we should define them
globally and let everyone use the same macros. If we decide not to have
them in global code, it would be even worse to have them in specific
platforms.

	Arnd

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel at lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help