[alsa-devel] [PATCH] ASoC: ux500: do not moan on deferrals

Linus Walleij linus.walleij at linaro.org
Tue Jan 24 13:49:15 CET 2017


On Mon, Jan 23, 2017 at 7:15 PM, Mark Brown <broonie at kernel.org> wrote:
> On Fri, Jan 20, 2017 at 02:20:11PM +0100, Linus Walleij wrote:

>> This removes the "error" print on probe deferral, it's
>> unnecessary and confusing. Also cut a few debug prints we do
>> not need.
>
> I know I took the other patch doing this (along with some other changes)
> but in general I'm not super comfortable with just silently ignoring
> probe deferral errors as it makes it hard to debug practical systems -
> the device is there, the driver is there but it fails to instantiate
> with no indication as to why.

You're right. Let's drop this, the only improvement could I guess
be to print explicitly that things get deferred instead, unless by now
every kernel developer in the world knows what -517 means.

Yours,
Linus Walleij


More information about the Alsa-devel mailing list