Re: [PATCH] android: Build btiotest tool

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

 



Hi Marcel, Szymon,

On Mon, Jun 16, 2014 at 10:40 PM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote:
> Hi Szymon,
>
>>>> This is useful for testing.
>>>> ---
>>>> android/Android.mk | 32 ++++++++++++++++++++++++++++++++
>>>> 1 file changed, 32 insertions(+)
>>>>
>>>> diff --git a/android/Android.mk b/android/Android.mk
>>>> index fc7b59d..ecff9c8 100644
>>>> --- a/android/Android.mk
>>>> +++ b/android/Android.mk
>>>> @@ -620,3 +620,35 @@ LOCAL_MODULE := avinfo
>>>> LOCAL_ADDITIONAL_DEPENDENCIES := $(LOCAL_PATH)/bluez/configure.ac
>>>>
>>>> include $(BUILD_EXECUTABLE)
>>>> +
>>>> +#
>>>> +# btiotest
>>>> +#
>>>> +
>>>> +include $(CLEAR_VARS)
>>>> +
>>>> +LOCAL_SRC_FILES := \
>>>> +   bluez/tools/btiotest.c \
>>>> +   bluez/btio/btio.c \
>>>> +   bluez/lib/bluetooth.c \
>>>
>>> I would actually prefer that we get rid of btio and integrated this in
>>> struct io properly. However what can be tested with btiotest that can not
>>> be tested with l2test and other tools.
>>
>> This is mostly for tests related to (updating) connection security level.
>>
>> We should be able to craft custom GATT HAL test command for that, although
>> this is not really related to GATT...
>
> if it helps right now, then include the tool. However keep in mind that long term all the GLib based stuff has to go away. And especially the Android part of BlueZ should be GLib free first. This means if you are missing test functionality, lets get it into the existing test tools.
>

For now we need it for couple tests. I think we can already try to
cover it with l2test if btiotest is on the black list ;)

> Regards
>
> Marcel
>
> --
> 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

\Lukasz
--
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