(May 19 2014 01:43), Dan Carpenter wrote: > 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@xxxxxxxxxx> Reviewd-by: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx> Tested-by: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx> > diff --git a/sound/firewire/bebob/bebob_stream.c b/sound/firewire/bebob/bebob_stream.c > index 514c7c9..bc4f827 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, > if (buf[2] == bridgeco_freq_table[i]) > break; > } > - if (i == sizeof(bridgeco_freq_table)) > + if (i == ARRAY_SIZE(bridgeco_freq_table)) > return -ENOSYS; > > /* Avoid double count by different entries for the same rate. */ -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
