On 2021-03-05 15:29, Mark Brown wrote:
On Fri, Mar 05, 2021 at 12:24:02PM +0530, Laxminath Kasam wrote:
Add debug print useful for issue analysis in soc-dapm. This print for dapm widgets useful for debugging to find widgets powering up and down.
There's already fairly extensive tracepoints for DAPM - this seems to correspond to snd_soc_dapm_widget_power, though it's at the application time rather than the queue time so we could add another tracepoint for this if you like? I can see that the separate tracepoint would be useful, I mostly use the regmap tracepoints for that purpose but not every widget has register I/O. Tracepoints are less noisy for the system as a whole than tracing every DAPM operation in dmesg, that tends to irritate other users if it's left in as standard.
Thanks Mark for the response. But this is not default err/info print and only dynamic debug print which should not result in noise. In case of customer OEMs/final product versions where less options of debugging, dynamic dmesg logging is preferable provided faster and this print is used to identify issues debug fastly sometimes.
-- Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc., is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.