[alsa-devel] [PATCH] ASoC: rl6347a: Use dev_err_once for I2C communication error prints

[Mark Brown]

On Wed, Oct 05, 2016 at 05:05:36PM +0300, Jarkko Nikula wrote:

> It's difficult to guess from bunch of "ret=-121" errors what driver and
> device actually caused them. Since struct i2c_client has the dev pointer
> use that for dev_err_once() in order to see the device and not spam the
> log needlessly.

dev_err() is obviously an improvement (as would a better message be) but
dev_err_once() seems like a big jump that we don't normally do for our
I/O errors - transient errors do happen from time to time (especially
when people get their power up sequences wrong) and it can be useful to
know if something is just dead, failed gradually over time or just had a
single transient error.  It's also helpful since a lot of the errors
that do occur result in long timeouts so it's not always clear that
we're grinding through lots of I/Os if we don't say anything.  Am I
missing something about this case?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 455 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20161006/1666217c/attachment.sig>