* [PATCH] clk: sunxi-ng: tighten SoC deps on explicit AllWinner SoCs
@ 2017-03-02 17:43 Peter Robinson
2017-03-03 7:12 ` Maxime Ripard
2017-03-06 9:06 ` Maxime Ripard
0 siblings, 2 replies; 4+ messages in thread
From: Peter Robinson @ 2017-03-02 17:43 UTC (permalink / raw)
To: linux-arm-kernel
Tighten the depends on the various AllWinn SoCs so we don't
inadvertantly get clock drivers when we're not wanting them
like 32 bit SoC clocks for 64 bit configs. Ensure there's
still test coverage though.
Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
---
drivers/clk/sunxi-ng/Kconfig | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/clk/sunxi-ng/Kconfig b/drivers/clk/sunxi-ng/Kconfig
index 695bbf9..8ab378e 100644
--- a/drivers/clk/sunxi-ng/Kconfig
+++ b/drivers/clk/sunxi-ng/Kconfig
@@ -63,6 +63,7 @@ config SUN50I_A64_CCU
select SUNXI_CCU_MP
select SUNXI_CCU_PHASE
default ARM64 && ARCH_SUNXI
+ depends (ARM64 && ARCH_SUNXI) || COMPILE_TEST
config SUN5I_CCU
bool "Support for the Allwinner sun5i family CCM"
@@ -74,6 +75,7 @@ config SUN5I_CCU
select SUNXI_CCU_MP
select SUNXI_CCU_PHASE
default MACH_SUN5I
+ depends MACH_SUN5I || COMPILE_TEST
config SUN6I_A31_CCU
bool "Support for the Allwinner A31/A31s CCU"
@@ -84,6 +86,7 @@ config SUN6I_A31_CCU
select SUNXI_CCU_MP
select SUNXI_CCU_PHASE
default MACH_SUN6I
+ depends MACH_SUN6I || COMPILE_TEST
config SUN8I_A23_CCU
bool "Support for the Allwinner A23 CCU"
@@ -96,6 +99,7 @@ config SUN8I_A23_CCU
select SUNXI_CCU_MP
select SUNXI_CCU_PHASE
default MACH_SUN8I
+ depends MACH_SUN8I || COMPILE_TEST
config SUN8I_A33_CCU
bool "Support for the Allwinner A33 CCU"
@@ -108,6 +112,7 @@ config SUN8I_A33_CCU
select SUNXI_CCU_MP
select SUNXI_CCU_PHASE
default MACH_SUN8I
+ depends MACH_SUN8I || COMPILE_TEST
config SUN8I_H3_CCU
bool "Support for the Allwinner H3 CCU"
@@ -119,6 +124,7 @@ config SUN8I_H3_CCU
select SUNXI_CCU_MP
select SUNXI_CCU_PHASE
default MACH_SUN8I
+ depends MACH_SUN8I || COMPILE_TEST
config SUN8I_V3S_CCU
bool "Support for the Allwinner V3s CCU"
@@ -130,6 +136,7 @@ config SUN8I_V3S_CCU
select SUNXI_CCU_MP
select SUNXI_CCU_PHASE
default MACH_SUN8I
+ depends MACH_SUN8I || COMPILE_TEST
config SUN9I_A80_CCU
bool "Support for the Allwinner A80 CCU"
@@ -140,5 +147,6 @@ config SUN9I_A80_CCU
select SUNXI_CCU_MP
select SUNXI_CCU_PHASE
default MACH_SUN9I
+ depends MACH_SUN9I || COMPILE_TEST
endif
--
2.9.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] clk: sunxi-ng: tighten SoC deps on explicit AllWinner SoCs
2017-03-02 17:43 [PATCH] clk: sunxi-ng: tighten SoC deps on explicit AllWinner SoCs Peter Robinson
@ 2017-03-03 7:12 ` Maxime Ripard
2017-03-06 8:44 ` Chen-Yu Tsai
2017-03-06 9:06 ` Maxime Ripard
1 sibling, 1 reply; 4+ messages in thread
From: Maxime Ripard @ 2017-03-03 7:12 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Mar 02, 2017 at 05:43:57PM +0000, Peter Robinson wrote:
> Tighten the depends on the various AllWinn SoCs so we don't
> inadvertantly get clock drivers when we're not wanting them
> like 32 bit SoC clocks for 64 bit configs. Ensure there's
> still test coverage though.
>
> Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
Applied, thanks!
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170303/a285001e/attachment-0001.sig>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] clk: sunxi-ng: tighten SoC deps on explicit AllWinner SoCs
2017-03-03 7:12 ` Maxime Ripard
@ 2017-03-06 8:44 ` Chen-Yu Tsai
0 siblings, 0 replies; 4+ messages in thread
From: Chen-Yu Tsai @ 2017-03-06 8:44 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Mar 3, 2017 at 3:12 PM, Maxime Ripard
<maxime.ripard@free-electrons.com> wrote:
> On Thu, Mar 02, 2017 at 05:43:57PM +0000, Peter Robinson wrote:
>> Tighten the depends on the various AllWinn SoCs so we don't
>> inadvertantly get clock drivers when we're not wanting them
>> like 32 bit SoC clocks for 64 bit configs. Ensure there's
>> still test coverage though.
>>
>> Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
>
> Applied, thanks!
> Maxime
Just built your for-next branch:
$ make -j8 zImage dtbs modules
scripts/kconfig/conf --silentoldconfig Kconfig
drivers/clk/sunxi-ng/Kconfig:67: syntax error
drivers/clk/sunxi-ng/Kconfig:66: invalid option
drivers/clk/sunxi-ng/Kconfig:79: syntax error
drivers/clk/sunxi-ng/Kconfig:78: invalid option
drivers/clk/sunxi-ng/Kconfig:91: syntax error
drivers/clk/sunxi-ng/Kconfig:90: invalid option
drivers/clk/sunxi-ng/Kconfig:104: syntax error
drivers/clk/sunxi-ng/Kconfig:103: invalid option
drivers/clk/sunxi-ng/Kconfig:117: syntax error
drivers/clk/sunxi-ng/Kconfig:116: invalid option
drivers/clk/sunxi-ng/Kconfig:129: syntax error
drivers/clk/sunxi-ng/Kconfig:128: invalid option
drivers/clk/sunxi-ng/Kconfig:141: syntax error
drivers/clk/sunxi-ng/Kconfig:140: invalid option
drivers/clk/sunxi-ng/Kconfig:152: syntax error
drivers/clk/sunxi-ng/Kconfig:151: invalid option
scripts/kconfig/Makefile:37: recipe for target 'silentoldconfig' failed
make[2]: *** [silentoldconfig] Error 1
Makefile:548: recipe for target 'silentoldconfig' failed
make[1]: *** [silentoldconfig] Error 2
make: *** No rule to make target 'include/config/auto.conf', needed by
'include/config/kernel.release'. Stop.
make: *** Waiting for unfinished jobs....
This patch is the culprit. The proper syntax is "depends on", not "depends".
Regards
ChenYu
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] clk: sunxi-ng: tighten SoC deps on explicit AllWinner SoCs
2017-03-02 17:43 [PATCH] clk: sunxi-ng: tighten SoC deps on explicit AllWinner SoCs Peter Robinson
2017-03-03 7:12 ` Maxime Ripard
@ 2017-03-06 9:06 ` Maxime Ripard
1 sibling, 0 replies; 4+ messages in thread
From: Maxime Ripard @ 2017-03-06 9:06 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Thu, Mar 02, 2017 at 05:43:57PM +0000, Peter Robinson wrote:
> Tighten the depends on the various AllWinn SoCs so we don't
> inadvertantly get clock drivers when we're not wanting them
> like 32 bit SoC clocks for 64 bit configs. Ensure there's
> still test coverage though.
>
> Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
It turns out that this was making Kconfig crash, because the keyword
is depends on and not depends.
I fixed it, but please make sure the syntax is correct next time.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170306/71c49958/attachment.sig>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-03-06 9:06 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-02 17:43 [PATCH] clk: sunxi-ng: tighten SoC deps on explicit AllWinner SoCs Peter Robinson
2017-03-03 7:12 ` Maxime Ripard
2017-03-06 8:44 ` Chen-Yu Tsai
2017-03-06 9:06 ` Maxime Ripard
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).