Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1205545 > unrolled thread
| Started by | Stephen Boyd <sboyd@codeaurora.org> |
|---|---|
| First post | 2015-08-12 03:40 +0200 |
| Last post | 2015-08-18 23:50 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH 1/2] ARM: dts: vexpress: Use assigned-clock-parents for sp810 Stephen Boyd <sboyd@codeaurora.org> - 2015-08-12 03:40 +0200
Re: [PATCH 1/2] ARM: dts: vexpress: Use assigned-clock-parents for sp810 Olof Johansson <olof@lixom.net> - 2015-08-18 23:50 +0200
| From | Stephen Boyd <sboyd@codeaurora.org> |
|---|---|
| Date | 2015-08-12 03:40 +0200 |
| Subject | [PATCH 1/2] ARM: dts: vexpress: Use assigned-clock-parents for sp810 |
| Message-ID | <pWtbj-5Z7-5@gated-at.bofh.it> |
The sp810 clk driver is calling the clk consumer APIs from clk_prepare ops to change the parent to a 1 MHz fixed rate clock for each of the clocks that the driver provides. Use assigned-clock-parents for this instead of doing it in the driver to avoid using the consumer API in provider code. This also allows us to remove the usage of clk provider APIs that take a struct clk as an argument from the sp810 driver. Cc: Pawel Moll <pawel.moll@arm.com> Cc: Linus Walleij <linus.walleij@linaro.org> Tested-by: Sudeep Holla <sudeep.holla@arm.com> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> --- This came out of a larger series to change this binding[1]. This patch can go through arm-soc independently though, so please apply. [1] http://lkml.kernel.org/r/1438386285-5070-1-git-send-email-sboyd@codeaurora.org arch/arm/boot/dts/vexpress-v2m-rs1.dtsi | 2 ++ arch/arm/boot/dts/vexpress-v2m.dtsi | 2 ++ 2 files changed, 4 insertions(+) diff --git a/arch/arm/boot/dts/vexpress-v2m-rs1.dtsi b/arch/arm/boot/dts/vexpress-v2m-rs1.dtsi index 2efb2058ba49..21b02874bea3 100644 --- a/arch/arm/boot/dts/vexpress-v2m-rs1.dtsi +++ b/arch/arm/boot/dts/vexpress-v2m-rs1.dtsi @@ -101,6 +101,8 @@ clock-names = "refclk", "timclk", "apb_pclk"; #clock-cells = <1>; clock-output-names = "timerclken0", "timerclken1", "timerclken2", "timerclken3"; + assigned-clocks = <&v2m_sysctl 0>, <&v2m_sysctl 1>, <&v2m_sysctl 3>, <&v2m_sysctl 3>; + assigned-clock-parents = <&v2m_refclk1mhz>, <&v2m_refclk1mhz>, <&v2m_refclk1mhz>, <&v2m_refclk1mhz>; }; /* PCI-E I2C bus */ diff --git a/arch/arm/boot/dts/vexpress-v2m.dtsi b/arch/arm/boot/dts/vexpress-v2m.dtsi index cb3090f919a7..e712c0af149b 100644 --- a/arch/arm/boot/dts/vexpress-v2m.dtsi +++ b/arch/arm/boot/dts/vexpress-v2m.dtsi @@ -100,6 +100,8 @@ clock-names = "refclk", "timclk", "apb_pclk"; #clock-cells = <1>; clock-output-names = "timerclken0", "timerclken1", "timerclken2", "timerclken3"; + assigned-clocks = <&v2m_sysctl 0>, <&v2m_sysctl 1>, <&v2m_sysctl 3>, <&v2m_sysctl 3>; + assigned-clock-parents = <&v2m_refclk1mhz>, <&v2m_refclk1mhz>, <&v2m_refclk1mhz>, <&v2m_refclk1mhz>; }; /* PCI-E I2C bus */ -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Olof Johansson <olof@lixom.net> |
|---|---|
| Date | 2015-08-18 23:50 +0200 |
| Subject | Re: [PATCH 1/2] ARM: dts: vexpress: Use assigned-clock-parents for sp810 |
| Message-ID | <pYWVz-2nZ-15@gated-at.bofh.it> |
| In reply to | #1205545 |
On Tue, Aug 11, 2015 at 06:36:50PM -0700, Stephen Boyd wrote: > The sp810 clk driver is calling the clk consumer APIs from > clk_prepare ops to change the parent to a 1 MHz fixed rate clock > for each of the clocks that the driver provides. Use > assigned-clock-parents for this instead of doing it in the driver > to avoid using the consumer API in provider code. This also > allows us to remove the usage of clk provider APIs that take a > struct clk as an argument from the sp810 driver. > > Cc: Pawel Moll <pawel.moll@arm.com> > Cc: Linus Walleij <linus.walleij@linaro.org> > Tested-by: Sudeep Holla <sudeep.holla@arm.com> > Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> > --- > > This came out of a larger series to change this binding[1]. This > patch can go through arm-soc independently though, so please apply. Applied, thanks. -Olof -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web