Re: [PATCH 2/3] kconfig: add a function to get the CONFIG_ prefix

From: Yann E. MORIN
Date: Fri Oct 19 2012 - 13:59:39 EST


Michal, All,

On Friday 19 October 2012 Michal Marek wrote:
> On 18.10.2012 21:50, Yann E. MORIN wrote:
> > -#ifndef CONFIG_
> > -#define CONFIG_ "CONFIG_"
> > +/* Those two defines copied from include/linux/stringify.h */
> > +#define __stringify_1(x...) #x
> > +#define __stringify(x...) __stringify_1(x)
> > +static inline const char *CONFIG_prefix(void)
> > +{
> > + return __stringify(CONFIG_);
>
> This changes the semantics of the CONFIG_ macro. Previously, a
> double-quoted string was accepted, now you have to define it to a token
> that is turned into a string. Why don't you keep it as before and just do a
>
> return CONFIG_;
>
> in the function?

Yes, that's as simple as it looks, indeed. /me hides.

Thank you!
Regards,
Yann E. MORIN.

--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/