[PATCH 3/7] debug: use printk() instead pr_debug()

Kalle Valo kvalo at qca.qualcomm.com
Thu May 16 10:20:58 EDT 2013


Eugene Krasnikov <k.eugene.e at gmail.com> writes:

> Why not to use dynamic debug? it has more flexibility let's say i want
> to enable or disable debugging only in specific function. How would
> you do that with this implementation?

To me the problem with dynamic debugging is that it's either too fine
grained (line level) or too vast (file level). When working with
wireless drivers most of the time you need something in-between.

So instead of enabling a certain debug message you enable certain log
level (or levels) related to the task at hand. For example, when trying
to debug a tranmission stall you would most likely enable
WCN36XX_DBG_DXE and WCN36XX_DBG_TX (0x41) using the debug_mask module
parameter, or any other level combination you want.

-- 
Kalle Valo



More information about the wcn36xx mailing list