Skip to content

[Backport v4.0.99-ncs1-branch] [nrf fromlist] tests: drivers: Turn on HFCLK before LFCLK calibration in nRF devices #2796

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3,002 commits into
base: v4.0.99-ncs1-branch
Choose a base branch
from

Conversation

NordicBuilder
Copy link
Contributor

Backport 926f142 from #2791.

rlubos and others added 30 commits March 28, 2025 13:48
…QTT 5.0 enabled

Add test variant which enables MQTT 5.0 support. MQTT 3.1.1 should still
work just fine in such case, so it needs to be verified that's the case.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit 08f8e2f)
…ngth

The macro did not take into account the 2-byte binary data length
encoded before the actual binary data.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit 1eb9a5c)
Add support for CONNECT message specified in MQTT 5.0, along with
property encoders required to encode MQTT properties which is a new
concept in MQTT 5.0. Connect and will properties can be specified by the
application in the mqtt_client structure before connecting.

Introduce a helper function which allows to verify whether MQTT 5.0 is
used or not, so that it's still possible to use MQTT 3.1 even if MQTT
5.0 support is enabled in Kconfig.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit 8fe2965)
Add support for CONNACK message specified in MQTT 5.0, along with
property decoders required to decode MQTT properties. Decoded properties
are provided to the application within mqtt_connack_param structure,
accompanying the CONNACK event.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit 6038676)
Add support for PUBLISH message specified in MQTT 5.0. The message
encoder and decoder were updated to support MQTT properties.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit 15ad90a)
Add support for PUBACK, PUBREC, PUBREL and PUBCOMP specified in MQTT
5.0. As all of these acknowledgment packets have similar format,
introduced a common encoder/decoder to handle ACK packets.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit c21e642)
Add support for SUBSCRIBE/UNSUBSCRIBE messages specified in MQTT 5.0.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit efd795b)
Add support for SUBACK/UNSUBACK messaged specified in MQTT 5.0.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit 2459ffa)
Add support for DICONNECT message specified in MQTT 5.0.
As with MQTT 5.0, the disconnect can now also be initiated by the
broker, it was needed to add decoder support for the message.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit 55e1c10)
Add support for a new AUTH message introduced in MQTT 5.0.
This is a new mechanism specified by MQTT 5.0, which allows clients and
brokers for enhanced authentication in between CONNECT and CONNACK
exchange. An additional MQTT event (MQTT_EVT_AUTH) was specified which
is triggered when the AUTH packet arrives from the broker.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit 1d743fa)
Add support for handling topic aliases received from the broker.
The MQTT client implementation will store received topic for further use
in case consecutive PUBLISH received from the broker contains no topic
but alias only.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit e9da3b3)
…QTT 5.0

As MQTT 5.0 allows to specify the disconnect reason in the Disconnect
packet, use this new feature to improve error notification to the
broker, according to the error guidelines in the MQTT 5.0 spec.
For most cases, a generic arbitrary mapping between errno values and
reason codes is used, however the parser can specify the disconnect
reason code manually to better handle certain corner cases (like invalid
topic alias used).

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit 8935579)
…tests

Add new test suite covering MQTT 5.0 packet format.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit 347653e)
…ntation

Make sure conditionally compiled parts of the public header related to
MQTT 5.0 support are rendered by doxygen.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit f9b4e27)
MQTT 5.0 is now supported so reflect this in the library documentation.

Signed-off-by: Robert Lubos <[email protected]>
(cherry picked from commit e380473)
…ompile errors

Upstream PR #: 87741

Add some extra ifdefs around MQTT 5.0 specific structures to prevent
compiler errors due to undefined symbols in certain configurations.

Signed-off-by: Robert Lubos <[email protected]>
This commit provides an extension to the i2c_nrfx_twim driver.
It introduces possibility to acquire/release exclusive access to the
i2c bus. While the exclusive access to the i2c bus is acquired
you can access the underlying nrfx_twim driver directly.

Signed-off-by: Andrzej Kuros <[email protected]>
(cherry picked from commit d15c9fe)
This is needed for privacy tests. Specifically, this fixes
GAP/PRIV/CONN/BV-10-C and GAP/CONN/UCON/BV-06-C on nRF54H20.

Signed-off-by: Pavel Vasilyev <[email protected]>
…tion Ack PDU

The Generic Provisioning protocol requires that a transaction must be
acknowledged before the next Generic Provisioning PDU is processed
(section 5.3.3, MshPRT_v1.1).

In the current PB-ADV implementation, if there is an ongoing transaction
(link.tx.adv points to a valid advertisement), any received Generic
Provisioning Start and Continuation PDUs are ignored.

This creates a scenario where a remote device misses the Transaction
Acknowledgment PDU and retransmits the last PDUs, but they are ignored
by the local device because the local device has already started sending
the next Provisioning PDU.

In particular, in the observed failure, the Provisioner missed the
Generic Transaction Acknowledgment PDU during the transmission of the
Provisioning Data PDU.

This commit modifies the processing logic for Generic Provisioning Start
and Continuation PDUs by checking if the received PDU has already been
received. If so, it attempts to re-send the Generic Transaction
Acknowledgment PDU.

Signed-off-by: Pavel Vasilyev <[email protected]>
(cherry picked from commit 7592b04)
Signed-off-by: Pavel Vasilyev <[email protected]>
…TT server enabled

When the GATT server is enabled the proxy advertiser
server need to be stopped. This solved a bug where the
Mesh Proxy Service is not restared.
This fix is done to fix several failing PTS test that
was failing when CONFIG_BT_MESH_ADV_EXT_GATT_SEPARATE
was enabled. Calling the stop_proxy_adv was left out
in a prevouis commit when doing code cleanup.

Signed-off-by: Ingar Kulbrandstad <[email protected]>
(cherry picked from commit a65a164)
Signed-off-by: Pavel Vasilyev <[email protected]>
… valid yet

Commit sets valid state when mesh is actualy provisioned.

Signed-off-by: Aleksandr Khromykh <[email protected]>
(cherry picked from commit 928208c)
Signed-off-by: Pavel Vasilyev <[email protected]>
Previously, PB-GATT unprovisioned advs would continue being sent if a
device is provisioned or in the process of being provisioned (an active
provisioning link) over PB-ADV.

This commit introduces the `link_cancel` callback for the `prov_bearer`
struct. When a provisioning link is established over a prov bearer, all
other (active) prov bearers will be suspended through this callback, and
re-enabled when the link is closed. An exception is made for PB-REMOTE,
which can utilize the other bearers.

The call to `bt_mesh_pb_gatt_srv_disable` in settings.c is removed. This
is called in `mesh_commit`, and is not needed as this is only used
during boot, at which point the Mesh provisioning service has not yet
been enabled/registered, meaning the function does nothing.

Signed-off-by: Håvard Reierstad <[email protected]>
(cherry picked from commit 200628e)
Signed-off-by: Pavel Vasilyev <[email protected]>
Changes Brearer --> Bearer.

Signed-off-by: Håvard Reierstad <[email protected]>
(cherry picked from commit 062fdc3)
Signed-off-by: Pavel Vasilyev <[email protected]>
Adds a provision test to check that provisioning bearers are cancelled
when a provisioning link is opened. Modifies the functionality for
receiving unprovisioned beacons to fail the test if a unprovisioned
beacon is received from the current provisionee after a threshold.

Signed-off-by: Håvard Reierstad <[email protected]>
(cherry picked from commit acccd46306b2de026add59ea3d75c4a1b81cfc44)
Signed-off-by: Pavel Vasilyev <[email protected]>
When the board has been provisioned and the board is rebooted
it was not possible to connect to the board through the proxy
servcie. This fixes that the Mesh Proxy servive is restarted
after the board is rebooted.

Signed-off-by: Ingar Kulbrandstad <[email protected]>
(cherry picked from commit db84181)
Signed-off-by: Pavel Vasilyev <[email protected]>
Commit removes device key candidate if provisionee failed
during device key refresh procedure or mesh reset has been
called in between. Otherwise, device key candidate gets
stuck in the internal trusted storage of crypto library.

Signed-off-by: Aleksandr Khromykh <[email protected]>
(cherry picked from commit bf22fa6)
Signed-off-by: Pavel Vasilyev <[email protected]>
The TMP435 is a remote temperature sensor monitor
with a built-in local temperature sensor.

Signed-off-by: Jaakko Rautiainen <[email protected]>
(cherry picked from commit 516e5d6)
This Introduces a driver for the i2c interface of Phosense XBR818.
XBR818 is a 10.525Ghz Radar chip with builtin detection algorithm.

Signed-off-by: Camille BAUD <[email protected]>
(cherry picked from commit 6376866)
- add driver for Vishay VEML6031 High Accuracy Ambient Light Sensor.
- add new compatible "vishay,veml6031".
- read and write consecutive 8 bit registers as bulk operation.
- add driver to build all test of sensors.
- support fetch and get.
- triggered mode and interrupt is not yet supported.

Signed-off-by: Andreas Klinger <[email protected]>
(cherry picked from commit 72b2f01)
Add driver for Bosch BMM350 magnetometer sensor.

Signed-off-by: Maximilian Deubel <[email protected]>
(cherry picked from commit 878ab53)
frkv and others added 24 commits April 16, 2025 12:39
-This commit adds "sysbuild: true" for ns build-targets for the
 following nordic boards:
  - nrf5340_audio_dk
  - nrf54l15dk
  - nrf9131ek
  - nrf9151dk
  - nrf9161dks (multiple versions)
  - thingy53
-Added missing vendor: nordic for some boards for consistency

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
(cherry picked from commit fde1159)
(cherry picked from commit bdc3de9)
new arg base-branch defaults to main,
but can map zephyr release branch to nrf release base

Signed-off-by: Thomas Stilwell <[email protected]>
Update WiFi driver to cleanup extra logging newlines for consistency.

Signed-off-by: Jordan Yates <[email protected]>
(cherry picked from commit 729f50e)
(cherry picked from commit 2baf4a8)
Memory allocations in HAL are using data pool. These operations
are in control plane. Add APIs for allocation/free operations on
control pool.

Signed-off-by: Ravi Dondaputi <[email protected]>
(cherry picked from commit ba5b897)
This reverts commit 9679838.

Signed-off-by: Chaitanya Tata <[email protected]>
(cherry picked from commit b475795)
…rts"

This reverts commit 6f71dc6.

Signed-off-by: Chaitanya Tata <[email protected]>
(cherry picked from commit dc70900)
…ing delete"

This reverts commit e3a891b.

Signed-off-by: Chaitanya Tata <[email protected]>
(cherry picked from commit 409d6e7)
Deletion of credential should use the pointer from the reference slot
not the temporary buffer, this causes a crash (unknown error).

Upstream PR #: 87656

Signed-off-by: Chaitanya Tata <[email protected]>
(cherry picked from commit 60d9aa7)
For enterprise mode we need to install multiple certs to the TLS
credentials store, so, add a helper script in python to make it work
cross-platforms.

Upstream PR #: 88682

Signed-off-by: Chaitanya Tata <[email protected]>
(cherry picked from commit 62a0c4b)
Commit 6a2cbc7 ("boards: nrf53*: add L|HFXO configurations") missed
the fix for nRF7002DK which is also based on nRF53 SoC.

Upstream PR #: 88680

Signed-off-by: Chaitanya Tata <[email protected]>
Signed-off-by: Aleksandar Stanoev <[email protected]>
(cherry picked from commit dcdbbd6)
Change wifi_cred to wifi cred.

Upstream PR #: 88674

Signed-off-by: Kapil Bhatt <[email protected]>
(cherry picked from commit 0f0ac07)
… credentials"

This reverts commit 8e325a5.

Signed-off-by: Chaitanya Tata <[email protected]>
Instead of having an overlay move the Enterprise configurations to a
dedicated snippet so that it can be enabled with any sample.

Can be used along with Wi-Fi snippet e.g., `-S
"wifi-ipv4;wifi-enterprise"`.

Upstream PR #: 87656

Signed-off-by: Chaitanya Tata <[email protected]>
…ials

Enable TLS credentials shell to manager Wi-Fi enterprise certs.

Upstream PR #: 87656

Signed-off-by: Chaitanya Tata <[email protected]>
Add locally generated info for deauth process. If deauthentication
frame is coming from AP it will be set, in other cases
(Beacon loss, New connection from user in connected state,
disconnection from user) flag will not be set.

Upstream PR #: 88489

Signed-off-by: Kapil Bhatt <[email protected]>
This helps install certificates to the TLS credentials store using TLS
credentials shell.

Upstream PR #: 88553

Signed-off-by: Chaitanya Tata <[email protected]>
(cherry picked from commit 87459b4)
Adds missing qualifiers for grouping flash runner configuration

Upstream PR #: 88699

Signed-off-by: Jamie McCrae <[email protected]>
(cherry picked from commit 37e7692)
…ossible

Add nrf_cache_power_up and nrf_cache_power_down functions. In case of
s2ram power up cache as early as possible, before restoring ARM core
registers. It improves restore time from 180 us to 33 us.

As a minor optimization nrf_memconf_ramblock_control_mask_enable_set is
used which allows to control ram blocks for icache and dcache in a
single register write.

Signed-off-by: Krzysztof Chruściński <[email protected]>
(cherry picked from commit a36b154)
(cherry picked from commit da14f33)
…timing

Update exit-latency-us with newly measured values. Measurement is
taking into account additional timing compared to wakeup from
WFI.

Signed-off-by: Krzysztof Chruściński <[email protected]>
(cherry picked from commit a4a6130)
(cherry picked from commit d2534ea)
Cache was not enabled when s2ram did not completed which
lead to system malfunction. Always power up cache when
returning from s2ram function.

Upstream PR #: 88709

Signed-off-by: Krzysztof Chruściński <[email protected]>
ISR safe runtime PM can only be used for all instances except for
spis120 which requires standard runtime PM.

Added compilation guard against using CONFIG_PM_DEVICE_SYSTEM_MANAGED
with spis120.

Signed-off-by: Krzysztof Chruściński <[email protected]>
(cherry picked from commit 8d6ab28)
… in nRF devices

This change plays with situation in nRF54L devices where XOTUNE make
LF clock calibration process starts a bit longer. Now HF clock
is configured to be ready right before LF calibration test.

Upstream PR #: 88797

Signed-off-by: Karol Lasończyk <[email protected]>
(cherry picked from commit 926f142)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.