[alsa-devel] [PATCH] ak4642: show error if register write fails

Lars-Peter Clausen lars at metafoo.de
Tue Mar 11 14:59:55 CET 2014

On 03/11/2014 12:21 PM, Mark Brown wrote:
> On Tue, Mar 11, 2014 at 11:18:30AM +0000, Ben Dooks wrote:
>> On 10/03/14 23:40, Mark Brown wrote:
>>> Two things here.  One is that this should be a dev_err() and the other
>>> is that if this is worth doing shouldn't it just be in the core - I see
>>> nothing driver specific here?
>> Sorry, didn't see a device in "struct snd_soc_codec *codec" so I went
>> for a printk (although it was pr_info instead of pr_err).
> codec->dev.
>> If you think that changing the two snd_soc calls to print errors
>> when anything bad happens then that would also be a good idea then
>> I can send a patch for that.
> That would be better, yes.

In my opinion it's better to pass the error on to the upper levels. E.g. if 
userspace opens the PCM device and there is an IO error in the startup 
callback then that error should be passed on to the userspace application 
rather than doing a out of band error reporting and adding a entry to the 
kernel log.

- Lars

More information about the Alsa-devel mailing list