Re: [PATCH 5/6] Watchdog: introdouce ARM SBSA watchdog driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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.

--
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the
Code Aurora Forum, hosted by The Linux Foundation.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux