On Thu, Aug 08, 2019 at 03:16:53PM +0200, Stefan Agner wrote:
On 2019-08-08 15:14, Takashi Iwai wrote:
Mark Brown wrote:
I guess we can use dev_printk() with the conditional level choice.
How about use dev_info always? We get a dev_err message from soc_init_dai_link in error cases...
ret = soc_init_dai_link(card, dai_link); if (ret && ret != -EPROBE_DEFER) { dev_info(card->dev, "ASoC: failed to init link %s: %d\n", dai_link->name, ret); }
Well, if there's adequate error reporting in init_dai_link() it's a bit different - we can just remove the print entirely regardless of what the return code is. The point is to ensure that we don't just silently fail. Unfortunately there's no prints in the probe deferral case there so they need adding, that'll actually improve things though since we can make it print the name of the thing it's mising which will be useful to people trying to figure out what's going on (we used to do that but it got lost in reshufflings).