linux-arm-msm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] Drop the clock-frequency property from the MMIO timer node
@ 2022-02-02 16:35 Kathiravan T
  2022-02-02 16:35 ` [PATCH 1/2] arm64: dts: qcom: ipq8074: drop the clock-frequency property Kathiravan T
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Kathiravan T @ 2022-02-02 16:35 UTC (permalink / raw)
  To: agross, bjorn.andersson, robh+dt, linux-arm-msm, devicetree,
	linux-kernel
  Cc: Kathiravan T

Drop the clock-frequency property from the MMIO timer node in both IPQ8074
and IPQ6018 DTSI. There are couple of reasons for this change

1) As per the DT binding, this property should be present only if the
firmware is not configuring the clock frequency properly. In both IPQ8074
and IPQ6018, bootloader is configuring it properly.

2) For IPQ6018, clock frequency should be 24MHz, not 19.2MHz. Rather than
correcting it, drop it based on the above mentioned point.

Kathiravan T (2):
  arm64: dts: qcom: ipq8074: drop the clock-frequency property
  arm64: dts: qcom: ipq6018: drop the clock-frequency property

 arch/arm64/boot/dts/qcom/ipq6018.dtsi | 1 -
 arch/arm64/boot/dts/qcom/ipq8074.dtsi | 1 -
 2 files changed, 2 deletions(-)

-- 
2.7.4


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/2] arm64: dts: qcom: ipq8074: drop the clock-frequency property
  2022-02-02 16:35 [PATCH 0/2] Drop the clock-frequency property from the MMIO timer node Kathiravan T
@ 2022-02-02 16:35 ` Kathiravan T
  2022-02-02 16:35 ` [PATCH 2/2] arm64: dts: qcom: ipq6018: " Kathiravan T
  2022-02-11 17:10 ` [PATCH 0/2] Drop the clock-frequency property from the MMIO timer node patchwork-bot+linux-arm-msm
  2 siblings, 0 replies; 4+ messages in thread
From: Kathiravan T @ 2022-02-02 16:35 UTC (permalink / raw)
  To: agross, bjorn.andersson, robh+dt, linux-arm-msm, devicetree,
	linux-kernel
  Cc: Kathiravan T

Drop the clock-frequency property from the MMIO timer node, since it
is already configured by the bootloader.

Signed-off-by: Kathiravan T <quic_kathirav@quicinc.com>
---
 arch/arm64/boot/dts/qcom/ipq8074.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/ipq8074.dtsi b/arch/arm64/boot/dts/qcom/ipq8074.dtsi
index 26ba7ce9222c..a0ff4de8ea75 100644
--- a/arch/arm64/boot/dts/qcom/ipq8074.dtsi
+++ b/arch/arm64/boot/dts/qcom/ipq8074.dtsi
@@ -647,7 +647,6 @@
 			ranges;
 			compatible = "arm,armv7-timer-mem";
 			reg = <0x0b120000 0x1000>;
-			clock-frequency = <19200000>;
 
 			frame@b120000 {
 				frame-number = <0>;
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 2/2] arm64: dts: qcom: ipq6018: drop the clock-frequency property
  2022-02-02 16:35 [PATCH 0/2] Drop the clock-frequency property from the MMIO timer node Kathiravan T
  2022-02-02 16:35 ` [PATCH 1/2] arm64: dts: qcom: ipq8074: drop the clock-frequency property Kathiravan T
@ 2022-02-02 16:35 ` Kathiravan T
  2022-02-11 17:10 ` [PATCH 0/2] Drop the clock-frequency property from the MMIO timer node patchwork-bot+linux-arm-msm
  2 siblings, 0 replies; 4+ messages in thread
From: Kathiravan T @ 2022-02-02 16:35 UTC (permalink / raw)
  To: agross, bjorn.andersson, robh+dt, linux-arm-msm, devicetree,
	linux-kernel
  Cc: Kathiravan T

clock-frequency for IPQ6018 SoCs should be 24MHz, not 19.2MHz. Rather
than correcting it, drop the property itself since its already
configured by the bootloader.

Signed-off-by: Kathiravan T <quic_kathirav@quicinc.com>
---
 arch/arm64/boot/dts/qcom/ipq6018.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
index 66ec5615651d..2803b060126b 100644
--- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
+++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
@@ -520,7 +520,6 @@
 			ranges;
 			compatible = "arm,armv7-timer-mem";
 			reg = <0x0 0x0b120000 0x0 0x1000>;
-			clock-frequency = <19200000>;
 
 			frame@b120000 {
 				frame-number = <0>;
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 0/2] Drop the clock-frequency property from the MMIO timer node
  2022-02-02 16:35 [PATCH 0/2] Drop the clock-frequency property from the MMIO timer node Kathiravan T
  2022-02-02 16:35 ` [PATCH 1/2] arm64: dts: qcom: ipq8074: drop the clock-frequency property Kathiravan T
  2022-02-02 16:35 ` [PATCH 2/2] arm64: dts: qcom: ipq6018: " Kathiravan T
@ 2022-02-11 17:10 ` patchwork-bot+linux-arm-msm
  2 siblings, 0 replies; 4+ messages in thread
From: patchwork-bot+linux-arm-msm @ 2022-02-11 17:10 UTC (permalink / raw)
  To: Kathiravan T; +Cc: linux-arm-msm

Hello:

This series was applied to qcom/linux.git (for-next)
by Bjorn Andersson <bjorn.andersson@linaro.org>:

On Wed, 2 Feb 2022 22:05:07 +0530 you wrote:
> Drop the clock-frequency property from the MMIO timer node in both IPQ8074
> and IPQ6018 DTSI. There are couple of reasons for this change
> 
> 1) As per the DT binding, this property should be present only if the
> firmware is not configuring the clock frequency properly. In both IPQ8074
> and IPQ6018, bootloader is configuring it properly.
> 
> [...]

Here is the summary with links:
  - [1/2] arm64: dts: qcom: ipq8074: drop the clock-frequency property
    https://git.kernel.org/qcom/c/555ab09c7896
  - [2/2] arm64: dts: qcom: ipq6018: drop the clock-frequency property
    https://git.kernel.org/qcom/c/01b8c4aff332

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-02-11 17:10 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-02 16:35 [PATCH 0/2] Drop the clock-frequency property from the MMIO timer node Kathiravan T
2022-02-02 16:35 ` [PATCH 1/2] arm64: dts: qcom: ipq8074: drop the clock-frequency property Kathiravan T
2022-02-02 16:35 ` [PATCH 2/2] arm64: dts: qcom: ipq6018: " Kathiravan T
2022-02-11 17:10 ` [PATCH 0/2] Drop the clock-frequency property from the MMIO timer node patchwork-bot+linux-arm-msm

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).