linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [BlueZ PATCH] tools/mgmt-tester: Fix Read Experiemental Feature test case
@ 2021-09-29 21:00 Tedd Ho-Jeong An
  2021-09-29 21:32 ` [BlueZ] " bluez.test.bot
  2021-09-29 22:44 ` [BlueZ PATCH] " Luiz Augusto von Dentz
  0 siblings, 2 replies; 3+ messages in thread
From: Tedd Ho-Jeong An @ 2021-09-29 21:00 UTC (permalink / raw)
  To: linux-bluetooth

From: Tedd Ho-Jeong An <tedd.an@intel.com>

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


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* RE: [BlueZ] tools/mgmt-tester: Fix Read Experiemental Feature test case
  2021-09-29 21:00 [BlueZ PATCH] tools/mgmt-tester: Fix Read Experiemental Feature test case Tedd Ho-Jeong An
@ 2021-09-29 21:32 ` bluez.test.bot
  2021-09-29 22:44 ` [BlueZ PATCH] " Luiz Augusto von Dentz
  1 sibling, 0 replies; 3+ messages in thread
From: bluez.test.bot @ 2021-09-29 21:32 UTC (permalink / raw)
  To: linux-bluetooth, hj.tedd.an

[-- Attachment #1: Type: text/plain, Size: 884 bytes --]

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


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [BlueZ PATCH] tools/mgmt-tester: Fix Read Experiemental Feature test case
  2021-09-29 21:00 [BlueZ PATCH] tools/mgmt-tester: Fix Read Experiemental Feature test case Tedd Ho-Jeong An
  2021-09-29 21:32 ` [BlueZ] " bluez.test.bot
@ 2021-09-29 22:44 ` Luiz Augusto von Dentz
  1 sibling, 0 replies; 3+ messages in thread
From: Luiz Augusto von Dentz @ 2021-09-29 22:44 UTC (permalink / raw)
  To: Tedd Ho-Jeong An; +Cc: linux-bluetooth

Hi Tedd,

On Wed, Sep 29, 2021 at 2:34 PM Tedd Ho-Jeong An <hj.tedd.an@gmail.com> wrote:
>
> From: Tedd Ho-Jeong An <tedd.an@intel.com>
>
> 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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-09-29 22:44 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-29 21:00 [BlueZ PATCH] tools/mgmt-tester: Fix Read Experiemental Feature test case Tedd Ho-Jeong An
2021-09-29 21:32 ` [BlueZ] " bluez.test.bot
2021-09-29 22:44 ` [BlueZ PATCH] " Luiz Augusto von Dentz

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).