Re: [Linaro-acpi] [PATCH v3 6/7] Watchdog: add debug option into Kconfig/Makefile

Hi Timur,

thanks , I just feel it's convenient if we have that, it is a option, it can be ignore or merge, :-)

see if people like it :-)

On 12 May 2015 at 22:40, Timur Tabi timur@codeaurora.org wrote:

fu.wei@linaro.org wrote:

...

+config WATCHDOG_DEBUG

•   bool "Debug support for WATCHDOG drivers"
  

•   depends on DEBUG_KERNEL
  

•   help
  

•     Say "yes" to enable debug messaging (like dev_dbg and pr_debug),
  

•     sysfs, and debugfs support in WatchDog controller and protocol
  

drivers.

I personally don't like Kconfig debugging options like that. If I want to debug a driver, I'll just add "#define DEBUG" to the top of the file and rebuild.

-- 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.