Message ID | 1465883938-25194-5-git-send-email-yamada.masahiro@socionext.com |
---|---|
State | New |
Headers | show |
Dne 14.6.2016 v 07:58 Masahiro Yamada napsal(a): > The use of config_enabled() against config options is ambiguous. > > Now, IS_ENABLED() is implemented purely with macro expansion, so > let's replace config_enabled() with IS_ENABLED(). > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> I applied the whole series to kbuild.git#kbuild. Michal
2016-06-21 5:45 GMT+09:00 Michal Marek <mmarek@suse.cz>: > Dne 14.6.2016 v 07:58 Masahiro Yamada napsal(a): >> The use of config_enabled() against config options is ambiguous. >> >> Now, IS_ENABLED() is implemented purely with macro expansion, so >> let's replace config_enabled() with IS_ENABLED(). >> >> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > I applied the whole series to kbuild.git#kbuild. > This series was applied by Andrew Morton last week. I do not want to double it. Could you drop it? -- Best Regards Masahiro Yamada
On Tuesday, June 21, 2016 8:20:10 AM CEST Masahiro Yamada wrote: > 2016-06-21 5:45 GMT+09:00 Michal Marek <mmarek@suse.cz>: > > Dne 14.6.2016 v 07:58 Masahiro Yamada napsal(a): > >> The use of config_enabled() against config options is ambiguous. > >> > >> Now, IS_ENABLED() is implemented purely with macro expansion, so > >> let's replace config_enabled() with IS_ENABLED(). > >> > >> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > > > I applied the whole series to kbuild.git#kbuild. > > > > This series was applied by Andrew Morton last week. > > I do not want to double it. Could you drop it? Andrew usually drops patches from his tree when they show up in another maintainer tree. Arnd
Hi Andrew, 2016-06-21 18:25 GMT+09:00 Arnd Bergmann <arnd@arndb.de>: > On Tuesday, June 21, 2016 8:20:10 AM CEST Masahiro Yamada wrote: >> 2016-06-21 5:45 GMT+09:00 Michal Marek <mmarek@suse.cz>: >> > Dne 14.6.2016 v 07:58 Masahiro Yamada napsal(a): >> >> The use of config_enabled() against config options is ambiguous. >> >> >> >> Now, IS_ENABLED() is implemented purely with macro expansion, so >> >> let's replace config_enabled() with IS_ENABLED(). >> >> >> >> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> >> > >> > I applied the whole series to kbuild.git#kbuild. >> > >> >> This series was applied by Andrew Morton last week. >> >> I do not want to double it. Could you drop it? > > Andrew usually drops patches from his tree when they show up in > another maintainer tree. Now, each patch of this series appears twice in linux-next. Andrew, Is it possible to drop this series from your tree? Thanks -- Best Regards Masahiro Yamada
Hi Andrew, Michal, 2016-06-21 8:20 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>: > 2016-06-21 5:45 GMT+09:00 Michal Marek <mmarek@suse.cz>: >> Dne 14.6.2016 v 07:58 Masahiro Yamada napsal(a): >>> The use of config_enabled() against config options is ambiguous. >>> >>> Now, IS_ENABLED() is implemented purely with macro expansion, so >>> let's replace config_enabled() with IS_ENABLED(). >>> >>> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> >> >> I applied the whole series to kbuild.git#kbuild. >> > > This series was applied by Andrew Morton last week. > > I do not want to double it. Could you drop it? Michal, It is OK. Everything is fine now. Andrew, Thanks a lot for taking care of this! -- Best Regards Masahiro Yamada
diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h index 6a67ab9..faa4d2b 100644 --- a/include/asm-generic/vmlinux.lds.h +++ b/include/asm-generic/vmlinux.lds.h @@ -164,7 +164,7 @@ #define ___OF_TABLE(cfg, name) _OF_TABLE_##cfg(name) #define __OF_TABLE(cfg, name) ___OF_TABLE(cfg, name) -#define OF_TABLE(cfg, name) __OF_TABLE(config_enabled(cfg), name) +#define OF_TABLE(cfg, name) __OF_TABLE(IS_ENABLED(cfg), name) #define _OF_TABLE_0(name) #define _OF_TABLE_1(name) \ . = ALIGN(8); \
The use of config_enabled() against config options is ambiguous. Now, IS_ENABLED() is implemented purely with macro expansion, so let's replace config_enabled() with IS_ENABLED(). Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> --- include/asm-generic/vmlinux.lds.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 1.9.1