- 30 Nov, 2021 20 commits
-
-
Daniel N. Hansten authored
PCA9633 driver does not cunnetly support multiple devices. Updated the driver to use DT_INST_FOREACH_STATUS_OKAY to configure all devices defined in the device tree. Convert driver to use `i2c_dt_spec` helpers. Fixes #40076 Signed-off-by:
Daniel N. Hansten <dnh2000@gmail.com>
-
Ilya Makarov authored
bt_ccm_encrypt only works when encrypting in place. To fix this ccm_auth() inside bt_ccm_encrypt() must take plaintext instead of enc_data, to not rely on assumption that plain and cypher data are the same memory. Signed-off-by:
Ilya Makarov <ilya.makarov.592@gmail.com> Fixes: #40069
-
Vinayak Kariappa Chettimada authored
Fix DTM HCI command returned error codes. Signed-off-by:
Vinayak Kariappa Chettimada <vich@nordicsemi.no>
-
Vinayak Kariappa Chettimada authored
Fix missing implementation to reset DTM Tx/Rx reset on HCI Reset Command. Signed-off-by:
Vinayak Kariappa Chettimada <vich@nordicsemi.no>
-
Piotr Pryga authored
The RSSI measurement was not enabled while receiving periodic advertising. The function responsible for enable the feature in radio was called, but it was done too early. It was overwritten by radio_switch_XXX function that assigns a value to RADIO->SHORTS register. Signed-off-by:
Piotr Pryga <piotr.pryga@nordicsemi.no>
-
Piotr Pryga authored
Handling of HCI_LE_Periodic_Advertising_Sync_Established didn't have implemented handling of possible failures of periodic advertising synchronization. There are two situations definded by BT 5.3 Core spec: - There is no AUX_SYNC_IND pdu within 6 periodic advertising events. If that happens, status of the command is set to (0x3E) Connection Failed To Be Established / Synchronization Timeout. - Periodic advertising has wrong CTE type while periodic advertising list is not used to determine the advertiser to listen. In this case status of the command is set to (0x1A) Unsupported Remote Feature. The commit provides missing functionality. In case of error, the periodic advertising will be deleted and application will be notified by call to terminated callback. The callback data were extended by err member. It provides information why periodic advertising was terminated. Signed-off-by:
Piotr Pryga <piotr.pryga@nordicsemi.no>
-
Johann Fischer authored
Just always prove interface association descriptor for RNDIS function instead of forcing it via Kconfig USB_COMPOSITE_DEVICE option. Signed-off-by:
Johann Fischer <johann.fischer@nordicsemi.no>
-
Johann Fischer authored
Set reasonable range for the request buffer in case RNDIS function is used. Align net_buf size from rndis_cmd_pool to request buffer size since request is copied there before it is queued. Signed-off-by:
Johann Fischer <johann.fischer@nordicsemi.no>
-
Szymon Janc authored
This allows to do directed advertising with peer address set to RPA. To do this according to specification IUT must first read Central Address Resolution characteristic to check if peer supports it. This is affecting GAP/CONN/DCON/BV-05-C qualification test. Signed-off-by:
Szymon Janc <szymon.janc@codecoup.pl>
-
Szymon Janc authored
This makes implementation in sync with autopts. Signed-off-by:
Szymon Janc <szymon.janc@codecoup.pl>
-
Ilhan Ates authored
GAP/CONN/DCON/BV-01-C test case needs directed advertising support. Signed-off-by:
Ilhan Ates <ilhan.ates@nordicsemi.no>
-
Szymon Janc authored
This allows to track security levels and check for lost bond of peer peripherals. This was affecting GAP/SEC/AUT/BV-21-C qualification test. Signed-off-by:
Szymon Janc <szymon.janc@codecoup.pl>
-
Szymon Janc authored
GATT/SR/GAW/BV-10-C requires more buffers as it tests nested long writes. Signed-off-by:
Szymon Janc <szymon.janc@codecoup.pl>
-
Szymon Janc authored
This event is sent if pairing failed. Signed-off-by:
Szymon Janc <szymon.janc@codecoup.pl>
-
Szymon Janc authored
This error code informs that peer device rejected key during keys distribution phase. Signed-off-by:
Szymon Janc <szymon.janc@codecoup.pl>
-
Szymon Janc authored
This allows for better control over IUT behaviour by Upper Tester. PTS and TS require inconsistent behaviour in terms of how IUT should return credits. Some tests require return on explicit UT request and some require that IUT returns credits autonomously. Signed-off-by:
Szymon Janc <szymon.janc@codecoup.pl>
-
Daniel Leung authored
XCC doesn't like having quotes in the section name, so workaround it by overriding __in_section_unique_named() similar to __in_section_unique(). Signed-off-by:
Daniel Leung <daniel.leung@intel.com>
-
Marek Pieta authored
Zephyr device that is not a GATT Client, should ignore indication. An Android device may send an indication even if Zephyr device does not support GATT Client role. In that case, the sent error response was improperly matched to subsequent GATT request of the Android device which caused issues. Signed-off-by:
Marek Pieta <Marek.Pieta@nordicsemi.no>
-
Martin Jäger authored
The task_wdt was getting stuck after approx. 36 hours on e.g. nRF52840, which has a SysTick with 32768 Hz. This corresponds to an overflow of the uint32_t current_ticks in schedule_next_timeout. This commit fixes the accidentally introduced narrowing conversion. Fixes #40152 Signed-off-by:
Martin Jäger <martin@libre.solar>
-
Sylvio Alves authored
PR #39980 added qsort to minimal libc but caused shell_modules sample to fail building. Signed-off-by:
Sylvio Alves <sylvio.alves@espressif.com>
-
- 26 Nov, 2021 8 commits
-
-
Anas Nashif authored
We now cleanup at the beginning. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
Anas Nashif authored
Check if test plan exist before trying to read it. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
Anas Nashif authored
Merged 3 files used to generate the test plan per PR based on the changes. 2 python scripts and a shell script are now all merged into 1 python script that generates the input file for twister based on a list of changed files by the PR. This remove lots of old and obsolete code and simplifies things a bit, no need anymore for an intermediate script to call twister, we call it directly in the workflow and use the new test_plan script to generate the test plan. This also reenables the recently disabled tag based filtering which had a bug, bug is resolved in this new implementation. On push events, we now run twister without the --integration option to catch any issues in the main branch that were not caught in PRs. PRs continue to run with --integration enabled. This event (push) is now run on 15 builders due to the increased size. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
Anas Nashif authored
Update CI docs and the badge with links pointing to GH actions rather than buildkite. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
Anas Nashif authored
Adapt events and cron to the 2.7 branch. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
Anas Nashif authored
Cancel using GH runner for faster execution. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
Anas Nashif authored
Add branch name to the ccache key to avoid cache contamination from old branches. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
Anas Nashif authored
This action replaces current buildkite workflow and uses github actions to build and run tests in the zephyr tree using twister. The main differences to current builtkite workflow: - the action handles all 3 events: pull requests, push and schedule - the action determines size of matrix (number of build hosts) based on the change with a minimum of 1 builder. If more tests are built/run due to changes to boards or tests/samples, the matrix size is increased. This will avoid timeouts when running over capacity due to board/test changes. - We use ccache and store cache files on amazon S3 for more flexibility - Results are collected per build host and merged in the final step and failures are posted into github action check runs. - It runs on more powerful instances that can handle more load. Currently we have 10 build hosts per run (that can increase depending on number of tests run) and can deliver results within 1 hour. - the action can deal with non code changes and will not allocate more than required to deal with changes to documentation and other files that do not require running twister The goal long-term is better integrate this workflow with other actions and not run unncessarily if other workflows have failed, for example, if commit message is bogus, we should stop at that check, to avoid wasting resources given that the commit message will have to be fixed anyways which would later trigger another run on the same code. Currently there is 1 open issue with this action related to a github workflow bug where the final results are not posted to the same workflow and might appear under other workflows. Github is working on this bug. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
- 24 Nov, 2021 4 commits
-
-
Alexandre Bourdiol authored
In case of MSIS selected as system clock source it is necessary to set Voltage scaling (VOS) when freqency is greater than 24MHz Signed-off-by:
Alexandre Bourdiol <alexandre.bourdiol@st.com>
-
Alexandre Bourdiol authored
When MSI trimming values where set to 0, and MSIS is used as system core clock and MSI > 4 MHz, it causes uart to fail. There is no need to set thoses trimming values. So keep the default reset value. (keep ST Factory calibration) Signed-off-by:
Alexandre Bourdiol <alexandre.bourdiol@st.com>
-
Alexandre Bourdiol authored
Because on stm32u5 MSIS is the default clock after reset, changing MSIS range means immediate frequency change. Thus it is important to do it after flash latency change in case of higher new frequency. Signed-off-by:
Alexandre Bourdiol <alexandre.bourdiol@st.com>
-
Alexandre Bourdiol authored
Missing definition of STM32_SYSCLK_SRC_MSIS especially needed for STM32U5 Signed-off-by:
Alexandre Bourdiol <alexandre.bourdiol@st.com>
-
- 21 Nov, 2021 1 commit
-
-
Johan Lundin authored
Sets SID in bt_le_per_adv_sync_recv_info when host receives a Periodic Advertising Report Signed-off-by:
Johan Lundin <johan.lundin@nordicsemi.no>
-
- 19 Nov, 2021 7 commits
-
-
Szymon Janc authored
This fix not being able to re-connect channel after disconnect. Signed-off-by:
Szymon Janc <szymon.janc@codecoup.pl>
-
Szymon Janc authored
If an LTK or an STK is available and encryption is required (LE security mode 1) but encryption is not enabled, the service request shall be rejected with the error code "Insufficient Encryption". This is affecting L2CAP/LE/CFC/BV-25-C and L2CAP/ECFC/BV-32-C qualification test cases. Signed-off-by:
Szymon Janc <szymon.janc@codecoup.pl>
-
Anas Nashif authored
Many tests and CI activties are being missed by excluding tests mistakingly when running twister. This is visibile when you change one or more tests in kernel/ for example, twister does not run those tests that have changed at all and marking the PR as tested and ready to be merged. Temporary fix for #40235. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
Anas Nashif authored
Otherwise reporting is skipped and failures are not recorded. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
Anas Nashif authored
Use ccache to speed up builds. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
Anas Nashif authored
Retry west when update fails and use update.narrow configuration. Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-
Anas Nashif authored
Add missing ")". Signed-off-by:
Anas Nashif <anas.nashif@intel.com>
-