diff mbox series

[BlueZ,1/2] monitor: Fix handling of BT_HCI_EVT_LE_BIG_COMPLETE

Message ID 20210223190904.1718994-1-luiz.dentz@gmail.com (mailing list archive)
State New, archived
Headers show
Series [BlueZ,1/2] monitor: Fix handling of BT_HCI_EVT_LE_BIG_COMPLETE | expand

Commit Message

Luiz Augusto von Dentz Feb. 23, 2021, 7:09 p.m. UTC 
From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

This adds the missing fields for BT_HCI_EVT_LE_BIG_COMPLETE event.
---
 monitor/bt.h     | 6 ++++++
 monitor/packet.c | 6 ++++++
 2 files changed, 12 insertions(+)

Comments

bluez.test.bot@gmail.com Feb. 23, 2021, 7:37 p.m. UTC | #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=437331

---Test result---

##############################
Test: CheckPatch - PASS

##############################
Test: CheckGitLint - PASS

##############################
Test: CheckBuild - FAIL
Output:
configure.ac:21: installing './compile'
configure.ac:33: installing './config.guess'
configure.ac:33: installing './config.sub'
configure.ac:5: installing './install-sh'
configure.ac:5: installing './missing'
Makefile.am: installing './depcomp'
parallel-tests: installing './test-driver'
configure: error: Embedded Linux library >= 0.37 is required


##############################
Test: MakeCheck - SKIPPED
Output:
checkbuild not success



---
Regards,
Linux Bluetooth
An, Tedd Feb. 23, 2021, 10:47 p.m. UTC | #2 
Hi Luiz

On 2/23/21, 11:40 AM, "bluez.test.bot@gmail.com" <bluez.test.bot@gmail.com> wrote:

    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=437331

    ---Test result---

    ##############################
    Test: CheckPatch - PASS

    ##############################
    Test: CheckGitLint - PASS

    ##############################
    Test: CheckBuild - FAIL
    Output:
    configure.ac:21: installing './compile'
    configure.ac:33: installing './config.guess'
    configure.ac:33: installing './config.sub'
    configure.ac:5: installing './install-sh'
    configure.ac:5: installing './missing'
    Makefile.am: installing './depcomp'
    parallel-tests: installing './test-driver'
    configure: error: Embedded Linux library >= 0.37 is required

There was an issue with Docker image and it is resolved by now.
Please wait for the next test result.

    ##############################
    Test: MakeCheck - SKIPPED
    Output:
    checkbuild not success



    ---
    Regards,
    Linux Bluetooth
Luiz Augusto von Dentz Feb. 23, 2021, 10:56 p.m. UTC | #3 
Hi,

On Tue, Feb 23, 2021 at 11:37 AM <bluez.test.bot@gmail.com> wrote:
>
> 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=437331
>
> ---Test result---
>
> ##############################
> Test: CheckPatch - PASS
>
> ##############################
> Test: CheckGitLint - PASS
>
> ##############################
> Test: CheckBuild - FAIL
> Output:
> configure.ac:21: installing './compile'
> configure.ac:33: installing './config.guess'
> configure.ac:33: installing './config.sub'
> configure.ac:5: installing './install-sh'
> configure.ac:5: installing './missing'
> Makefile.am: installing './depcomp'
> parallel-tests: installing './test-driver'
> configure: error: Embedded Linux library >= 0.37 is required
>
>
> ##############################
> Test: MakeCheck - SKIPPED
> Output:
> checkbuild not success
>
>
>
> ---
> Regards,
> Linux Bluetooth

Pushed.
bluez.test.bot@gmail.com Feb. 23, 2021, 10:59 p.m. UTC | #4 
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=437331

---Test result---

##############################
Test: CheckPatch - PASS

##############################
Test: CheckGitLint - PASS

##############################
Test: CheckBuild - PASS

##############################
Test: MakeCheck - PASS



---
Regards,
Linux Bluetooth
diff mbox series

Patch

diff --git a/monitor/bt.h b/monitor/bt.h
index aca847d7c..394dd7a2d 100644
--- a/monitor/bt.h
+++ b/monitor/bt.h
@@ -3533,6 +3533,12 @@  struct bt_hci_evt_le_big_complete {
 	uint8_t  sync_delay[3];
 	uint8_t  latency[3];
 	uint8_t  phy;
+	uint8_t  nse;
+	uint8_t  bn;
+	uint8_t  pto;
+	uint8_t  irc;
+	uint16_t max_pdu;
+	uint16_t interval;
 	uint8_t  num_bis;
 	uint16_t handle[0];
 } __attribute__ ((packed));
diff --git a/monitor/packet.c b/monitor/packet.c
index fcd698d92..55ba16ccc 100644
--- a/monitor/packet.c
+++ b/monitor/packet.c
@@ -10575,6 +10575,12 @@  static void le_big_complete_evt(const void *data, uint8_t size)
 	print_usec_interval("BIG Synchronization Delay", evt->sync_delay);
 	print_usec_interval("Transport Latency", evt->latency);
 	print_le_phy("PHY", evt->phy);
+	print_field("NSE: %u", evt->nse);
+	print_field("BN: %u", evt->bn);
+	print_field("PTO: %u", evt->bn);
+	print_field("IRC: %u", evt->irc);
+	print_field("Maximum PDU: %u", evt->max_pdu);
+	print_slot_125("ISO Interval", evt->interval);
 	print_list(evt->handle, size, evt->num_bis, sizeof(*evt->handle),
 						print_bis_handle);
 }