| Message ID | 20230620224354.729781-1-luiz.dentz@gmail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [BlueZ,1/2] monitor: Fix decoding of HCI CIS Established Event | expand |
| Context | Check | Description |
|---|---|---|
| tedd_an/pre-ci_am | success | Success |
| tedd_an/CheckPatch | success | CheckPatch PASS |
| tedd_an/GitLint | success | Gitlint PASS |
| tedd_an/BuildEll | success | Build ELL PASS |
| tedd_an/BluezMake | success | Bluez Make PASS |
| tedd_an/MakeCheck | success | Bluez Make Check PASS |
| tedd_an/MakeDistcheck | success | Make Distcheck PASS |
| tedd_an/CheckValgrind | success | Check Valgrind PASS |
| tedd_an/CheckSmatch | warning | CheckSparse WARNING monitor/packet.c: note: in included file:monitor/display.h:82:26: warning: Variable length array is used.monitor/packet.c:1832:26: warning: Variable length array is used.monitor/packet.c: note: in included file:monitor/bt.h:3552:52: warning: array of flexible structuresmonitor/bt.h:3540:40: warning: array of flexible structuresemulator/btdev.c:417:29: warning: Variable length array is used. |
| tedd_an/bluezmakeextell | success | Make External ELL PASS |
| tedd_an/IncrementalBuild | success | Incremental Build PASS |
| tedd_an/ScanBuild | success | Scan Build PASS |
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=758879 ---Test result--- Test Summary: CheckPatch PASS 0.73 seconds GitLint PASS 0.52 seconds BuildEll PASS 26.96 seconds BluezMake PASS 786.35 seconds MakeCheck PASS 11.94 seconds MakeDistcheck PASS 156.16 seconds CheckValgrind PASS 249.75 seconds CheckSmatch WARNING 337.65 seconds bluezmakeextell PASS 102.01 seconds IncrementalBuild PASS 1316.00 seconds ScanBuild PASS 1018.60 seconds Details ############################## Test: CheckSmatch - WARNING Desc: Run smatch tool with source Output: monitor/packet.c: note: in included file:monitor/display.h:82:26: warning: Variable length array is used.monitor/packet.c:1832:26: warning: Variable length array is used.monitor/packet.c: note: in included file:monitor/bt.h:3552:52: warning: array of flexible structuresmonitor/bt.h:3540:40: warning: array of flexible structuresemulator/btdev.c:417:29: warning: Variable length array is used. --- Regards, Linux Bluetooth
diff --git a/monitor/packet.c b/monitor/packet.c index 4473fda29a46..cd567231449e 100644 --- a/monitor/packet.c +++ b/monitor/packet.c @@ -11557,7 +11557,7 @@ static void le_cis_established_evt(struct timeval *tv, uint16_t index, print_field("Peripheral to Central Flush Timeout: %u", evt->p_ft); print_field("Central to Peripheral MTU: %u", le16_to_cpu(evt->c_mtu)); print_field("Peripheral to Central MTU: %u", le16_to_cpu(evt->p_mtu)); - print_field("ISO Interval: %u", le16_to_cpu(evt->interval)); + print_slot_125("ISO Interval", evt->interval); } static void le_req_cis_evt(struct timeval *tv, uint16_t index,
From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> The ISO Interval is actually using set using 1.25ms slots: BLUETOOTH CORE SPECIFICATION Version 5.3 | Vol 4, Part E page 2304: Time = N * 1.25 ms --- monitor/packet.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)