* [PATCH v1] clk: qcom: msm8916: Don't build support by default
@ 2019-06-12 15:52 Marc Gonzalez
2019-06-12 19:13 ` Bjorn Andersson
0 siblings, 1 reply; 6+ messages in thread
From: Marc Gonzalez @ 2019-06-12 15:52 UTC (permalink / raw)
To: Stephen Boyd, Michael Turquette
Cc: linux-clk, MSM, Georgi Djakov, Amit Kucheria, Bjorn Andersson
Build QCOM_A53PLL and QCOM_CLK_APCS_MSM8916 by default only when
we're building MSM_GCC_8916.
Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
---
drivers/clk/qcom/Kconfig | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig
index e1ff83cc361e..d5b065f64afc 100644
--- a/drivers/clk/qcom/Kconfig
+++ b/drivers/clk/qcom/Kconfig
@@ -21,7 +21,7 @@ if COMMON_CLK_QCOM
config QCOM_A53PLL
tristate "MSM8916 A53 PLL"
- default ARCH_QCOM
+ default MSM_GCC_8916
help
Support for the A53 PLL on MSM8916 devices. It provides
the CPU with frequencies above 1GHz.
@@ -31,7 +31,7 @@ config QCOM_A53PLL
config QCOM_CLK_APCS_MSM8916
tristate "MSM8916 APCS Clock Controller"
depends on QCOM_APCS_IPC || COMPILE_TEST
- default ARCH_QCOM
+ default MSM_GCC_8916
help
Support for the APCS Clock Controller on msm8916 devices. The
APCS is managing the mux and divider which feeds the CPUs.
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v1] clk: qcom: msm8916: Don't build support by default
2019-06-12 15:52 [PATCH v1] clk: qcom: msm8916: Don't build support by default Marc Gonzalez
@ 2019-06-12 19:13 ` Bjorn Andersson
2019-06-12 19:49 ` Stephen Boyd
2019-06-13 15:05 ` Marc Gonzalez
0 siblings, 2 replies; 6+ messages in thread
From: Bjorn Andersson @ 2019-06-12 19:13 UTC (permalink / raw)
To: Marc Gonzalez
Cc: Stephen Boyd, Michael Turquette, linux-clk, MSM, Georgi Djakov,
Amit Kucheria
On Wed 12 Jun 08:52 PDT 2019, Marc Gonzalez wrote:
> Build QCOM_A53PLL and QCOM_CLK_APCS_MSM8916 by default only when
> we're building MSM_GCC_8916.
>
> Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
Not sure why these are default at all.
But both drivers are used on platforms other than 8916 as well, so if
anything a fix would be to rename the APCS_MSM8916 to something more
generic (such as QCOM_CLK_APCS_GLOBAL) - but then the content should be
updated and the APCS mailbox driver as well...
Regards,
Bjorn
> ---
> drivers/clk/qcom/Kconfig | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig
> index e1ff83cc361e..d5b065f64afc 100644
> --- a/drivers/clk/qcom/Kconfig
> +++ b/drivers/clk/qcom/Kconfig
> @@ -21,7 +21,7 @@ if COMMON_CLK_QCOM
>
> config QCOM_A53PLL
> tristate "MSM8916 A53 PLL"
> - default ARCH_QCOM
> + default MSM_GCC_8916
> help
> Support for the A53 PLL on MSM8916 devices. It provides
> the CPU with frequencies above 1GHz.
> @@ -31,7 +31,7 @@ config QCOM_A53PLL
> config QCOM_CLK_APCS_MSM8916
> tristate "MSM8916 APCS Clock Controller"
> depends on QCOM_APCS_IPC || COMPILE_TEST
> - default ARCH_QCOM
> + default MSM_GCC_8916
> help
> Support for the APCS Clock Controller on msm8916 devices. The
> APCS is managing the mux and divider which feeds the CPUs.
> --
> 2.17.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v1] clk: qcom: msm8916: Don't build support by default
2019-06-12 19:13 ` Bjorn Andersson
@ 2019-06-12 19:49 ` Stephen Boyd
2019-06-12 20:03 ` Marc Gonzalez
2019-06-13 15:05 ` Marc Gonzalez
1 sibling, 1 reply; 6+ messages in thread
From: Stephen Boyd @ 2019-06-12 19:49 UTC (permalink / raw)
To: Bjorn Andersson, Marc Gonzalez
Cc: Michael Turquette, linux-clk, MSM, Georgi Djakov, Amit Kucheria
Quoting Bjorn Andersson (2019-06-12 12:13:47)
> On Wed 12 Jun 08:52 PDT 2019, Marc Gonzalez wrote:
>
> > Build QCOM_A53PLL and QCOM_CLK_APCS_MSM8916 by default only when
> > we're building MSM_GCC_8916.
> >
> > Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
>
> Not sure why these are default at all.
>
> But both drivers are used on platforms other than 8916 as well, so if
> anything a fix would be to rename the APCS_MSM8916 to something more
> generic (such as QCOM_CLK_APCS_GLOBAL) - but then the content should be
> updated and the APCS mailbox driver as well...
>
I don't see any use in being this specific. I'd prefer we just leave
this at the ARCH_FOO config level and not try anything more fancy.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v1] clk: qcom: msm8916: Don't build support by default
2019-06-12 19:49 ` Stephen Boyd
@ 2019-06-12 20:03 ` Marc Gonzalez
2019-06-12 22:42 ` Stephen Boyd
0 siblings, 1 reply; 6+ messages in thread
From: Marc Gonzalez @ 2019-06-12 20:03 UTC (permalink / raw)
To: Stephen Boyd, Bjorn Andersson
Cc: Michael Turquette, linux-clk, MSM, Georgi Djakov, Amit Kucheria
On 12/06/2019 21:49, Stephen Boyd wrote:
> Quoting Bjorn Andersson (2019-06-12 12:13:47)
>
>> On Wed 12 Jun 08:52 PDT 2019, Marc Gonzalez wrote:
>>
>>> Build QCOM_A53PLL and QCOM_CLK_APCS_MSM8916 by default only when
>>> we're building MSM_GCC_8916.
>>>
>>> Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
>>
>> Not sure why these are default at all.
>>
>> But both drivers are used on platforms other than 8916 as well, so if
>> anything a fix would be to rename the APCS_MSM8916 to something more
>> generic (such as QCOM_CLK_APCS_GLOBAL) - but then the content should be
>> updated and the APCS mailbox driver as well...
>
> I don't see any use in being this specific. I'd prefer we just leave
> this at the ARCH_FOO config level and not try anything more fancy.
As Bjorn pointed out, why do these default "on" at all?
https://elixir.bootlin.com/linux/latest/source/drivers/clk/qcom/Kconfig
There are dozens of config knobs in drivers/clk/qcom/Kconfig
and only these two force the default.
Let's remove the default altogether.
Regards.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v1] clk: qcom: msm8916: Don't build support by default
2019-06-12 20:03 ` Marc Gonzalez
@ 2019-06-12 22:42 ` Stephen Boyd
0 siblings, 0 replies; 6+ messages in thread
From: Stephen Boyd @ 2019-06-12 22:42 UTC (permalink / raw)
To: Bjorn Andersson, Marc Gonzalez
Cc: Michael Turquette, linux-clk, MSM, Georgi Djakov, Amit Kucheria
Quoting Marc Gonzalez (2019-06-12 13:03:22)
> On 12/06/2019 21:49, Stephen Boyd wrote:
>
> > Quoting Bjorn Andersson (2019-06-12 12:13:47)
> >
> >> On Wed 12 Jun 08:52 PDT 2019, Marc Gonzalez wrote:
> >>
> >>> Build QCOM_A53PLL and QCOM_CLK_APCS_MSM8916 by default only when
> >>> we're building MSM_GCC_8916.
> >>>
> >>> Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
> >>
> >> Not sure why these are default at all.
> >>
> >> But both drivers are used on platforms other than 8916 as well, so if
> >> anything a fix would be to rename the APCS_MSM8916 to something more
> >> generic (such as QCOM_CLK_APCS_GLOBAL) - but then the content should be
> >> updated and the APCS mailbox driver as well...
> >
> > I don't see any use in being this specific. I'd prefer we just leave
> > this at the ARCH_FOO config level and not try anything more fancy.
>
> As Bjorn pointed out, why do these default "on" at all?
>
> https://elixir.bootlin.com/linux/latest/source/drivers/clk/qcom/Kconfig
>
> There are dozens of config knobs in drivers/clk/qcom/Kconfig
> and only these two force the default.
>
> Let's remove the default altogether.
>
Sure.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v1] clk: qcom: msm8916: Don't build support by default
2019-06-12 19:13 ` Bjorn Andersson
2019-06-12 19:49 ` Stephen Boyd
@ 2019-06-13 15:05 ` Marc Gonzalez
1 sibling, 0 replies; 6+ messages in thread
From: Marc Gonzalez @ 2019-06-13 15:05 UTC (permalink / raw)
To: Bjorn Andersson
Cc: Stephen Boyd, Michael Turquette, linux-clk, MSM, Georgi Djakov,
Amit Kucheria
On 12/06/2019 21:13, Bjorn Andersson wrote:
> On Wed 12 Jun 08:52 PDT 2019, Marc Gonzalez wrote:
>
>> Build QCOM_A53PLL and QCOM_CLK_APCS_MSM8916 by default only when
>> we're building MSM_GCC_8916.
>>
>> Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
>
> Not sure why these are default at all.
>
> But both drivers are used on platforms other than 8916 as well, so if
> anything a fix would be to rename the APCS_MSM8916 to something more
> generic (such as QCOM_CLK_APCS_GLOBAL) - but then the content should be
> updated and the APCS mailbox driver as well...
Used on platforms other than 8916? do you see that?
$ git grep compatible drivers/clk/qcom/a53-pll.c
{ .compatible = "qcom,msm8916-a53pll" },
$ git grep qcom,msm8916-a53pll arch/arm64/boot/dts
arch/arm64/boot/dts/qcom/msm8916.dtsi: compatible = "qcom,msm8916-a53pll";
drivers/clk/qcom/apcs-msm8916.c doesn't seem to support DT...
$ git grep qcom-apcs-msm8916-clk
drivers/clk/qcom/apcs-msm8916.c: .name = "qcom-apcs-msm8916-clk",
drivers/mailbox/qcom-apcs-ipc-mailbox.c: "qcom-apcs-msm8916-clk",
if (of_device_is_compatible(np, "qcom,msm8916-apcs-kpss-global")) {
apcs->clk = platform_device_register_data(&pdev->dev, "qcom-apcs-msm8916-clk", -1, NULL, 0);
$ git grep qcom,msm8916-apcs-kpss-global
Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.txt: "qcom,msm8916-apcs-kpss-global",
Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.txt: compatible = "qcom,msm8916-apcs-kpss-global";
arch/arm64/boot/dts/qcom/msm8916.dtsi: compatible = "qcom,msm8916-apcs-kpss-global", "syscon";
drivers/mailbox/qcom-apcs-ipc-mailbox.c: if (of_device_is_compatible(np, "qcom,msm8916-apcs-kpss-global")) {
drivers/mailbox/qcom-apcs-ipc-mailbox.c: { .compatible = "qcom,msm8916-apcs-kpss-global", .data = (void *)8 },
Are you sure about other platforms?
Regards.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-06-13 17:10 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-12 15:52 [PATCH v1] clk: qcom: msm8916: Don't build support by default Marc Gonzalez
2019-06-12 19:13 ` Bjorn Andersson
2019-06-12 19:49 ` Stephen Boyd
2019-06-12 20:03 ` Marc Gonzalez
2019-06-12 22:42 ` Stephen Boyd
2019-06-13 15:05 ` Marc Gonzalez
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.