Re: [PATCH 2/4] monitor: add device filter support

[Vinicius Costa Gomes <vinicius.gomes@openbossa.org>]

Hi Gustavo,

On 13:18 Fri 11 May, Gustavo Padovan wrote:
> -i hciX option will show only data from the specified device
> ---
>  monitor/main.c   |   12 ++++++++++--
>  monitor/packet.c |   10 +++++++++-
>  monitor/packet.h |    2 +-
>  3 files changed, 20 insertions(+), 4 deletions(-)
> 
> diff --git a/monitor/main.c b/monitor/main.c
> index 90e32c5..99b53e9 100644
> --- a/monitor/main.c
> +++ b/monitor/main.c
> @@ -28,6 +28,7 @@
>  
>  #include <stdio.h>
>  #include <stdlib.h>
> +#include <string.h>
>  #include <getopt.h>
>  
>  #include "mainloop.h"
> @@ -52,12 +53,14 @@ static void usage(void)
>  		"Usage:\n");
>  	printf("\tbtmon [options]\n");
>  	printf("options:\n"
> +		"\t-i dev                HCI device\n"
>  		"\t-b, --btsnoop <file>  Save dump in btsnoop format\n"
>  		"\t-h, --help            Show help options\n");
>  }
>  
>  static const struct option main_options[] = {
>  	{ "btsnoop",	required_argument, NULL, 'b'	},
> +	{ "",		required_argument, NULL, 'i'	},

btmgmt is calling this "index", how about using the same long name for
consistency?

[snip]


Cheers,
-- 
Vinicius