Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-03-02 Thread Ulf Magnusson
On Fri, Mar 02, 2018 at 10:03:26AM +0100, Ulf Magnusson wrote: > On Fri, Mar 02, 2018 at 02:50:39PM +0900, Masahiro Yamada wrote: > > 2018-02-22 6:39 GMT+09:00 Ulf Magnusson : > > > On Wed, Feb 21, 2018 at 09:57:03PM +0900, Masahiro Yamada wrote: > > >> 2018-02-21 19:52

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-03-02 Thread Ulf Magnusson
On Fri, Mar 02, 2018 at 10:03:26AM +0100, Ulf Magnusson wrote: > On Fri, Mar 02, 2018 at 02:50:39PM +0900, Masahiro Yamada wrote: > > 2018-02-22 6:39 GMT+09:00 Ulf Magnusson : > > > On Wed, Feb 21, 2018 at 09:57:03PM +0900, Masahiro Yamada wrote: > > >> 2018-02-21 19:52 GMT+09:00 Arnd Bergmann : >

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-03-02 Thread Ulf Magnusson
On Fri, Mar 02, 2018 at 02:50:39PM +0900, Masahiro Yamada wrote: > 2018-02-22 6:39 GMT+09:00 Ulf Magnusson : > > On Wed, Feb 21, 2018 at 09:57:03PM +0900, Masahiro Yamada wrote: > >> 2018-02-21 19:52 GMT+09:00 Arnd Bergmann : > >> > On Wed, Feb 21, 2018 at 11:20

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-03-02 Thread Ulf Magnusson
On Fri, Mar 02, 2018 at 02:50:39PM +0900, Masahiro Yamada wrote: > 2018-02-22 6:39 GMT+09:00 Ulf Magnusson : > > On Wed, Feb 21, 2018 at 09:57:03PM +0900, Masahiro Yamada wrote: > >> 2018-02-21 19:52 GMT+09:00 Arnd Bergmann : > >> > On Wed, Feb 21, 2018 at 11:20 AM, Masahiro Yamada > >> > wrote:

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-03-01 Thread Masahiro Yamada
2018-02-22 6:39 GMT+09:00 Ulf Magnusson : > On Wed, Feb 21, 2018 at 09:57:03PM +0900, Masahiro Yamada wrote: >> 2018-02-21 19:52 GMT+09:00 Arnd Bergmann : >> > On Wed, Feb 21, 2018 at 11:20 AM, Masahiro Yamada >> > wrote: >> >>

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-03-01 Thread Masahiro Yamada
2018-02-22 6:39 GMT+09:00 Ulf Magnusson : > On Wed, Feb 21, 2018 at 09:57:03PM +0900, Masahiro Yamada wrote: >> 2018-02-21 19:52 GMT+09:00 Arnd Bergmann : >> > On Wed, Feb 21, 2018 at 11:20 AM, Masahiro Yamada >> > wrote: >> >> 2018-02-21 18:56 GMT+09:00 Arnd Bergmann : >> >>> On Wed, Feb 21,

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Michael Ellerman
Masahiro Yamada writes: > > > (Case 3) > Compiler flag -foo is sensitive to endian-ness. > > > config CC_NEEDS_BIG_ENDIAN > def_bool $(cc-option -mbig-endian) && CPU_BIG_ENDIAN > > config CC_NEEDS_LITTLE_ENDIAN > def_bool $(cc-option

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Michael Ellerman
Masahiro Yamada writes: > > > (Case 3) > Compiler flag -foo is sensitive to endian-ness. > > > config CC_NEEDS_BIG_ENDIAN > def_bool $(cc-option -mbig-endian) && CPU_BIG_ENDIAN > > config CC_NEEDS_LITTLE_ENDIAN > def_bool $(cc-option -mlittle-endian) && CPU_LITTLE_ENDIAN > >

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Ulf Magnusson
On Wed, Feb 21, 2018 at 09:57:03PM +0900, Masahiro Yamada wrote: > 2018-02-21 19:52 GMT+09:00 Arnd Bergmann : > > On Wed, Feb 21, 2018 at 11:20 AM, Masahiro Yamada > > wrote: > >> 2018-02-21 18:56 GMT+09:00 Arnd Bergmann : > >>> On Wed,

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Ulf Magnusson
On Wed, Feb 21, 2018 at 09:57:03PM +0900, Masahiro Yamada wrote: > 2018-02-21 19:52 GMT+09:00 Arnd Bergmann : > > On Wed, Feb 21, 2018 at 11:20 AM, Masahiro Yamada > > wrote: > >> 2018-02-21 18:56 GMT+09:00 Arnd Bergmann : > >>> On Wed, Feb 21, 2018 at 8:38 AM, Masahiro Yamada > >>> wrote: >

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Arnd Bergmann
On Wed, Feb 21, 2018 at 1:57 PM, Masahiro Yamada wrote: > 2018-02-21 19:52 GMT+09:00 Arnd Bergmann : >> On Wed, Feb 21, 2018 at 11:20 AM, Masahiro Yamada >> wrote: >>> 2018-02-21 18:56 GMT+09:00 Arnd Bergmann

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Arnd Bergmann
On Wed, Feb 21, 2018 at 1:57 PM, Masahiro Yamada wrote: > 2018-02-21 19:52 GMT+09:00 Arnd Bergmann : >> On Wed, Feb 21, 2018 at 11:20 AM, Masahiro Yamada >> wrote: >>> 2018-02-21 18:56 GMT+09:00 Arnd Bergmann : On Wed, Feb 21, 2018 at 8:38 AM, Masahiro Yamada wrote: > 2018-02-20

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Masahiro Yamada
2018-02-21 19:52 GMT+09:00 Arnd Bergmann : > On Wed, Feb 21, 2018 at 11:20 AM, Masahiro Yamada > wrote: >> 2018-02-21 18:56 GMT+09:00 Arnd Bergmann : >>> On Wed, Feb 21, 2018 at 8:38 AM, Masahiro Yamada >>>

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Masahiro Yamada
2018-02-21 19:52 GMT+09:00 Arnd Bergmann : > On Wed, Feb 21, 2018 at 11:20 AM, Masahiro Yamada > wrote: >> 2018-02-21 18:56 GMT+09:00 Arnd Bergmann : >>> On Wed, Feb 21, 2018 at 8:38 AM, Masahiro Yamada >>> wrote: 2018-02-20 0:18 GMT+09:00 Ulf Magnusson : >> >> Let me clarify my concern. >>

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Arnd Bergmann
On Wed, Feb 21, 2018 at 11:20 AM, Masahiro Yamada wrote: > 2018-02-21 18:56 GMT+09:00 Arnd Bergmann : >> On Wed, Feb 21, 2018 at 8:38 AM, Masahiro Yamada >> wrote: >>> 2018-02-20 0:18 GMT+09:00 Ulf Magnusson

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Arnd Bergmann
On Wed, Feb 21, 2018 at 11:20 AM, Masahiro Yamada wrote: > 2018-02-21 18:56 GMT+09:00 Arnd Bergmann : >> On Wed, Feb 21, 2018 at 8:38 AM, Masahiro Yamada >> wrote: >>> 2018-02-20 0:18 GMT+09:00 Ulf Magnusson : > > Let me clarify my concern. > > When we test the compiler flag, is there a case >

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Masahiro Yamada
2018-02-21 18:56 GMT+09:00 Arnd Bergmann : > On Wed, Feb 21, 2018 at 8:38 AM, Masahiro Yamada > wrote: >> 2018-02-20 0:18 GMT+09:00 Ulf Magnusson : >> I'm not happy that we in one context can reference CONFIG variables

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Masahiro Yamada
2018-02-21 18:56 GMT+09:00 Arnd Bergmann : > On Wed, Feb 21, 2018 at 8:38 AM, Masahiro Yamada > wrote: >> 2018-02-20 0:18 GMT+09:00 Ulf Magnusson : >> I'm not happy that we in one context can reference CONFIG variables directly, but inside the $(call ...) and $(shell ...) needs the

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Arnd Bergmann
On Wed, Feb 21, 2018 at 8:38 AM, Masahiro Yamada wrote: > 2018-02-20 0:18 GMT+09:00 Ulf Magnusson : > >>> >>> I'm not happy that we in one context can reference CONFIG variables >>> directly, but inside the $(call ...) and $(shell ...) needs the

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-21 Thread Arnd Bergmann
On Wed, Feb 21, 2018 at 8:38 AM, Masahiro Yamada wrote: > 2018-02-20 0:18 GMT+09:00 Ulf Magnusson : > >>> >>> I'm not happy that we in one context can reference CONFIG variables >>> directly, but inside the $(call ...) and $(shell ...) needs the $ prefix. >>> But I could not come up with

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-20 Thread Masahiro Yamada
2018-02-20 0:18 GMT+09:00 Ulf Magnusson : >> >> I'm not happy that we in one context can reference CONFIG variables >> directly, but inside the $(call ...) and $(shell ...) needs the $ prefix. >> But I could not come up with something un-ambigious where this could be >>

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-20 Thread Masahiro Yamada
2018-02-20 0:18 GMT+09:00 Ulf Magnusson : >> >> I'm not happy that we in one context can reference CONFIG variables >> directly, but inside the $(call ...) and $(shell ...) needs the $ prefix. >> But I could not come up with something un-ambigious where this could be >> avoided. > > I think we

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-19 Thread Ulf Magnusson
Hello, On Sun, Feb 18, 2018 at 11:13 PM, Sam Ravnborg wrote: > Hi Masahiro. > > On Sat, Feb 17, 2018 at 03:38:28AM +0900, Masahiro Yamada wrote: >> I brushed up the implementation in this version. >> >> In the previous RFC, CC_HAS_ was described by using 'option shell=', >>

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-19 Thread Ulf Magnusson
Hello, On Sun, Feb 18, 2018 at 11:13 PM, Sam Ravnborg wrote: > Hi Masahiro. > > On Sat, Feb 17, 2018 at 03:38:28AM +0900, Masahiro Yamada wrote: >> I brushed up the implementation in this version. >> >> In the previous RFC, CC_HAS_ was described by using 'option shell=', >> like this: >> >>

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-18 Thread Sam Ravnborg
Hi Masahiro. On Sat, Feb 17, 2018 at 03:38:28AM +0900, Masahiro Yamada wrote: > I brushed up the implementation in this version. > > In the previous RFC, CC_HAS_ was described by using 'option shell=', > like this: > > config CC_HAS_STACKPROTECTOR > bool > option shell="$CC

Re: [PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-18 Thread Sam Ravnborg
Hi Masahiro. On Sat, Feb 17, 2018 at 03:38:28AM +0900, Masahiro Yamada wrote: > I brushed up the implementation in this version. > > In the previous RFC, CC_HAS_ was described by using 'option shell=', > like this: > > config CC_HAS_STACKPROTECTOR > bool > option shell="$CC

[PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-16 Thread Masahiro Yamada
I brushed up the implementation in this version. In the previous RFC, CC_HAS_ was described by using 'option shell=', like this: config CC_HAS_STACKPROTECTOR bool option shell="$CC -Werror -fstack-protector -c -x c /dev/null" After I thought a bit more, the following syntax is

[PATCH 00/23] kconfig: move compiler capability tests to Kconfig

2018-02-16 Thread Masahiro Yamada
I brushed up the implementation in this version. In the previous RFC, CC_HAS_ was described by using 'option shell=', like this: config CC_HAS_STACKPROTECTOR bool option shell="$CC -Werror -fstack-protector -c -x c /dev/null" After I thought a bit more, the following syntax is