clk: fix compile testing of scpi clk driver
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.
--- drivers/clk/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/clk/Kconfig +1 -1
@@ -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
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_PROTOCOLand may end up with the following error:
ERROR: "get_scpi_ops" [drivers/clk/clk-scpi.ko] undefined! Fix this by removingCOMPILE_TESTfromCOMMON_CLK_SCPIdependencies.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
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_PROTOCOLand may end up with the following error:
ERROR: "get_scpi_ops" [drivers/clk/clk-scpi.ko] undefined! Fix this by removingCOMPILE_TESTfromCOMMON_CLK_SCPIdependencies.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
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_PROTOCOLand may end up with the following error:
ERROR: "get_scpi_ops" [drivers/clk/clk-scpi.ko] undefined! Fix this by removingCOMPILE_TESTfromCOMMON_CLK_SCPIdependencies.
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
PATCH 1/1 clk: fix compile testing of scpi clk driver
In reply to Heiner KallweitQuoted 6 lines · Heiner Kallweit
We now have a hard dependency on
ARM_SCPI_PROTOCOLand may end up with the following error:
ERROR: "get_scpi_ops" [drivers/clk/clk-scpi.ko] undefined! Fix this by removingCOMPILE_TESTfromCOMMON_CLK_SCPIdependencies.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