Re: [PATCH wpan-tools 0/2] Update cca mode and options output

[Varka Bhadram <varkabhadram@gmail.com>]

Hi Christoffer Holmstedt,

On 06/04/2015 02:50 PM, Christoffer Holmstedt wrote:

> Earlier cca_modes and cca_opts were printed separately but these two patches
> now prints them together so cca_opts are only printed when the specified mode
>   is (3).
>
> Output for "current" mode (only one line is printed for each wpan):
> 	cca_mode: 1, energy above threshold.
> 	cca_mode: 2, carrier sense only.
> 	cca_mode: 3, carrier sense with energy above threshold (logical operator is 'and').
> 	cca_mode: 3, carrier sense with energy above threshold (logical operator is 'or').
> 	cca_mode: 3, carrier sense with energy above threshold (logical operator is unknown).
> 	cca_mode: 4, ALOHA.
> 	cca_mode: 5, UWB preamble sense based on the SHR of a frame.
> 	cca_mode: 6, UWB preamble sense based on the packet with the multiplexed preamble.

Do you think that cca_mode number is required overhear..?

What i mean is that:
	The cca mode number 1 indicates that its *energy above threshold*,
So what i feel is that printing the number is required.

May be Alex has to tell its preferred or not. I think we can remove the number.

>
> capabilities:
> 	...
> 	cca_modes:
> 		(1) energy above threshold
> 		(2) carrier sense only
> 		(3, cca_opt: 0) carrier sense with energy above threshold (logical operator is 'and')
> 		(3, cca_opt: 1) carrier sense with energy above threshold (logical operator is 'or')
>
>
> Christoffer Holmstedt (2):
>    info: add cca mode descriptive text to output
>    info: remove old cca options capabilities output
>
>   src/info.c | 99 +++++++++++++++++++++++++++++++++++++-------------------------
>   1 file changed, 60 insertions(+), 39 deletions(-)
>

-- 
Varka Bhadram