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

Arnd Bergmann arnd at arndb.de
Thu Jul 7 11:57:23 EDT 2011


On Thursday 07 July 2011, Huang Shijie wrote:
> > On Friday 01 July 2011 10:57:40 Huang Shijie wrote:
> >>>> 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
> 

Ok, so Uwe made a rather generic comment that he'd prefer you to use
macros for that part. I think your original code actually looks cleaner
in this case, mostly because it doesn't obfuscate the identifiers
and because the macro is specific to this file.

Uwe, do you think it's worthwhile introducing a global macro for
everyone to use for defining resources? Clearly this is not an
mxs specific problem, and I'd much rather have the same solution
everywhere.

I'm thinking that if we want to solve this, the macro should live
in include/linux/ioport.h along with struct resource.

	Arnd



More information about the linux-arm-kernel mailing list