Skip to contentSkip to conversation

clk: fix compile testing of scpi clk driver

PATCH New Heiner Kallweit 3 participants 5 messages Nov 28, 2017 – Dec 4, 2017 Common clk

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 clk: fix compile testing of scpi clk driver

We now have a hard dependency on ARM_SCPI_PROTOCOL and may end up with the following error:
ERROR: "get_scpi_ops" [drivers/clk/clk-scpi.ko] undefined! Fix this by removing COMPILE_TEST from COMMON_CLK_SCPI dependencies.

Fixes f7fee8be06a6 "firmware: arm_scpi: remove default for get_scpi_ops if not reachable"
---
 drivers/clk/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/clk/Kconfig +1 -1
index 1c4e1aa67..fe6fd705f 100644
@@ -64,7 +64,7 @@ config COMMON_CLK_HI655X
 
 config COMMON_CLK_SCPI
 	tristate "Clock driver controlled via SCPI interface"
-	depends on ARM_SCPI_PROTOCOL || COMPILE_TEST
+	depends on ARM_SCPI_PROTOCOL
 	  ---help---
 	  This driver provides support for clocks that are controlled
 	  by firmware that implements the SCPI interface.
Signature · 1 line
2.15.0
#1 lore raw

PATCH 1/1 clk: fix compile testing of scpi clk driver

Quoted 6 lines · Heiner Kallweit

We now have a hard dependency on ARM_SCPI_PROTOCOL and may end up with the following error:
ERROR: "get_scpi_ops" [drivers/clk/clk-scpi.ko] undefined! Fix this by removing COMPILE_TEST from COMMON_CLK_SCPI dependencies.

Fixes f7fee8be06a6 "firmware: arm_scpi: remove default for get_scpi_ops if not reachable"

I don't see this commit anywhere. Is this a future issue?

Signature · 2 lines
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
#2 lore raw

PATCH 1/1 clk: fix compile testing of scpi clk driver

Am 29.11.2017 um 18:04 schrieb Stephen Boyd:

Quoted 10 lines · Heiner Kallweit
Quoted 6 lines

We now have a hard dependency on ARM_SCPI_PROTOCOL and may end up with the following error:
ERROR: "get_scpi_ops" [drivers/clk/clk-scpi.ko] undefined! Fix this by removing COMPILE_TEST from COMMON_CLK_SCPI dependencies.

Fixes f7fee8be06a6 "firmware: arm_scpi: remove default for get_scpi_ops if not reachable"

I don't see this commit anywhere. Is this a future issue?

Sorry, yes it is and the patch was primarily addressed to Sudeep. See here:
https://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux.git/log/?h=for-next/scpi

#3 lore raw

PATCH 1/1 clk: fix compile testing of scpi clk driver

Quoted 7 lines · Heiner Kallweit

Am 29.11.2017 um 18:04 schrieb Stephen Boyd:

Quoted 6 lines · Heiner Kallweit
Quoted 5 lines

We now have a hard dependency on ARM_SCPI_PROTOCOL and may end up with the following error:
ERROR: "get_scpi_ops" [drivers/clk/clk-scpi.ko] undefined! Fix this by removing COMPILE_TEST from COMMON_CLK_SCPI dependencies.

Quoted 6 lines
Quoted 4 lines
Quoted 1 line
Fixes f7fee8be06a6 "firmware: arm_scpi: remove default for get_scpi_ops if not reachable"

I don't see this commit anywhere. Is this a future issue?

Sorry, yes it is and the patch was primarily addressed to Sudeep. See here:
https://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux.git/log/?h=for-next/scpi

I was trying to figure the build regression that the above commit would cause as I could not find it myself but I was sure about the regression.

If I had pulled it officially ;), I would have responded to your original patch. Sorry for the name of the brnach, I should have used something else to avoid the confusion.

--
Regards,
Sudeep

#4 lore raw

PATCH 1/1 clk: fix compile testing of scpi clk driver

In reply to Heiner Kallweit
Quoted 6 lines · Heiner Kallweit

We now have a hard dependency on ARM_SCPI_PROTOCOL and may end up with the following error:
ERROR: "get_scpi_ops" [drivers/clk/clk-scpi.ko] undefined! Fix this by removing COMPILE_TEST from COMMON_CLK_SCPI dependencies.

Fixes f7fee8be06a6 "firmware: arm_scpi: remove default for get_scpi_ops if not reachable"

This doesn't exist for all practical purposes. I pushed a branch to test the commit as I was sure that it will cause build regression but I could not figure to get that config. The Kbuild bot did find the issue.

You are introducing the bug in f7fee8be06a6 and trying to fix here, both of them are wrong.

--
Regards,
Sudeep