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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 04, 2015 at 05:21:25PM +0530, Varka Bhadram wrote:
> 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.
> 
...cut...
> -- 
> Varka Bhadram
> 

I was convinced before that I needed it there, but then I tried to make my
case and I ended up with the conclusion that the user don't need it. Only
scenario it is needed is when the user wants to change it and it that case the
number is listed in the capabilities section.

I will remove the number if no one objects.

-- 
Christoffer Holmstedt
--
To unsubscribe from this list: send the line "unsubscribe linux-wpan" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux