RE: [PATCH] Add PICS, PIXITs and PTS test results for L2CAP

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

 



Hi Johan,

[ '#' notation ]
True and I can simply change it however there was (is) one thing which prevented me from doing this in the first place: having set any PICS which is not in place however it will be eventually implemented to False and marking it (#), gives clear indication what are the development objectives and at the same time some not relevant PTS tests are simply N/A. That is of benefit to anyone who wishes to test L2CAP in current state. 

[testing tools]
Good point. I was thinking about having in place some more tools besides l2test. I was considering i.e. hcitool or bluetoothctl however it wasn't straight forward to bring these to Android.mk (i.e. readline).
Anyhow having l2test (in Android) was straight forward so I'm planning to do the second round of L2CAP testing. 

In essence my idea was to do testing (L2CAP or anything for the matter) in repetitive way so surely first txt could lack some elements however with next iterations it would improve (i.e. we would have more and more test tools and scripts in place on Android so we could better describe and understand needs for certain set of PTS test). 

To summarize: please let me know what is you stand on [ '#' notation ] - I'll act accordingly and then I'll push L2CAP PTS test results once I finish second round of testing.

Thanks,
Seb

Hi Sebastian,

On Sun, Dec 15, 2013, Sebastian Chlad wrote:
> +* - different than PTS defaults
> +# - not yet implemented/supported
> +
> +M - mandatory
> +O - optional
> +
> +             Roles
> +-------------------------------------------------------------------------------
> +Parameter Name       Selected        Description
> +-------------------------------------------------------------------------------
> +TSPC_L2CAP_1_1       True            Data Channel Initiator (C.1)
> +TSPC_L2CAP_1_2       True            Data Channel Acceptor (C.1)
> +TSPC_L2CAP_1_3       False (#)       LE Master (C.2)
> +TSPC_L2CAP_1_4       False (#)       LE Slave (C.2)

I thought the idea of the '#' notation was that we could mark entries as
True if they eventually will be supported but are not yet implemented.
Wouldn't these last two entries fall under this category. 

> +-------------------------------------------------------------------------------
> +Test Name              Result  Notes
> +-------------------------------------------------------------------------------
> +TC_COS_CED_BV_01_C     INC     IUT should introduce testing tools
> +TC_COS_CED_BV_03_C     INC     IUT should introduce testing tools

What does the note here mean. l2test? If so wouldn't it be more helpful
to list the exact l2test command that's needs to be run?

Johan
---------------------------------------------------------------------
Intel Finland Oy
Registered Address: PL 281, 00181 Helsinki 
Business Identity Code: 0357606 - 4 
Domiciled in Helsinki 

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.

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




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux