Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-16 Thread Stephen Boyd
On 10/16, Javier Martinez Canillas wrote: > I built on all those archs and neither allyesconfig nor allmodconfig > gives me build error or warnings for the partial build M=drivers/clk > > I only had an issue with h8300 since the arch has hardcoded stuff > in its Makefile like the toolchain file na

Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-16 Thread Javier Martinez Canillas
Hello Stephen, On 10/14/2015 11:13 PM, Stephen Boyd wrote: > On 10/14, Javier Martinez Canillas wrote: >> On 10/14/2015 09:08 PM, Javier Martinez Canillas wrote: >>> Hello Stephen, >>> >>> On 10/14/2015 08:38 PM, Stephen Boyd wrote: On 10/13, Javier Martinez Canillas wrote: > diff --git a

Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-15 Thread Javier Martinez Canillas
Hello Krzysztof, On 10/15/2015 09:22 AM, Krzysztof Kozlowski wrote: > On 15.10.2015 16:11, Javier Martinez Canillas wrote: >> Hello Krzysztof, > No, I only build tested on arm32 and x86. The 0-day bot haven't reported a build error yet and I didn't see any platform dependent cod

Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-15 Thread Krzysztof Kozlowski
On 15.10.2015 16:11, Javier Martinez Canillas wrote: > Hello Krzysztof, >>> >>> No, I only build tested on arm32 and x86. The 0-day bot haven't reported a >>> build error yet and I didn't see any platform dependent code in the drivers. >> >> I see you guys with Luis are adding a lot of COMPILE

Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-15 Thread Javier Martinez Canillas
Hello Krzysztof, On 10/15/2015 04:04 AM, Krzysztof Kozlowski wrote: > 2015-10-15 4:08 GMT+09:00 Javier Martinez Canillas : >> Hello Stephen, >> >> On 10/14/2015 08:38 PM, Stephen Boyd wrote: >>> On 10/13, Javier Martinez Canillas wrote: diff --git a/drivers/clk/versatile/Kconfig b/drivers/clk

Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-14 Thread Krzysztof Kozlowski
2015-10-15 4:08 GMT+09:00 Javier Martinez Canillas : > Hello Stephen, > > On 10/14/2015 08:38 PM, Stephen Boyd wrote: >> On 10/13, Javier Martinez Canillas wrote: >>> diff --git a/drivers/clk/versatile/Kconfig b/drivers/clk/versatile/Kconfig >>> index 1530c9352a76..fc50b6264bed 100644 >>> --- a/dri

Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-14 Thread Stephen Boyd
On 10/14, Javier Martinez Canillas wrote: > On 10/14/2015 09:08 PM, Javier Martinez Canillas wrote: > > Hello Stephen, > > > > On 10/14/2015 08:38 PM, Stephen Boyd wrote: > >> On 10/13, Javier Martinez Canillas wrote: > >>> diff --git a/drivers/clk/versatile/Kconfig b/drivers/clk/versatile/Kconfig

Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-14 Thread Javier Martinez Canillas
On 10/14/2015 09:08 PM, Javier Martinez Canillas wrote: > Hello Stephen, > > On 10/14/2015 08:38 PM, Stephen Boyd wrote: >> On 10/13, Javier Martinez Canillas wrote: >>> diff --git a/drivers/clk/versatile/Kconfig b/drivers/clk/versatile/Kconfig >>> index 1530c9352a76..fc50b6264bed 100644 >>> --- a

Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-14 Thread Javier Martinez Canillas
Hello Stephen, On 10/14/2015 08:38 PM, Stephen Boyd wrote: > On 10/13, Javier Martinez Canillas wrote: >> diff --git a/drivers/clk/versatile/Kconfig b/drivers/clk/versatile/Kconfig >> index 1530c9352a76..fc50b6264bed 100644 >> --- a/drivers/clk/versatile/Kconfig >> +++ b/drivers/clk/versatile/Kcon

Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-14 Thread Stephen Boyd
On 10/13, Javier Martinez Canillas wrote: > diff --git a/drivers/clk/versatile/Kconfig b/drivers/clk/versatile/Kconfig > index 1530c9352a76..fc50b6264bed 100644 > --- a/drivers/clk/versatile/Kconfig > +++ b/drivers/clk/versatile/Kconfig > @@ -1,6 +1,6 @@ > config COMMON_CLK_VERSATILE > bool

Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-13 Thread Scott Branden
For BCM portion: Acked-by: Scott Branden On 15-10-13 07:18 AM, Javier Martinez Canillas wrote: These drivers only have runtime but no build time dependencies so can be built for testing purposes if the Kconfig COMPILE_TEST option is enabled. This is useful to have more build coverage and make

Re: [PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-13 Thread kbuild test robot
Hi Javier, [auto build test WARNING on clk/clk-next -- if it's inappropriate base, please suggest rules for selecting the more suitable base] url: https://github.com/0day-ci/linux/commits/Javier-Martinez-Canillas/clk-Allow-drivers-to-build-if-COMPILE_TEST-is-enabled/20151013-222133 reproduce

[PATCH] clk: Allow drivers to build if COMPILE_TEST is enabled

2015-10-13 Thread Javier Martinez Canillas
These drivers only have runtime but no build time dependencies so can be built for testing purposes if the Kconfig COMPILE_TEST option is enabled. This is useful to have more build coverage and make sure that drivers are not affected by changes that could cause build regressions. Signed-off-by: J