Generic Linux architectural discussions
 help / color / mirror / Atom feed
* [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED()
       [not found] <1465883938-25194-1-git-send-email-yamada.masahiro@socionext.com>
@ 2016-06-14  5:58 ` Masahiro Yamada
  2016-06-14  5:58   ` Masahiro Yamada
  2016-06-20 20:45   ` Michal Marek
  0 siblings, 2 replies; 9+ messages in thread
From: Masahiro Yamada @ 2016-06-14  5:58 UTC (permalink / raw)
  To: linux-kernel, Andrew Morton
  Cc: Rusty Russell, Michal Simek, Linus Torvalds, Arnd Bergmann,
	Michal Marek, Nicolas Pitre, Masahiro Yamada, linux-arch

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(-)

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);							\
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED()
  2016-06-14  5:58 ` [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED() Masahiro Yamada
@ 2016-06-14  5:58   ` Masahiro Yamada
  2016-06-20 20:45   ` Michal Marek
  1 sibling, 0 replies; 9+ messages in thread
From: Masahiro Yamada @ 2016-06-14  5:58 UTC (permalink / raw)
  To: linux-kernel, Andrew Morton
  Cc: Rusty Russell, Michal Simek, Linus Torvalds, Arnd Bergmann,
	Michal Marek, Nicolas Pitre, Masahiro Yamada, linux-arch

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(-)

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);							\
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED()
  2016-06-14  5:58 ` [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED() Masahiro Yamada
  2016-06-14  5:58   ` Masahiro Yamada
@ 2016-06-20 20:45   ` Michal Marek
  2016-06-20 23:20     ` Masahiro Yamada
  1 sibling, 1 reply; 9+ messages in thread
From: Michal Marek @ 2016-06-20 20:45 UTC (permalink / raw)
  To: Masahiro Yamada, linux-kernel, Andrew Morton
  Cc: Rusty Russell, Michal Simek, Linus Torvalds, Arnd Bergmann,
	Nicolas Pitre, linux-arch

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED()
  2016-06-20 20:45   ` Michal Marek
@ 2016-06-20 23:20     ` Masahiro Yamada
  2016-06-21  9:25       ` Arnd Bergmann
  2016-06-23  8:07       ` Masahiro Yamada
  0 siblings, 2 replies; 9+ messages in thread
From: Masahiro Yamada @ 2016-06-20 23:20 UTC (permalink / raw)
  To: Michal Marek
  Cc: Linux Kernel Mailing List, Andrew Morton, Rusty Russell,
	Michal Simek, Linus Torvalds, Arnd Bergmann, Nicolas Pitre,
	linux-arch

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED()
  2016-06-20 23:20     ` Masahiro Yamada
@ 2016-06-21  9:25       ` Arnd Bergmann
  2016-06-21  9:25         ` Arnd Bergmann
  2016-06-22  6:52         ` Masahiro Yamada
  2016-06-23  8:07       ` Masahiro Yamada
  1 sibling, 2 replies; 9+ messages in thread
From: Arnd Bergmann @ 2016-06-21  9:25 UTC (permalink / raw)
  To: Masahiro Yamada
  Cc: Michal Marek, Linux Kernel Mailing List, Andrew Morton,
	Rusty Russell, Michal Simek, Linus Torvalds, Nicolas Pitre,
	linux-arch

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED()
  2016-06-21  9:25       ` Arnd Bergmann
@ 2016-06-21  9:25         ` Arnd Bergmann
  2016-06-22  6:52         ` Masahiro Yamada
  1 sibling, 0 replies; 9+ messages in thread
From: Arnd Bergmann @ 2016-06-21  9:25 UTC (permalink / raw)
  To: Masahiro Yamada
  Cc: Michal Marek, Linux Kernel Mailing List, Andrew Morton,
	Rusty Russell, Michal Simek, Linus Torvalds, Nicolas Pitre,
	linux-arch

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


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED()
  2016-06-21  9:25       ` Arnd Bergmann
  2016-06-21  9:25         ` Arnd Bergmann
@ 2016-06-22  6:52         ` Masahiro Yamada
  2016-06-22  6:52           ` Masahiro Yamada
  1 sibling, 1 reply; 9+ messages in thread
From: Masahiro Yamada @ 2016-06-22  6:52 UTC (permalink / raw)
  To: Arnd Bergmann, Andrew Morton
  Cc: Michal Marek, Linux Kernel Mailing List, Rusty Russell,
	Michal Simek, Linus Torvalds, Nicolas Pitre, linux-arch

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED()
  2016-06-22  6:52         ` Masahiro Yamada
@ 2016-06-22  6:52           ` Masahiro Yamada
  0 siblings, 0 replies; 9+ messages in thread
From: Masahiro Yamada @ 2016-06-22  6:52 UTC (permalink / raw)
  To: Arnd Bergmann, Andrew Morton
  Cc: Michal Marek, Linux Kernel Mailing List, Rusty Russell,
	Michal Simek, Linus Torvalds, Nicolas Pitre, linux-arch

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED()
  2016-06-20 23:20     ` Masahiro Yamada
  2016-06-21  9:25       ` Arnd Bergmann
@ 2016-06-23  8:07       ` Masahiro Yamada
  1 sibling, 0 replies; 9+ messages in thread
From: Masahiro Yamada @ 2016-06-23  8:07 UTC (permalink / raw)
  To: Michal Marek, Andrew Morton
  Cc: Linux Kernel Mailing List, Rusty Russell, Michal Simek,
	Linus Torvalds, Arnd Bergmann, Nicolas Pitre, linux-arch

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2016-06-23  8:08 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1465883938-25194-1-git-send-email-yamada.masahiro@socionext.com>
2016-06-14  5:58 ` [PATCH 5/5] vmlinux.lds.h: replace config_enabled() with IS_ENABLED() Masahiro Yamada
2016-06-14  5:58   ` Masahiro Yamada
2016-06-20 20:45   ` Michal Marek
2016-06-20 23:20     ` Masahiro Yamada
2016-06-21  9:25       ` Arnd Bergmann
2016-06-21  9:25         ` Arnd Bergmann
2016-06-22  6:52         ` Masahiro Yamada
2016-06-22  6:52           ` Masahiro Yamada
2016-06-23  8:07       ` Masahiro Yamada

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox