-
Notifications
You must be signed in to change notification settings - Fork 687
[nrf noup] ci: autoscan: adding sonar properties #3111
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
leur-nordic
wants to merge
1,149
commits into
nrfconnect:main
Choose a base branch
from
leur-nordic:sonarproperties
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+28,150
−6,610
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
Initial implementation of nRF54L quirks necessary for nRF54LM20A. Upstream PR #: 92842 Signed-off-by: Tomasz Moń <[email protected]> Signed-off-by: Johann Fischer <[email protected]> (cherry picked from commit f48fd8fad88f5cdb8a6600cfc3c8b66980fe0378)
Avoid using the PPI channels that are currently reserved for Radiocore through the default configuration found in `uicr_periphconf_table.c`. Otherwise, the samples may build but will not work as expected. Upstream PR #: 91826 Signed-off-by: Grzegorz Swiderski <[email protected]> (cherry picked from commit bc26475c262d6dcb1ec1b4d8bf53bf4f262382dc) Signed-off-by: Tomasz Moń <[email protected]>
It was observed that device ceases to work in Buffer DMA mode after GET DEVICE QUALIFIER request is STALLed (when USB stack is limited to Full-Speed only operation). The issue is due to missing dout feed. Clear pending dout feed flag after bus reset (enumeration done) and after stalled control read transfer to allow dout to be feed when necessary. Upstream PR #: 93103 Signed-off-by: Tomasz Moń <[email protected]> (cherry picked from commit 83c6c402e206b7c68538594599ea495eb84ae967)
…frequencies Disabled frequency set approximation algorithm for spim's with prescaler. Upstream PR #: 92436 Signed-off-by: Michal Frankiewicz <[email protected]>
…requencies. Added test cases for nrf chips for non standard frequencies. Upstream PR #: 92436 Signed-off-by: Michal Frankiewicz <[email protected]>
Allows usage of settings when multithreading is disabled Upstream PR #: 92886 Signed-off-by: Jamie McCrae <[email protected]>
…_alloc nrf-squash! [nrf noup] Bluetooth: Mesh: zero randomization for friend's adv bt_hci_cmd_create will be deprecated. Signed-off-by: Olivier Lesage <[email protected]>
…/IW416 SoC - Add support for default Annex-55 Bluetooth calibration data load for both IW612 and IW416 SoC. - Add support for default Annex-100 Bluetooth calibration data load for both IW612 and IW416 SoC. Signed-off-by: Nirav Agrawal <[email protected]> (cherry picked from commit ffe8daa)
Use bt_hci_cmd_alloc() instead of the soon to be deprecated bt_hci_cmd_create(). Signed-off-by: Johan Hedberg <[email protected]> (cherry picked from commit c186a31)
Mark bt_hci_cmd_create() as deprecated (superceded by bt_hci_cmd_alloc()), and update any references to the new function. Signed-off-by: Johan Hedberg <[email protected]> (cherry picked from commit a22a4b1)
…eate() Add a note about bt_hci_cmd_create() deprecation and which API should be used instead. Signed-off-by: Johan Hedberg <[email protected]> (cherry picked from commit 607ff85)
Add the resolve and get_startup_time call Upstream PR #: 93141 Signed-off-by: Piotr Krzyzanowski <[email protected]> (cherry picked from commit 632aad17eaab2a0fa50258b47d8732d37d274c08)
Commit 3c47f91 introduced alignment validation in K_MEM_SLAB_DEFINE macros. A couple PTP message slabs wer using alignment 8, while the size of the elements wasn't a multiple of 8, causing a static assertion failure. Fix by changing the alignment from 8 to 4 bytes. Signed-off-by: Benjamin Cabé <[email protected]> (cherry picked from commit 222a601) Signed-off-by: Robert Lubos <[email protected]>
… initialization" This reverts commit de4fad1. Signed-off-by: Krzysztof Chruściński <[email protected]>
…ization The GRTC counter is not cleared at startup, therefore the `last_count` variable needs to be initialized accordingly. This change: - Prevents overflow of the `sys_clock_announce()` int32_t parameter - Ensures the correct uptime value, which should be reset during initialization Signed-off-by: Adam Kondraciuk <[email protected]> (cherry picked from commit e77f942)
…ister access Speed up execution of the interrupt handler and sys_clock_set_timeout(). Sys_clock_set_timeout() can be called in two scenarios: from previous timeout expiration handler or freely. If the former case fast path can be used since CC value in the GRTC register just expired and it can be used as a reference for CCADD setting. This is only a single register write so it's much faster. In the latter a longer procedure is applied which also happens in two variants. If value which is set in CC is further in the future (e.g. K_FOREVER was set before) then CC can be safely overwritten with a new value without a risk of triggering unexpected COMPARE event. If value in CC is earlier than the new CC value (if earlier timeout was aborted) then there is a risk of COMPARE event happening while it is being overwritten. That case requires long and safer procedure of setting CC. Update hal_nordic with changes in the nrfx_grtc driver which are needed for nrf_grtc_timer changes. Upstream PR #: 87944 Signed-off-by: Krzysztof Chruściński <[email protected]>
Add stress test that randomly starts and aborts multiple timers from various contexts. Test checks if timers do not expire prematurely. Upstream PR #: 87944 Signed-off-by: Krzysztof Chruściński <[email protected]>
Adds a retention backend for settings Upstream PR #: 93151 Signed-off-by: Jamie McCrae <[email protected]>
A clone of the zms test, using the retention backend, with support for nrf52840dk and qemu_cortex_m3 Upstream PR #: 93151 Signed-off-by: Jamie McCrae <[email protected]>
Add APP_DIR as a kconfig environment variable. This is useful in cases where you want to set a config path relative to the directory of the application. An example of this is how many sysbuild.cmake files sets the source directory like this: ExternalZephyrProject_Add( APPLICATION remote SOURCE_DIR ${APP_DIR}/remote BOARD ${SB_CONFIG_REMOTE_BOARD} ) The same however cannot be done in Kconfig.sysbuild: config NETCORE_IMAGE_PATH default "${APP_DIR}/<image_path>" if NETCORE_ABC Instead they must use ZEPHYR_MY_MODULE_MODULE_DIR, however not all applications are part of a zephyr module. Signed-off-by: Joakim Andersson <[email protected]> (cherry picked from commit 12fdde6)
Add FPU power management for suspend to RAM procedures. Add FPU save/restore procedures when `FPU_SHARING` feature is disabled. Upstream PR #: 92264 Signed-off-by: Adam Kondraciuk <[email protected]>
Adds aliases so that these devices can use MCUboot serial recovery Upstream PR #: 93090 Signed-off-by: Jamie McCrae <[email protected]>
Ref: NCSDK-34052 Signed-off-by: Krzysztof Szromek <[email protected]>
Certain implementations require more stack to operate. Increase the stack size to avoid overflows. Signed-off-by: Tomi Fontanilles <[email protected]> Upstream PR #: 93147 Signed-off-by: Tomi Fontanilles <[email protected]>
…duce register access" This reverts commit 6346fbb. Signed-off-by: Krzysztof Chruściński <[email protected]>
…ister access Speed up execution of the interrupt handler and sys_clock_set_timeout(). Sys_clock_set_timeout() can be called in two scenarios: from previous timeout expiration handler or freely. If the former case fast path can be used since CC value in the GRTC register just expired and it can be used as a reference for CCADD setting. This is only a single register write so it's much faster. In the latter a longer procedure is applied which also happens in two variants. If value which is set in CC is further in the future (e.g. K_FOREVER was set before) then CC can be safely overwritten with a new value without a risk of triggering unexpected COMPARE event. If value in CC is earlier than the new CC value (if earlier timeout was aborted) then there is a risk of COMPARE event happening while it is being overwritten. That case requires long and safer procedure of setting CC. Update hal_nordic with changes in the nrfx_grtc driver which are needed for nrf_grtc_timer changes. Upstream PR #: 87944 Signed-off-by: Krzysztof Chruściński <[email protected]>
This reverts commit 0d44bac. Signed-off-by: Ravi Dondaputi <[email protected]>
With NRF security, higher heap requirement is seen for operating with RSA-3072 based certificates. Add a NRF SoC specific conf in snippets to handle this. Upstream PR #: 93096 Signed-off-by: Ravi Dondaputi <[email protected]>
Add AES-only cipher certificates. Useful for systems which have only AES support and DES is not enabled. Upstream PR #: 93096 Signed-off-by: Ravi Dondaputi <[email protected]>
…cted storage Add config options required for storing certificates in protected storage, and to free up space of ROM to accommodate the additional features that are enabled for TFM. Upstream PR #: 93096 Signed-off-by: Ravi Dondaputi <[email protected]>
…titioning file Adds a common vendor dts file specifying the default partition layout for nRF54L10-based cpuapp board targets and updates boards to use this common file. This also drops the secure/non-secure split in the partitioning as this was reducing NVM storage that is not used by this board target and fixes the wrong flash field in twister yaml files Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit 9dc6040)
… partitions Adds a workaround for partitions on nrf54l10 non-secure devices to prevent build issues, this needs to be fixed properly when a TF-M update is done which sets the partition sizes properly Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit 002838a)
…titioning file Adds a common vendor dts file specifying the default partition layout for nRF54L15-based cpuapp board targets and updates boards to use this common file. This also drops the secure/non-secure split in the partitioning as this was reducing NVM storage that is not used by this board target and fixes the wrong flash field in twister yaml files Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit 124a803)
…partitioning file Adds a common vendor dts file specifying the default partition layout for nRF54L15-based cpuapp_ns board targets and updates boards to use this common file. Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit e542188)
…titioning file Adds a common vendor dts file specifying the default partition layout for nRF54L05-based cpuapp board targets and updates boards to use this common file. This also drops the secure/non-secure split in the partitioning as this was reducing NVM storage that is not used by this board target and fixes the wrong flash field in twister yaml files Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit 5bc71e6)
Fixes these files to no lonegr delete non-secure partitions as these no longer exist Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit 97e5d33)
…overlay" This reverts commit 8468245. Signed-off-by: Jamie McCrae <[email protected]>
This reverts commit ed81da7. Signed-off-by: Jamie McCrae <[email protected]>
nrf-squash! [nrf noup] samples: smp_svr: configuration for nrf54l15 Fixes this file to be correctly formatted and not delete the (now) non-existent non-secure partitions Signed-off-by: Jamie McCrae <[email protected]>
Add an nPM1304 EK shield, update samples/shields/npm1300_ek to support both shields, improve the sample description. Signed-off-by: Sergei Ovchinnikov <[email protected]> (cherry picked from commit f0f2b94)
Some nRF SoCs (i.e. nRF54H20) can peform DMA transfers only from specific memory regions - `dmm` facilitates that. Upstream PR #: 93487 Signed-off-by: Nikodem Kastelik <[email protected]>
… variant Direct variant of this test disables intermediate buffer in the nRF SPI driver and verifies whether devices like nRF54H20, which can peform DMA transcations only from specific memory regions, are handled properly. Upstream PR #: 93487 Signed-off-by: Nikodem Kastelik <[email protected]>
Bounce buffers should be freed rather than user buffers. Upstream PR #: 93487 Signed-off-by: Nikodem Kastelik <[email protected]>
Improve corss-channel test by setting different duty cycle on channels. This modification will enable detection of inter-channel effects like crosstalk. Upstream PR #: 93776 Signed-off-by: Piotr Krzyzanowski <[email protected]>
Add more channels testing nrf platforms. Upstream PR #: 93776 Signed-off-by: Piotr Krzyzanowski <[email protected]>
…e data Stop processing microphone data on error or when streaming ends. This avoids I2S read timeouts due to audio data not being available while streaming. Signed-off-by: Tomasz Moń <[email protected]> (cherry picked from commit 943fc6f)
This commit resolves a bug where the USB Endpoint provided by the HID driver was incorrect when in High Speed mode. Signed-off-by: Victor Brzeski <[email protected]> (cherry picked from commit c0e1268) Signed-off-by: Tomasz Moń <[email protected]>
Change the macros to have only one plus operator per term. This is purely stylistic change, no functional changes. Signed-off-by: Tomasz Moń <[email protected]> (cherry picked from commit 673f324)
…ndpoints Enable double buffering on data OUT endpoints to allow USB stack to enqueue next transfer as soon as possible. This is especially useful when operating at High-Speed where there is significantly less time between subsequent SOF packets. Signed-off-by: Tomasz Moń <[email protected]> (cherry picked from commit 2f343bf)
…nqueue Release receive buffer on failed endpoint enqueue. Signed-off-by: Tomasz Moń <[email protected]> (cherry picked from commit f9ac318)
…ndpoint Enable double buffering on isochronous feedback endpoint to avoid sending ZLP instead of feedback information. Signed-off-by: Tomasz Moń <[email protected]> (cherry picked from commit 223d23a)
…ing incompisoout Rearm isochronous endpoints when handling incomplete iso out interrupt to make it possible to rearm the endpoint in time (before SOF), especially when operating at High-Speed. Signed-off-by: Tomasz Moń <[email protected]> (cherry picked from commit 6eb2fa8)
Incomplete iso IN/OUT is just informative and its occurrence does not prevent the endpoint from actually transmitting/receiving data. Such "late" isochronous transfers, which are perfectly fine according to USB specification, were observed on Windows host with nRF54H20 running explicit feedback sample operating at High-Speed. The incorrect handling manifested itself with "ISO RX buffer too small" error message. The faulty scenario was: * incompISOIN handler does not find any matching endpoint * incompISOOUT handler disables endpoint, discards buffer and sets rearm flag * next DWC2 interrupt handler iteration after reading GINTSTS * XferCompl interrupt on iso IN endpoint * XferCompl interrupt on iso OUT endpoint - transfer was actually happening to the buffer discarded in incompISOOUT handler - XferCompl handler modified the next buffer * GOUTNakEff interrupt, iso OUT endpoint EPDIS bit is set * EPDisbld interrupt, rearm flag set - the buffer modified by XferCompl is used and fails because it is not large enough Modify the sequence so it accounts for host actions and the above faulty scenario no longer causes any problems. Signed-off-by: Tomasz Moń <[email protected]> (cherry picked from commit 8d1f7b3bef5fcbb63b8dab99f85320f02835fe63)
09193f6
to
8787442
Compare
autoscan: definiton of scan scope and exclusions Signed-off-by: Leif Harald Urlaub <[email protected]>
8787442
to
69433d9
Compare
thst-nordic
approved these changes
Jul 31, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
autoscan: definiton of scan scope and exclusions