Re: [PATCH 1/2] kconfig: change config_enabled() to accept X insteadof CONFIG_X

[Geert Uytterhoeven]

On Wed, May 9, 2012 at 8:56 PM, Sam Ravnborg <sam@xxxxxxxxxxxx> wrote:
> On Wed, May 09, 2012 at 11:46:35AM -0700, Suresh Siddha wrote:
>> change config_enabled() to use it as config_enabled(SMP) instead of
>> config_enabled(CONFIG_SMP).
>
> You are breaking all existing users.
> And CONFIG_ was included to allow better grepability.

Indeed.

> If you insist on having a CONFIG less variant
> introduce a new variant for this.

It's already bad enough that you have to use
`git grep -w FOO $(git ls-files "*Kconf*")` [*] to find the non-prefixed
symbols in the Kconfig logic, followed by `git grep -w CONFIG_FOO`
for the rest.

[*] Doh, recently I discovered that `git grep -w FOO -- "*Kconf*"' also
works, but my fingers are not yet used to it...

Gr{oetje,eeting}s,

            Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
             Â Â -- Linus Torvalds
--
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/