Fu Wei wrote:
yes. you are right. According to SBSA doc, the clocksource of SBSA watchdog is System Counter. And System Counter is in (arm_)arch_timer. So I think we should do
depends on ARM_ARCH_TIMER
and use the relevant interface :
clk = arch_timer_get_rate();
will improve it, thanks for your suggestion!
If you use arch_timer_get_rate(), then you will not be able to compile the driver as a module.
The clock API doesn't work for me, either, because no clocks are defined (clk_get_sys() always fails).
That's why I use arch_timer_get_cntfrq().
I would probably drop the various pr_debug() calls here. Once the driver works fine, they are normally not that useful any more.
yes, for this drive, if it works fine, we can drop it, but I keep these info for some reason: (1)they can help engineer debug GTDT table or DTS, if the info of watchdog goes wrong.
Any engineer will add his own printks when debugging. You don't need to do that job for someone else. You just have too many pr_debug() statements.