Hi Marcel,
On 05/25/2015 02:52 AM, Marcel Holtmann wrote:
Hi Florian,
patch set introducing the infrastructure for multi-advertising
capability to the HCI core and mgmt API.
the patch set is actually breaking functionality. If you want to overwrite the advertising data of an existing instance, then you can just do that by issuing another management command.
add-adv -u 180d 1
add-adv -u 180f 1
This will first enable advertising with the heart rate UUID and then with the battery service UUID. With your patch set, this basic feature does not work anymore.
I'm going to check this, write a test against this case and make sure
the problem is being fixed.
It's true that I relied on the test suite to already contain tests for
all basic features. That's why I did my regression testing there.
I think first we need a lot more mgmt-tester commands that actually test this functionality.
Sure. I can introduce missing tests for prior features that are not yet
being tested. You can see that I already added a test case earlier that
was obviously missing (already accepted).
Therefore an important question to you: Which other cases should be
tested? It's hard for me to find that out without your help.
Regards
Marcel
Thanks!
Florian
--
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
