diff options
author | Takashi Sakamoto <o-takashi@sakamocchi.jp> | 2015-05-22 16:21:13 +0200 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2015-05-24 08:23:43 +0200 |
commit | 12e0f438d312fd40815249198ec1e4f3abddacc5 (patch) | |
tree | 9bb2576dd6a9242cd9411390b0e01462fd5d3ecb /sound/firewire/amdtp.c | |
parent | ALSA: firewire-lib: macro arrangement for code cleanup (diff) | |
download | linux-12e0f438d312fd40815249198ec1e4f3abddacc5.tar.xz linux-12e0f438d312fd40815249198ec1e4f3abddacc5.zip |
ALSA: firewire-lib: use dev_err() when detecting incoming streaming error
When detecting invalid value in 'dbs' field of CIP header or packet
discontinuity, current implementation reports the status by err_info().
In most cases this state is caused by model-specific issue due to
vendor's customization and should be reported to developers.
This commit use dev_err() instead of dev_info() for this purpose.
In the cases, packet streaming is aborted, thus no message floading
occurs.
Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Acked-by: Clemens Ladisch <clemens@ladisch.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to '')
-rw-r--r-- | sound/firewire/amdtp.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sound/firewire/amdtp.c b/sound/firewire/amdtp.c index 29efbdad572f..93cf93a66aed 100644 --- a/sound/firewire/amdtp.c +++ b/sound/firewire/amdtp.c @@ -723,7 +723,7 @@ static int handle_in_packet(struct amdtp_stream *s, (cip_header[0] & CIP_DBS_MASK) >> CIP_DBS_SHIFT; /* avoid division by zero */ if (data_block_quadlets == 0) { - dev_info_ratelimited(&s->unit->device, + dev_err(&s->unit->device, "Detect invalid value in dbs field: %08X\n", cip_header[0]); return -EIO; @@ -756,9 +756,9 @@ static int handle_in_packet(struct amdtp_stream *s, } if (lost) { - dev_info(&s->unit->device, - "Detect discontinuity of CIP: %02X %02X\n", - s->data_block_counter, data_block_counter); + dev_err(&s->unit->device, + "Detect discontinuity of CIP: %02X %02X\n", + s->data_block_counter, data_block_counter); return -EIO; } |