diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-05-28 12:43:30 -0400 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2014-06-01 12:16:04 -0400 |
| commit | 33a5f989de41622ae691b12f008390309b59ad74 (patch) | |
| tree | 0475b9d045c9695584c1c093907cb74debba6f48 | |
| parent | 14577c25169beb8676577490bf2f0bd539d5e999 (diff) | |
ALSA: bebob: sizeof() vs ARRAY_SIZE() typo
ARRAY_SIZE() was intended here instead of sizeof(). The
"bridgeco_freq_table" array holds integers so the original condition is
never true.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewd-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Tested-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
| -rw-r--r-- | sound/firewire/bebob/bebob_stream.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/firewire/bebob/bebob_stream.c b/sound/firewire/bebob/bebob_stream.c index 514c7c9fa1cd..bc4f82776fda 100644 --- a/sound/firewire/bebob/bebob_stream.c +++ b/sound/firewire/bebob/bebob_stream.c | |||
| @@ -749,7 +749,7 @@ parse_stream_formation(u8 *buf, unsigned int len, | |||
| 749 | if (buf[2] == bridgeco_freq_table[i]) | 749 | if (buf[2] == bridgeco_freq_table[i]) |
| 750 | break; | 750 | break; |
| 751 | } | 751 | } |
| 752 | if (i == sizeof(bridgeco_freq_table)) | 752 | if (i == ARRAY_SIZE(bridgeco_freq_table)) |
| 753 | return -ENOSYS; | 753 | return -ENOSYS; |
| 754 | 754 | ||
| 755 | /* Avoid double count by different entries for the same rate. */ | 755 | /* Avoid double count by different entries for the same rate. */ |
