Re: [PATCH] sound: soc-core: make kernel complaints on -EPROBE_DEFER dev_dbg

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

 



On Wed, Nov 09, 2016 at 03:22:09PM +0000, Mark Brown wrote:
> On Wed, Nov 09, 2016 at 04:14:26PM +0100, Ladislav Michl wrote:
> > On Wed, Nov 09, 2016 at 02:36:42PM +0000, Mark Brown wrote:
> 
> > > No, errors are errors and not displaying them just makes it harder for
> > > people to debug things.  If you don't want to see errors just change
> > > your system configuratiion to hide them.
> 
> > For sure I want to see all errors, but this is not hardware error nor
> > kernel misconfiguration, so showing it to the user is a bit pointless.
> 
> How do we know that it's not a kernel misconfiguration?  It's common for
> people to not build some of the component drivers they need.

Is what you described really a misconfiguration? Enabling debug when
something does not work seems obvious thing to do, but okay, perhaps
anything bellow error level would make me happy enough.

> > > If you don't like deferred probing please contribute to the efforts
> > > to order probing.

As a side note, which efforts are you reffering to here?
 
> > I just tried to make it consistend to other subsystems where patches to
> > silence deferred probing warnings are accepted...
> 
> Which subsystems are these?  We should look at fixing them...

tty and usb for example. I do not consider wise to looking at them until
this very subsystem gets fixed first to not distract ourselves ;-)
(Also I have admit, that accepted patches hide error message on deferred
probe only, but above occurs _also_ on deferred probe and yes, it would be
nice to have that fixed)

Best regards,
	ladis
_______________________________________________
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