On Fri, Feb 28, 2020 at 06:45:33PM +0100, Krzysztof Kozlowski wrote:
On Fri, 28 Feb 2020 at 17:12, Mark Brown broonie@kernel.org wrote:
This is completely removing the diagnostics so if things do get stuck the user will generally not have any hint as to what's going wrong - it's better with this sort of change to lower to dev_dbg() for deferral rather than completely removing the error message so the user has some ability to figure out what was missing. However in this case it's also the case that snd_soc_register_card() is fairly noisy.
This dev_dbg could be achieved with ftrace for probe() as well. I think it is quite common to skip all messages for deferred probes.
Yes... and having had the pleasure of debugging problems with things not instantiating I'm not a fan of this trend.