STM32WB55 ble not working #76558
Closed
STM32WB55 ble not working #76558
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
"prototype" is ambiguous in this regard, as it doesn't really tell what the test suite is testing. As tests in this test suite verify core HTTP server functionality, rename it to "core". Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Do some cleanup regarding naming used in the test suite to prepare it for adding more test cases. Extract common setup/teardown into respective before and after ztest functions to simplify test code. Proper before and after functions also ensure that individual test case failure does not disrupt other tests operation. Split the tests into two separate test suites, one for those which require pre-setup and second for those which don't. Finally, enable receive timeout on the client socket to ensure tests won't stall in case of errors. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Ensure it's possible to restart the server w/o issues after a client connected. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Remove pointless helper functions, the test code can simply be included in the test case w/o code duplication. Clarify the naming of existing test cases. Backward compatibility simply tests HTTP1. The upgrade test wasn't really testing HTTP upgrade, so rename the test case too. Add a new test case actually verifying HTTP upgrade. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Add test case verifying that the server keeps the connection open if no Connection: close header is present in the request. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Add tests cases covering interaction with dynamic resources with POST/GET requests. Simplify HTTP2 request generation to facilitate adding more tests. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Add tests verifying that requests which include frames with padding are processed correctly. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Add test cases verifying that request containing header frame with priority flag set is processed properly. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Add test cases for continuation frame processing. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Add test case covering a scenario when a client sends a trailing header frame in a request. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Add test case verifying that stream is closed when RST_STREAM frame is received. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
When implementing vendor specific HCI APIs and events, we want to be able to convert between host objects, handles and back again. Exposing this API makes that possible. Signed-off-by: Rubin Gerritsen <rubin.gerritsen@nordicsemi.no>
When implementing vendor specific HCI APIs and events, we want to be able to convert between host objects, handles and back again. Exposing this API makes that possible. Signed-off-by: Rubin Gerritsen <rubin.gerritsen@nordicsemi.no>
When implementing vendor specific HCI APIs and events, we want to be able to convert between host objects, handles and back again. Exposing this API makes that possible Signed-off-by: Rubin Gerritsen <rubin.gerritsen@nordicsemi.no>
Looks like this option is not tested with userspace enabled. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
k_usermode_string_copy does not accept const in its arguements. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Fix the NM iface type check error, should use (1 << WIFI_TYPE_STA), instead of WIFI_TYPE_STA. Same for WIFI_TYPE_SAP. Signed-off-by: Maochen Wang <maochen.wang@nxp.com>
Fix Advertising PDU memory allocation for redundant Periodic Advertising related PDU allocations. The buffer count related to Periodic Advertising was included twice. Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
There were code paths that could have lead to divide by zero given an invalid scale setting for accel or gyro. In practice this should be an invalid setup even before getting to these conversion functions. The conversion functions now better show all valid values are accounted for by using CODE_UNREACHABLE in the default case. Signed-off-by: Tom Burdick <thomas.burdick@intel.com>
Apparently, disabling the intterupt is not enough, because without clearing the flag, some errors are occurring. Signed-off-by: Declan Snyder <declan.snyder@nxp.com>
When RISCV_ALWAYS_SWITCH_THROUGH_ECALL is enabled, do_swap() enables PMP checking in is_kernel_syscall. If a user thread violates memory protection and do_swap() is called from the fault handler, a PMP error occurs because the thread is in privileged mode but still using the old user mode PMP setting. Update the PMP setting to privileged mode for fault handler. This also enables the stack guard for user thread's privileged stack in fault handler. Signed-off-by: Jimmy Zheng <jimmyzhe@andestech.com>
When RISCV_ALWAYS_SWITCH_THROUGH_ECALL is enabled, do_swap() enables PMP checking in is_kernel_syscall. If the PMP stack guard is triggered and do_swap() is called from the fault handler, a PMP error occurs because the stack usage violates the previous PMP setting. Remove the stack guard setting during a stack overflow handler to allow enabling PMP checking safely in fault handler. Signed-off-by: Jimmy Zheng <jimmyzhe@andestech.com>
If the application is not explicitly scanning, then there is not really any need to parse advertising reports nor send them to the application. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
the dts binding of the ethernet-phy has the option "no-reset" and the according driver already has the right logic to make use of it, but unfortunately the connection between them was missing. Signed-off-by: Fin Maaß <f.maass@vogl-electronic.com>
Updates dts files to use the same GPIOTE interrupt lines as NRFX for zephyr when TF-M is used. Signed-off-by: Vidar Lillebø <vidar.lillebo@nordicsemi.no>
In case POLLIN is set, and no new application data has been detected, the ztls_poll_update_ctx() should only return -EAGAIN if no other events are available for the socket. Otherwise, the function may end up busy-looping, in case for example POLLOUT is also monitored for the socket. Current check verifying that was wrong, as it caused to function to return -EAGAIN even if some other events could've been reported for the socket. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Include what is strictly needed. Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
This avoids the need to define a position shift macro, and makes code less verbose. Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
Instead use Doxygen anchors to reference to a single source of truth. Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
- Use references whenever possible instead of literals (e.g. `@c`)
- Remove incorrect usages of `@see`
- Fix incorrect usage of INTERNAL_HIDDEN blocks
- Detail Kconfig-dependent options with `@kconfig{}`
- Other minor enhancements
Ref. https://www.doxygen.nl/manual/autolink.html
Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
asm5878
requested review from
ChiHuaL,
MulinChao,
SgrrZhf,
Vudentz,
XenuIsWatching,
abonislawski,
aescolar,
alwa-nordic,
daor-oti,
de-nordic,
finikorg,
jhedberg,
kv2019i,
marcinszkudlinski,
nandojve and
nordic-krch
as code owners
|
The following west manifest projects have been modified in this Pull Request: Note: This message is automatically posted and updated by the Manifest GitHub Action. |
Contributor
Author
|
Sorry for the rebasing issue, new (cleaner) PR will come |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Implementing (and enabling) an HCI setup function to have a correct and proper
initialization procedure to fix #75318 issue.