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