Re: [PATCH] ASoC: ux500: do not moan on deferrals

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 23, 2017 at 7:15 PM, Mark Brown <broonie@xxxxxxxxxx> 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
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux