-
Notifications
You must be signed in to change notification settings - Fork 720
[IGNORE THIS] cmake: introduce instrumentation flag #2600
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
Closed
bjarki-andreasen
wants to merge
2,568
commits into
nrfconnect:main
from
bjarki-andreasen:add_instrumentation
Closed
[IGNORE THIS] cmake: introduce instrumentation flag #2600
bjarki-andreasen
wants to merge
2,568
commits into
nrfconnect:main
from
bjarki-andreasen:add_instrumentation
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
…ration for nrf54h20 Extend CONFIG_SKIP_EDGE_NUM. Signed-off-by: Piotr Kosycarz <[email protected]> (cherry picked from commit e3e4548)
…BT_HCI_RAW In zephyrproject-rtos/zephyr#84268 the ability to use the controller for ECDH was removed from the host. This means that BT_CTLR_ECDH is now only useful when using BT_HCI_RAW. Signed-off-by: Sean Madigan <[email protected]> (cherry picked from commit 706938d)
Recently I have had to debug issues with PSA and having the returns values from PSA is very useful in order to find the root cause of the issue. Signed-off-by: Sean Madigan <[email protected]> (cherry picked from commit 8cff70a)
…baudrate enum Add faster baudrates (2M, 4M and 8M). Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit e737895)
Rename UARTE_ANY_FAST to UARTE_ANY_FAST_PD. There are 2 types of 'fast' UARTE instances. In nrf54h20 instance uart120 is in fast power domain that requires additional power and clock management of that domain. In nrf54lx fast uart00 instance does not require that. Add _PD to indicate fast power domain. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 1573be5)
…udrates nrf54x devices have UARTE instance capable of using baudrate higher than 1M. Higher baudrates does not have predefined values for BAUDRATE register. A formula can be used to calculate BAUDRATE value that shall be used for desired baudrate. Add UARTE_ANY_HIGH_SPEED macro which is set when high speed is enabled (uart00 in nrf54lx or uart120 in nrf54h20). For high speed instance use formula for getting value that shall be written to BAUDRATE register. When runtime configuration is not used then same formula is used to calculate fixed BAUDRATE value. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 972ec82)
Fast instance in nrf54h (uart120) can generate a spurious RXTO event some time after RXTO event that indicates that RX path is disabled. The time when event is generated depends on baudrate and when slower baudrates are used peripheral is disabled on time to not notice it in the test but with higher baudates issue become visible. In order to avoid spurious interrupt, RXTO interrupt is disabled during RXTO event handling and enabled when RX is enabled. This workaround is applied only for fast instance to avoid unnecessary register accesses for slower instances. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit e235e7b)
…abort test Test was staring a TX transfer and aborting it after 300 us from the timer handler. Test was assuming that ongoing transfer will not finish in those 300 us. This might not be true for higher baudrates. Instead of using fixed timeout, a value is calculated from the baudrate and targets to abort the transfer after approx. 20 bytes of 95 byte long transfer. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 38c129d)
…rf54h20dk uart120 Use high baudrate when testing uart120 on nrf54h20dk/nrf54h20/cpuapp. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit fe6a4d0)
Changes the priority of imgtool so that the preferred version is the one inside of the MCUboot directory Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit 6db2c86)
Uses imgtool which has already been located by zephyr Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit 8b7beff)
…lection Adds API for Advertising Coding Selection. Introduces two new advertising options to configure the advertiser's requirement concerning coding scheme when LE Coded PHY is configured. While the Bluetooth v6.0 specification makes a distinction betweeen preferred and required advertising PHY options, a simplification is made to only expose the required PHY options. Inline with how LE Coded PHY is implemented; this API will set both the primary and secondary advertising PHY's to the same coding scheme. The support is enabled by CONFIG_BT_EXT_ADV_CODING_SELECTION, and requires a controller that selects CONFIG_BT_CTLR_ADV_EXT_CODING_SELECTION_SUPPORT. Signed-off-by: Thomas Deppe <[email protected]> (cherry picked from commit f3bdd2b)
…ng Selection Extends the API for Advertising Coding Selection. The API is extended to set the Advertising Coding Selection (Host Support) bit. With this feature, the primary and secondary PHY can now explicitly report S=2 or S=8 coding in the extended advertising report. Previously, the report only indicated LE Coded regardless of whether S=2 or S=8 data coding was used. The API now sets the host support bit and ensures that the advertising PHY coding scheme is conveyed to the application via the scan callback. The support is enabled by CONFIG_BT_EXT_ADV_CODING_SELECTION, and requires a controller that selects CONFIG_BT_CTLR_ADV_EXT_CODING_SELECTION_SUPPORT. Signed-off-by: Thomas Deppe <[email protected]> (cherry picked from commit dcbcbe8)
This adds temporary entropy driver simulation for nRF54l09 device since final entropy source is not available yet. TODO: Remove this commit when proper solution will be available. Signed-off-by: Rafał Kuźnia <[email protected]>
…opback too A test is going to use this UART also, so we'll need it. Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit dc87bb0)
Add the fast uart instance of 54l15 for testing Signed-off-by: Piotr Krzyzanowski <[email protected]> (cherry picked from commit 6d8eb18)
…ck issue In 923d313 the clock frequency in DTS for the UART00 was fixed, but not for the simulated target. This was likely due to the HW models modeling it as 16MHz instead of 128MHz for this particular one as it is in reality. Now that the HW models have been fixed, let's let this clock be configured like for real HW. Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit 5e7df92)
Disable insted of deletion of spi00 node on 54l15bsim Signed-off-by: Piotr Krzyzanowski <[email protected]> (cherry picked from commit ae80fb1)
* The `bt_dev.le.conn_ready` list is accessed by the tx_processor which runs in a workqueue, but `bt_conn_data_ready` can be called from different threads so we need to make sure that nothing will trigger a context switch while we are manipulating the list since sys_slist_*() functions are not thread safe. * This only happens if call to `bt_conn_data_ready` is performed from a preemptive task which can happen depending on the application. Signed-off-by: Yago Fontoura do Rosario <[email protected]> (cherry picked from commit 9f77362) Signed-off-by: alperen sener <[email protected]>
* Two mocks were missing in kernel mocks Signed-off-by: Yago Fontoura do Rosario <[email protected]> (cherry picked from commit ee32613) Signed-off-by: alperen sener <[email protected]>
uuid needs to be keept in pb_gatt_cli server context until the client is connected to server, otherwise, since we only kept the pointer from the net_buffer, any incoming unprovisioned beacon before the connection is established may overwrite uuid. Signed-off-by: alperen sener <[email protected]> (cherry picked from commit 37cdfe8) Signed-off-by: alperen sener <[email protected]>
…cases Extending existing provisioning test code to use PB-GATT and adding test cases for provisioning over PB-GATT. Test names are changed to represent both provisioning brearers. Enabled required kconfigs for PB-GATT and GATT proxy features in overlay_gatt.conf. Test argument prov-bearer is added to provisioning tests to select bearer. Following tests are renamed and extended to use PB-GATT - pb_adv_multi - pb_adv_no_oob - pb_adv_oob_auth_ib_pk - pb_adv_oob_auth_ignore_oob_pk - pb_adv_oob_auth_oob_pk - pb_adv_reprovision PB-GATT test cases will run only with PSA encryption since tinycrypt is to be deprecated. Signed-off-by: alperen sener <[email protected]> (cherry picked from commit 917683a) Signed-off-by: alperen sener <[email protected]>
By using `SETTINGS_STATIC_HANDLER_DEFINE_WITH_CPRIO` we can ensure that BT subsystems always get loaded after BT, and BT Mesh after both of them. This solves the host having to register a GATT service in a delayed manner, as we are sure to now register the GATT service after sc_commit sets `SC_LOAD`. Signed-off-by: Kyra Lengfeld <[email protected]> (cherry picked from commit 64d8713) Signed-off-by: alperen sener <[email protected]>
Commit: - adds dependency of the mbedtls psa usage on secure storage - removes PSA ITS emulator and enables usage of the secure storage in ble mesh bsim tests - enables secure storage in all ble mesh and related samples Signed-off-by: Aleksandr Khromykh <[email protected]> (cherry picked from commit 967b096) Signed-off-by: alperen sener <[email protected]>
…lots for mesh MESH PTS tests require different number of keys, as an example; only one netkey requires 8 key slots and there are test cases that use at least two netkeys which consume the default 16 slots, so it is better to have enough slots all times. Setting slot count to 32. Signed-off-by: alperen sener <[email protected]> (cherry picked from commit 35aea49) Signed-off-by: alperen sener <[email protected]>
…ssage PDU size According to MshDFUv1.0, section 6.2.3.5, the Firmware Distribution Receivers List message is not limited by the size of the receivers list: ``` The number of selected entries shall be limited by the following: - The number of entries shall not exceed the value of the Entries Limit field from the Firmware Distribution Receivers Get message. - The number of entries shall not cause the message payload to exceed the maximum Access PDU size. ``` Thus, this assertion is incorrect as it doesn't allow to have the receivers list bigger than number of maximal Access PDU size. Signed-off-by: Pavel Vasilyev <[email protected]> (cherry picked from commit 6463d15) Signed-off-by: alperen sener <[email protected]>
Removed similar prj.conf files and added overlays instead to avoid duplicate code. Renamed some test files for consistency. Signed-off-by: Stine Akredalen <[email protected]> (cherry picked from commit 724d32c) Signed-off-by: alperen sener <[email protected]>
Iterate over the list of connections to find a connection that is able to send data, instead of returning NULL if the first connection can't send data. This solves the problem with starvation of other connections. Signed-off-by: Ahmed Ahmed <[email protected]> (cherry picked from commit 0998716) Signed-off-by: alperen sener <[email protected]>
Assert that the previous connection in the list and the tmp element in the loop are different to the current connection. Otherwise, if the same connection was somehow added twice it could result in an infinite loop. Signed-off-by: Ahmed Ahmed <[email protected]> (cherry picked from commit 2edddbd) Signed-off-by: alperen sener <[email protected]>
…ning As the connection was removed from the list, we now own its reference. Keep this reference and return it to the TX processor. Signed-off-by: Ahmed Ahmed <[email protected]> (cherry picked from commit 6a45e17) Signed-off-by: alperen sener <[email protected]>
String location information should only be sent for core which do not append strings to the log message (PPR, FLPR). Without this, cpurad was also sending that information and that was redundant. Upstream PR #: 86242 Signed-off-by: Krzysztof Chruściński <[email protected]>
Update the HW models module to:
b84bd7314a239f818e78f6927f5673247816df53
Including the following:
b84bd73 UART PTY backend: Correct a few comments
f945d62 RADIO 54: Fix reset value of TIMING register
9fd4133 MDK nrfx_erratas.h: Ensure backwards compatibility with old
nrfx versions
85061b1 MDK: Provide empty replacement for nrf_erratas.h
Signed-off-by: Alberto Escolar Piedras <[email protected]>
(cherry picked from commit b932e14)
…ipherals Add dmic to the list of supported peripherals on nRF54L20pdk. Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit 0015571)
…20pdk Add test configuration on DMIC nodes that are compatible with 'nordic_nrf_pdm'. Add missing DTS definitions in a board overlay. Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit 22d9f0c)
…20pdk Enable execution of DMIC sample on nRF54L20pdk. Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit 773cf96)
…and MODE_QUAD_1_4_4" This reverts commit ee6cc76. Signed-off-by: Marcin Szymczyk <[email protected]>
…tion for nRF54H20 DK" This reverts commit 664042f. Signed-off-by: Marcin Szymczyk <[email protected]>
…to memory map" This reverts commit bd74349. Signed-off-by: Marcin Szymczyk <[email protected]>
… "jedec,mspi-nor" devices" This reverts commit 87a4e34. Signed-off-by: Marcin Szymczyk <[email protected]>
…MIF node" This reverts commit 3b45cae. Signed-off-by: Marcin Szymczyk <[email protected]>
…l handling" This reverts commit 48b0ca5. Signed-off-by: Marcin Szymczyk <[email protected]>
This reverts commit bb3a457. Signed-off-by: Marcin Szymczyk <[email protected]>
…or MSPI devices" This reverts commit d869c39. Signed-off-by: Marcin Szymczyk <[email protected]>
…ased controllers" This reverts commit 9140a90. Signed-off-by: Marcin Szymczyk <[email protected]>
This reverts commit ac886cc. Signed-off-by: Marcin Szymczyk <[email protected]>
…trollers Add a generic driver for MSPI controllers based on the DesignWare SSI core. With small vendor-specific adaptations covering integration details, it should be possible to use the driver for various devices. Signed-off-by: Andrzej Głąbek <[email protected]> (cherry picked from commit b31eeb6)
…devices Add a flash driver intended to handle various flash devices connected over MSPI bus as long as they support JEDEC SFDP. This is an initial commit providing only basic operations in Octal I/O mode with some hard-coded values for Macronix MX25Ux series chips. Signed-off-by: Andrzej Głąbek <[email protected]> (cherry picked from commit 29f3061)
This is a follow-up to commit 45d827a. Although routing for those pins is configured via UICR, pinctrl still needs to be involved so that it is possible to set desired drive mode for them etc. Add also the missing RWDS pin. Signed-off-by: Andrzej Głąbek <[email protected]> (cherry picked from commit 2e8c79f)
CAN is not supported yet, so remove it from the board for now. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 2512d3d)
This adds temporary entropy driver simulation for nRF54h20 device since final entropy source is not available yet. TODO: Remove this commit when proper solution will be available. Jira: NCSDK-25947 Signed-off-by: Kamil Gawor <[email protected]> Signed-off-by: Robert Lubos <[email protected]> Signed-off-by: Andreas Moltumyr <[email protected]> Signed-off-by: Karol Lasończyk <[email protected]> (cherry picked from commit 0d8345e) (cherry picked from commit ac886cc)
This is a follow-up to commit cdf45cb234077522b5cef2da084869af43d42dc1. Adjust the DTS node for the nRF EXMIF peripheral so that it is possible to handle the peripheral with the generic MSPI driver for DW SSI based controllers and use all its data lines in communication. Also adjust the related board files accordingly. Signed-off-by: Andrzej Głąbek <[email protected]> (cherry picked from commit 271a41a)
…mspi-nor" devices Extend several flash samples and tests so that they can also be used with "jedec,mspi-nor" devices. Add configurations needed for the nrf54h20dk/nrf54h20/cpuapp target. Signed-off-by: Andrzej Głąbek <[email protected]> (cherry picked from commit 2d003e3)
…y map Access to this region must be requested through UICR by a local domain that want to use the Execute In Place (XIP) feature of the EXMIF peripheral. Signed-off-by: Andrzej Głąbek <[email protected]> (cherry picked from commit a2ef8a6)
… nRF54H20 DK Add nRF54H20 DK specific entries to allow using the sample on this board. Signed-off-by: Andrzej Głąbek <[email protected]> (cherry picked from commit ac3355a)
…_QUAD_1_4_4 Extend driver to support single lane and 1-4-4 IO modes. Upstream PR #: 85520 Signed-off-by: Marcin Szymczyk <[email protected]>
This commit brings back modifications from these reverted commits: - f68b2ed - e606246 slightly adjusted so that the EXMIF peripheral is still by default handled by the mspi_dw driver, and in cases where this driver cannot be used because something still does not work correctly, one can switch to the old solution based on the tweaked spi_dw driver. Signed-off-by: Andrzej Głąbek <[email protected]> (cherry picked from commit 352fb6b)
Introduce support for enabling the instrumentation flag. Signed-off-by: Bjarki Arge Andreasen <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
None yet
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.
Introduce support for enabling the instrumentation flag.