On 16/09/15 03:04, Yingjoe Chen wrote:
From: Daniel Kurtz <[email protected]>

Add device node to enable GPT timer. This timer will be
used as sched clock source.


Interesting any known issues with or advantage over the arch timers
to prefer it as sched clock source. I see even arch timers are present
in DT, hence the question. Or is it just a incorrect commit log ?

How does this get selected as sched clock source ? I don't see
sched_clock_register in mtk_timer.c

To be clear, I am not against adding this timer support, but just want
to know is it preferred for sched clock source ? if yes why ? better
resolution ?

Change-Id: Idc4e3f0ee80b5c36cae6f0f2328f94aafcca1253

^ Should be dropped

Signed-off-by: Daniel Kurtz <[email protected]>
Signed-off-by: Eddie Huang <[email protected]>
Signed-off-by: Yingjoe Chen <[email protected]>
---
  arch/arm64/boot/dts/mediatek/mt8173.dtsi | 9 +++++++++
  1 file changed, 9 insertions(+)

diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi 
b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
index d18ee42..d763803 100644
--- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
@@ -238,6 +238,15 @@
                        reg = <0 0x10007000 0 0x100>;
                };

+               timer: timer@10008000 {
+                       compatible = "mediatek,mt8173-timer",

Missing documentation ? I am referring upstream and it might be in some patches already queued perhaps ?

Regards,
Sudeep
--
To unsubscribe from this list: send the line "unsubscribe linux-clk" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to