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

Mark Brown broonie at kernel.org
Tue Jan 24 16:45:32 CET 2017


On Tue, Jan 24, 2017 at 01:49:15PM +0100, Linus Walleij wrote:
> On Mon, Jan 23, 2017 at 7:15 PM, Mark Brown <broonie at kernel.org> wrote:

> > 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.

I'm guessing so but yeah.  Actually one thing I'd really like to do at
some point is introduce an error to string conversion function and use
that more widely for printing error messages, it's trivial and makes
reading log messages *so* much more easy in general if the computer does
the decoding for you if you get -EPROBE_DEFER instead of -517.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20170124/a47b6c36/attachment.sig>


More information about the Alsa-devel mailing list