[BlueZ PATCH] tools/mgmt-tester: Fix Read Experiemental Feature test case

Subsystems: the rest

STALE1806d

3 messages, 3 authors, 2021-09-29 · open the first message on its own page

[BlueZ PATCH] tools/mgmt-tester: Fix Read Experiemental Feature test case

From: Tedd Ho-Jeong An <hidden>
Date: 2021-09-29 21:00:55

From: Tedd Ho-Jeong An <redacted>

The quality report feature is supported only if the device support it.
Current emulator/btdev doesn't support it yet.

This patch updates the supported experimental feature list to align with
the current btdev implementation.
---
 tools/mgmt-tester.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/tools/mgmt-tester.c b/tools/mgmt-tester.c
index 9c5b26c9a..e62c4dcdc 100644
--- a/tools/mgmt-tester.c
+++ b/tools/mgmt-tester.c
@@ -9334,7 +9334,7 @@ static const struct generic_data set_dev_flags_fail_3 = {
 };
 
 static const uint8_t read_exp_feat_param_success[] = {
-	0x04, 0x00,				/* Feature Count */
+	0x03, 0x00,				/* Feature Count */
 	0xd6, 0x49, 0xb0, 0xd1, 0x28, 0xeb,	/* UUID - Simultaneous */
 	0x27, 0x92, 0x96, 0x46, 0xc0, 0x42,	/* Central Peripheral */
 	0xb5, 0x10, 0x1b, 0x67,
@@ -9343,14 +9343,10 @@ static const uint8_t read_exp_feat_param_success[] = {
 	0xde, 0xb3, 0xea, 0x11, 0x73, 0xc2,
 	0x48, 0xa1, 0xc0, 0x15,
 	0x02, 0x00, 0x00, 0x00,			/* Flags */
-	0x7f, 0x03, 0x14, 0x06, 0x6f, 0x9a,	/* UUID - Quality Report */
-	0x70, 0x93, 0x2d, 0x49, 0x06, 0x75,
-	0xbc, 0x59, 0x08, 0x33,
-	0x00, 0x00, 0x00, 0x00,			/* Flags */
 	0xaf, 0x29, 0xc6, 0x66, 0xac, 0x5f,	/* UUID - Codec Offload */
 	0x1a, 0x88, 0xb9, 0x4f, 0x7f, 0xee,
 	0xce, 0x5a, 0x69, 0xa6,
-	0x01, 0x00, 0x00, 0x00			/* Flags */
+	0x00, 0x00, 0x00, 0x00			/* Flags */
 };
 
 static const struct generic_data read_exp_feat_success = {
-- 
2.25.1

RE: [BlueZ] tools/mgmt-tester: Fix Read Experiemental Feature test case

From: <hidden>
Date: 2021-09-29 21:32:32

This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=555241

---Test result---

Test Summary:
CheckPatch                    PASS      1.29 seconds
GitLint                       PASS      0.92 seconds
Prep - Setup ELL              PASS      41.56 seconds
Build - Prep                  PASS      0.48 seconds
Build - Configure             PASS      7.62 seconds
Build - Make                  PASS      177.64 seconds
Make Check                    PASS      9.26 seconds
Make Distcheck                PASS      212.31 seconds
Build w/ext ELL - Configure   PASS      7.82 seconds
Build w/ext ELL - Make        PASS      168.61 seconds



---
Regards,
Linux Bluetooth

Re: [BlueZ PATCH] tools/mgmt-tester: Fix Read Experiemental Feature test case

From: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date: 2021-09-29 22:44:43

Hi Tedd,

On Wed, Sep 29, 2021 at 2:34 PM Tedd Ho-Jeong An [off-list ref] wrote:
quoted hunk
From: Tedd Ho-Jeong An <redacted>

The quality report feature is supported only if the device support it.
Current emulator/btdev doesn't support it yet.

This patch updates the supported experimental feature list to align with
the current btdev implementation.
---
 tools/mgmt-tester.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/tools/mgmt-tester.c b/tools/mgmt-tester.c
index 9c5b26c9a..e62c4dcdc 100644
--- a/tools/mgmt-tester.c
+++ b/tools/mgmt-tester.c
@@ -9334,7 +9334,7 @@ static const struct generic_data set_dev_flags_fail_3 = {
 };

 static const uint8_t read_exp_feat_param_success[] = {
-       0x04, 0x00,                             /* Feature Count */
+       0x03, 0x00,                             /* Feature Count */
        0xd6, 0x49, 0xb0, 0xd1, 0x28, 0xeb,     /* UUID - Simultaneous */
        0x27, 0x92, 0x96, 0x46, 0xc0, 0x42,     /* Central Peripheral */
        0xb5, 0x10, 0x1b, 0x67,
@@ -9343,14 +9343,10 @@ static const uint8_t read_exp_feat_param_success[] = {
        0xde, 0xb3, 0xea, 0x11, 0x73, 0xc2,
        0x48, 0xa1, 0xc0, 0x15,
        0x02, 0x00, 0x00, 0x00,                 /* Flags */
-       0x7f, 0x03, 0x14, 0x06, 0x6f, 0x9a,     /* UUID - Quality Report */
-       0x70, 0x93, 0x2d, 0x49, 0x06, 0x75,
-       0xbc, 0x59, 0x08, 0x33,
-       0x00, 0x00, 0x00, 0x00,                 /* Flags */
        0xaf, 0x29, 0xc6, 0x66, 0xac, 0x5f,     /* UUID - Codec Offload */
        0x1a, 0x88, 0xb9, 0x4f, 0x7f, 0xee,
        0xce, 0x5a, 0x69, 0xa6,
-       0x01, 0x00, 0x00, 0x00                  /* Flags */
+       0x00, 0x00, 0x00, 0x00                  /* Flags */
 };

 static const struct generic_data read_exp_feat_success = {
--
2.25.1
Applied, thanks.

-- 
Luiz Augusto von Dentz
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help