On Mon, Jan 23, 2017 at 7:15 PM, Mark Brown broonie@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