Re: [PATCH 1/3] Kconfig: disable PROFILE_ALL_BRANCHES for compile testing

2018-02-20 Thread Masahiro Yamada
2018-02-17 6:41 GMT+09:00 Arnd Bergmann : > This can easily double the time for compiling a driver but does not > provide any benefit for the compile tester, so it's better left disabled. > > In addition, any 'inline' function that is not also 'static' and that > contains an 'if' causes a warning l

Re: [PATCH 1/3] Kconfig: disable PROFILE_ALL_BRANCHES for compile testing

2018-02-17 Thread Greg Kroah-Hartman
On Fri, Feb 16, 2018 at 11:14:42PM +0100, Arnd Bergmann wrote: > On Fri, Feb 16, 2018 at 11:03 PM, Steven Rostedt wrote: > > On Fri, 16 Feb 2018 22:41:11 +0100 > > Arnd Bergmann wrote: > > > >> This can easily double the time for compiling a driver but does not > >> provide any benefit for the co

Re: [PATCH 1/3] Kconfig: disable PROFILE_ALL_BRANCHES for compile testing

2018-02-16 Thread Steven Rostedt
On Fri, 16 Feb 2018 23:40:22 +0100 Arnd Bergmann wrote: > ((CAPI_MSG *) msg)->info.facility_req.structs[1] = > LI_REQ_SILENT_UPDATE & 0xff; > > ~^ > drivers/isdn/hardware/eicon/message.c:11163:54: error: array

Re: [PATCH 1/3] Kconfig: disable PROFILE_ALL_BRANCHES for compile testing

2018-02-16 Thread Arnd Bergmann
On Fri, Feb 16, 2018 at 11:14 PM, Arnd Bergmann wrote: > On Fri, Feb 16, 2018 at 11:03 PM, Steven Rostedt wrote: >> On Fri, 16 Feb 2018 22:41:11 +0100 >> Arnd Bergmann wrote: >> >>> This can easily double the time for compiling a driver but does not >>> provide any benefit for the compile tester

Re: [PATCH 1/3] Kconfig: disable PROFILE_ALL_BRANCHES for compile testing

2018-02-16 Thread Arnd Bergmann
On Fri, Feb 16, 2018 at 11:03 PM, Steven Rostedt wrote: > On Fri, 16 Feb 2018 22:41:11 +0100 > Arnd Bergmann wrote: > >> This can easily double the time for compiling a driver but does not >> provide any benefit for the compile tester, so it's better left disabled. >> >> In addition, any 'inline'

Re: [PATCH 1/3] Kconfig: disable PROFILE_ALL_BRANCHES for compile testing

2018-02-16 Thread Steven Rostedt
On Fri, 16 Feb 2018 22:41:11 +0100 Arnd Bergmann wrote: > This can easily double the time for compiling a driver but does not > provide any benefit for the compile tester, so it's better left disabled. > > In addition, any 'inline' function that is not also 'static' and that > contains an 'if' c