On 06/07/2018 01:31 AM, Dan Carpenter wrote:
I think the num_loops variable was intended to be signed so that we could set it to -1 and it would loop forever. Right now it just loops for a very long time. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> diff --git a/tools/iio/iio_generic_buffer.c b/tools/iio/iio_generic_buffer.c index 3040830d7797..84545666a09c 100644 --- a/tools/iio/iio_generic_buffer.c +++ b/tools/iio/iio_generic_buffer.c @@ -330,7 +330,7 @@ static const struct option longopts[] = {int main(int argc, char **argv){ - unsigned long long num_loops = 2; + long long num_loops = 2; unsigned long timedelay = 1000000; unsigned long buf_len = 128;
Good catch, thanks. Reviewed-by: Martin Kelly <mkelly@xxxxxxxx> -- 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
