[3/4] arm64: dts: qcom: msm8916: Add clock properties to the APCS node

Message ID 20180205144651.5934-3-georgi.djakov@linaro.org
State Accepted
Commit 025b995f47daece7b9856b9509748db02a0971be
Headers show
Series
  • [1/4] arm64: dts: qcom: msm8916: Add msm8916 A53 PLL DT node
Related show

Commit Message

Georgi Djakov Feb. 5, 2018, 2:46 p.m.
There are clock controller registers in the APCS block, which purpose
is to control the main CPU mux and divider. Add the clock properties as
part of the APCS device-tree node.

Signed-off-by: Georgi Djakov <georgi.djakov@linaro.org>

---
 arch/arm64/boot/dts/qcom/msm8916.dtsi | 2 ++
 1 file changed, 2 insertions(+)

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Amit Kucheria Feb. 7, 2018, 10 a.m. | #1
On Mon, Feb 5, 2018 at 8:16 PM, Georgi Djakov <georgi.djakov@linaro.org> wrote:
> There are clock controller registers in the APCS block, which purpose


If you respin this, s/which/whose

> is to control the main CPU mux and divider. Add the clock properties as

> part of the APCS device-tree node.

>

> Signed-off-by: Georgi Djakov <georgi.djakov@linaro.org>

> ---

>  arch/arm64/boot/dts/qcom/msm8916.dtsi | 2 ++

>  1 file changed, 2 insertions(+)

>

> diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi

> index 5414f53a0fa1..4539571a36b2 100644

> --- a/arch/arm64/boot/dts/qcom/msm8916.dtsi

> +++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi

> @@ -336,6 +336,8 @@

>                         compatible = "qcom,msm8916-apcs-kpss-global", "syscon";

>                         reg = <0xb011000 0x1000>;

>                         #mbox-cells = <1>;

> +                       clocks = <&a53pll>;

> +                       #clock-cells = <0>;

>                 };

>

>                 blsp1_uart2: serial@78b0000 {

> --

> To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in

> the body of a message to majordomo@vger.kernel.org

> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Andy Gross Feb. 27, 2018, 8:10 p.m. | #2
On Wed, Feb 07, 2018 at 03:30:47PM +0530, Amit Kucheria wrote:
> On Mon, Feb 5, 2018 at 8:16 PM, Georgi Djakov <georgi.djakov@linaro.org> wrote:

> > There are clock controller registers in the APCS block, which purpose

> 

> If you respin this, s/which/whose


I can fix this when I apply them.  I'll make a note.  No need to respin.

Regards,

Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi
index 5414f53a0fa1..4539571a36b2 100644
--- a/arch/arm64/boot/dts/qcom/msm8916.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi
@@ -336,6 +336,8 @@ 
 			compatible = "qcom,msm8916-apcs-kpss-global", "syscon";
 			reg = <0xb011000 0x1000>;
 			#mbox-cells = <1>;
+			clocks = <&a53pll>;
+			#clock-cells = <0>;
 		};
 
 		blsp1_uart2: serial@78b0000 {