mgmt: Set offload codec mgmt feature
PATCH v2 1/1 mgmt: Set offload codec mgmt feature
Add mgmt support to toggle offload codec feature
---
changes in v2:
- fix warnings reported by checkpatch
- add uuid to /src/shared/util.c:uuid128_table
- remove sign-off from commit message
src/shared/util.c | 1 + tools/btmgmt.c | 44 ++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 45 insertions(+)
src/shared/util.c +1 -0
@@ -1027,6 +1027,7 @@ static const struct { "BlueZ Experimental LL privacy" }, {"330859bc-7506-492d-9370-9a6f0614037f" , "BlueZ Experimental Bluetooth Quality Report" }, + { "a6695ace-ee7f-4fb9-881a-5fac66c629af", "BlueZ Offload Codecs"}, { } };
tools/btmgmt.c +44 -0
56 lines not on this page · raw message
Signature · 1 line
2.17.1
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=532441
---Test result---
Test Summary:
CheckPatch PASS 0.35 seconds GitLint PASS 0.12 seconds Prep - Setup ELL PASS 48.10 seconds Build - Prep PASS 0.11 seconds Build - Configure PASS 8.40 seconds Build - Make PASS 208.49 seconds Make Check PASS 9.26 seconds Make Distcheck PASS 246.27 seconds Build w/ext ELL - Configure PASS 8.42 seconds Build w/ext ELL - Make PASS 194.12 seconds
Details
##############################
Test: CheckPatch - PASS
Desc: Run checkpatch.pl script with rule in .checkpatch.conf
##############################
Test: GitLint - PASS
Desc: Run gitlint with rule in .gitlint
##############################
Test: Prep - Setup ELL - PASS
Desc: Clone, build, and install ELL
##############################
Test: Build - Prep - PASS
Desc: Prepare environment for build
##############################
Test: Build - Configure - PASS
Desc: Configure the BlueZ source tree
##############################
Test: Build - Make - PASS
Desc: Build the BlueZ source tree
##############################
Test: Make Check - PASS
Desc: Run 'make check'
##############################
Test: Make Distcheck - PASS
Desc: Run distcheck to check the distribution
##############################
Test: Build w/ext ELL - Configure - PASS
Desc: Configure BlueZ source with '--enable-external-ell' configuration
##############################
Test: Build w/ext ELL - Make - PASS
Desc: Build BlueZ source with '--enable-external-ell' configuration
---
Regards,
Linux Bluetooth
Luiz Augusto von Dentz
maintainerHi Kiran,
Quoted 17 lines · Kiran K
Add mgmt support to toggle offload codec feature
---
changes in v2:
- fix warnings reported by checkpatch
- add uuid to /src/shared/util.c:uuid128_table
- remove sign-off from commit messagesrc/shared/util.c | 1 + tools/btmgmt.c | 44 ++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 45 insertions(+)src/shared/util.c +0
-0
src/shared/util.c static const struct { · line 1030
Quoted 4 lines
src/shared/util.c +1
-0@@ -1027,6 +1027,7 @@ static const struct { "BlueZ Experimental LL privacy" }, {"330859bc-7506-492d-9370-9a6f0614037f" , "BlueZ Experimental Bluetooth Quality Report" }, + { "a6695ace-ee7f-4fb9-881a-5fac66c629af", "BlueZ Offload Codecs"},
Please have it as a separate patch.
Quoted 68 lines
src/shared/util.c +0
-0{ } };tools/btmgmt.c +44
-1@@ -2554,6 +2554,48 @@ static void cmd_privacy(int argc, char **argv) } } +static void exp_offload_rsp (uint8_t status,uint16_t len,const void *param, +void *user_data) +{ +if (status !=0 ) +error ("Set offload codec failed with status 0x%02x (%s)" , + status,mgmt_errstr (status)); +else +"Offload codec feature successfully set" ); + +bt_shell_noninteractive_quit (EXIT_SUCCESS ); +} + +static void cmd_exp_offload_codecs (int argc,char **argv) +{ +/* a6695ace-ee7f-4fb9-881a-5fac66c629af */ +static const uint8_t uuid[16 ] = { +0xaf ,0x29 ,0xc6 ,0x66 ,0xac ,0x5f ,0x1a ,0x88 , +0xb9 ,0x4f ,0x7f ,0xee ,0xce ,0x5a ,0x69 ,0xa6 , + }; + +struct mgmt_cp_set_exp_feature cp; +uint8_t val; +uint16_t index; + +if (parse_setting (argc, argv, &val) == false) +return bt_shell_noninteractive_quit (EXIT_FAILURE ); + + index = mgmt_index; +if (index ==MGMT_INDEX_NONE ) + index =0 ; + +memset (&cp,0 ,sizeof (cp)); +memcpy (cp.uuid, uuid,16 ); + cp.action = val; + +if (mgmt_send (mgmt,MGMT_OP_SET_EXP_FEATURE , index, +sizeof (cp), &cp, exp_offload_rsp,NULL ,NULL ) ==0 ) { +error ("Unable to send offload codecs feature cmd" ); +return bt_shell_noninteractive_quit (EXIT_FAILURE ); + } +} +static void class_rsp (uint16_t op,uint16_t id,uint8_t status,uint16_t len,const void *param) {@@ -5595,6 +5637,8 @@ static const struct bt_shell_menu main_menu = { cmd_exp_privacy, "Set LL privacy feature" }, {"exp-quality" ,"<on/off>" , cmd_exp_quality, "Set bluetooth quality report feature" }, + { "exp-offload", "<on/off>", + cmd_exp_offload_codecs, "Toggle codec support" }, { "read-sysconfig", NULL, cmd_read_sysconfig, "Read System Configuration" }, { "set-sysconfig", "<-v|-h> [options...]", --2.17.1
Signature · 1 line
Luiz Augusto von Dentz
Hi Luiz,
Quoted 24 lines
-----Original Message-----
From: Luiz Augusto von Dentz <[email protected]>
Sent: Wednesday, August 18, 2021 5:19 AM
To: K, Kiran <[email protected]>
Cc: [email protected]; Srivatsa, Ravishankar <[email protected]>
Subject: Re: [PATCH v2] mgmt: Set offload codec mgmt featureHi Kiran,
Quoted 10 lines · Kiran K
Add mgmt support to toggle offload codec feature
---
changes in v2:
- fix warnings reported by checkpatch
- add uuid to /src/shared/util.c:uuid128_table
- remove sign-off from commit messagesrc/shared/util.c | 1 + tools/btmgmt.c | 44++++++++++++++++++++++++++++++++++++++++++++
Quoted 2 lines
2 files changed, 45 insertions(+)
Quoted 12 lines
src/shared/util.c static const struct { · line 1031
Quoted 10 lines
src/shared/util.c index renamed from src/shared/util.c +0
-0
854b48d38f09..2447564566ce100644src/shared/util.c +2
-0@@ -1027,6 +1027,7 @@ static const struct { "BlueZ Experimental LL privacy" }, {"330859bc-7506-492d-9370-9a6f0614037f" , "BlueZ Experimental Bluetooth Quality Report" }, + { "a6695ace-ee7f-4fb9-881a-5fac66c629af", "BlueZ Offload + Codecs"},Please have it as a separate patch.
Ack. I will send an updated patch.
Quoted 65 lines
tools/btmgmt.c static void cmd_privacy(int argc, char **argv) · line 2557
Quoted 13 lines
src/shared/util.c +0
-0{ } };tools/btmgmt.c index renamed from tools/btmgmt.c +0
-0
7d908238156d..a840c428cc74100644tools/btmgmt.c +1
-0@@ -2554,6 +2554,48 @@ static void cmd_privacy(int argc, char **argv) } } +static void exp_offload_rsp(uint8_t status, uint16_t len, const void*param,
Quoted 51 lines
tools/btmgmt.c +40
-0+void +*user_data) { +if (status !=0 ) +error ("Set offload codec failed with status 0x%02x (%s)" , + status,mgmt_errstr (status)); +else +"Offload codec feature successfully set" ); + +bt_shell_noninteractive_quit (EXIT_SUCCESS ); +} + +static void cmd_exp_offload_codecs (int argc,char **argv) { +/* a6695ace-ee7f-4fb9-881a-5fac66c629af */ +static const uint8_t uuid[16 ] = { +0xaf ,0x29 ,0xc6 ,0x66 ,0xac ,0x5f ,0x1a ,0x88 , +0xb9 ,0x4f ,0x7f ,0xee ,0xce ,0x5a ,0x69 ,0xa6 , + }; + +struct mgmt_cp_set_exp_feature cp; +uint8_t val; +uint16_t index; + +if (parse_setting (argc, argv, &val) == false) +return bt_shell_noninteractive_quit (EXIT_FAILURE ); + + index = mgmt_index; +if (index ==MGMT_INDEX_NONE ) + index =0 ; + +memset (&cp,0 ,sizeof (cp)); +memcpy (cp.uuid, uuid,16 ); + cp.action = val; + +if (mgmt_send (mgmt,MGMT_OP_SET_EXP_FEATURE , index, +sizeof (cp), &cp, exp_offload_rsp,NULL ,NULL ) ==0 ) { +error ("Unable to send offload codecs feature cmd" ); +return bt_shell_noninteractive_quit (EXIT_FAILURE ); + } +} +static void class_rsp (uint16_t op,uint16_t id,uint8_t status,uint16_t len,const void *param) { @@ -5595,6 +5637,8 @@ static const struct bt_shell_menu
main_menu= {
cmd_exp_privacy, "Set LL privacy feature" },
{ "exp-quality", "<on/off>",cmd_exp_quality,
"Set bluetooth quality report feature" },tools/btmgmt.c +2
-0+ {"exp-offload" ,"<on/off>" , + cmd_exp_offload_codecs, "Toggle codec support" }, { "read-sysconfig", NULL, cmd_read_sysconfig, "Read System Configuration" },
Quoted 8 lines
Quoted 4 lines
{ "set-sysconfig", "<-v|-h> [options...]",--
2.17.1--
Luiz Augusto von Dentz
Thanks,
Kiran