Update for rebasing #1
Merged
Conversation
This file contains 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
Usage of K_SPINLOCK with CONFIG_DEBUG=y seems to trigger a compiler warning about request not always being initialized. Fallback to k_spin_lock/unlock calls to fix this issue. Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
Make sure we are not able to overwrite string variables in sscanf call. Allocate also one extra byte for null terminator character. Fixes #80644 Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
Using strncpy gives this warning
warning: 'strncpy' output may be truncated copying 32 bytes from
a string of length 32 [-Wstringop-truncation]
strncpy(roaming_params.neighbor_rep.neighbor_ap[idx].bssid_info,
bssid_info,
sizeof(roaming_params.neighbor_rep.neighbor_ap->bssid_info));
There is '\0' at the end of the allocated buffer so we can safely
use memcpy() here to avoid any warnings.
Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
Deprecation has been postponed one version. Signed-off-by: Johann Fischer <johann.fischer@nordicsemi.no>
Reset control endpoint busy flags if configured and enabled, otherwise it could mark the wrong buffer as busy after endpoint disable/enable. Signed-off-by: Johann Fischer <johann.fischer@nordicsemi.no>
If the direction of the last setup packet is not to the device but to the host, then the transfer is not a status stage and should be queued. This is not checked and prevents a zero length control IN transfer to the host, e.g. used by the DFU class to indicate the end of the upload process. Signed-off-by: Johann Fischer <johann.fischer@nordicsemi.no>
Help text fixes from doc-team. Signed-off-by: Richa Pandey <richa.pandey@nordicsemi.no> Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
With the recent update to Sphinx RTD theme 3.0.0, a 19px-high element that used to display the version number is not visible anymore, so the JS code that hides the upper-left logo when scrolling down needs to be adjusted to account for this change. Signed-off-by: Benjamin Cabé <benjamin@zephyrproject.org>
Fixes: #80193 With #62395 the Zephyr kernel and app version customization values were moved to target properties to allow Zephyr modules to adjust the values. This had the consequence described by #80193 that version customization using CMake arguments, `-D`, or CMakeList.txt toplevel file can no longer be used for customizing the version. To support both CMake variable as well as Zephyr module version customization use-cases then this commit uses `zephyr_get()` to fetch any CMake variable adjustments and uses the value for default property setting. This allows users to set customized version while still allow Zephyr modules to overrule this as intended with #62395. Signed-off-by: Torsten Rasmussen <Torsten.Rasmussen@nordicsemi.no>
add missing newline for sub-list to render correctly Signed-off-by: Benjamin Cabé <benjamin@zephyrproject.org>
In some circumstances the struct eth_stm32_tx_context object that was allocated on eth_tx's stack is still referenced after the function exits. This usually happens when the network is disconnected, depending on the PHY hardware. When the network is reconnected there will eventually be a call to HAL_ETH_ReleaseTxPacket, which calls HAL_ETH_TxFreeCallback with the (now invalid) pointer to the tx context. When HAL_ETH_TxFreeCallback tries to dereference that pointer we get a bus error. Fix this by allocating struct eth_stm32_tx_context objects from a static array, similarly to how the buffers are allocated. This ensures that they remain valid until the HAL is finished with them. Fixes: #79037 Signed-off-by: Kevin ORourke <kevin.orourke@ferroamp.se>
The new update of clock device tree make the pll p q r clock source cannot be choose by other node This fix add 1 new dts binding for pll out p q r out line Signed-off-by: Duy Nguyen <duy.nguyen.xa@renesas.com>
Update hal renesas commit ID to resolve PLL clock config issue Signed-off-by: Duy Nguyen <duy.nguyen.xa@renesas.com>
The pm_policy_event_register() API takes absolute cycles as the second arg, like pm_policy_event_update(), but the arg is renamed time_us and treated as a relative time in us rather than abs cycles. Fix implementation of pm_policy_event_register() to treat cycles like pm_policy_event_update() and API docs suggest. Signed-off-by: Bjarki Arge Andreasen <bjarki.andreasen@nordicsemi.no>
According to the datasheet, the PHY address is 0x1. When changed to this value the PHY id is correctly read. Before: ``` [00:00:00.602,000] <err> phy_mii: No PHY found at address 0 ``` After: ``` [00:00:00.051,000] <inf> phy_mii: PHY (1) ID 7C111 ``` Signed-off-by: Tomáš Juřena <jurenatomas@gmail.com>
Add support for the W25Q512NW-IQ/IN with the FLEXSPI, using a custom LUT table. Fixes #80592 Signed-off-by: Daniel DeGrasse <daniel.degrasse@nxp.com>
RT11xx SOC init should check to see if the zephyr flash node is set to a device on the FLEXSPI bus to determine if the part is running in XIP mode. This check was incorrect, so the FLEXSPI was being reclocked in XIP mode to 24 MHz. Fix this check so the FlexSPI is not downclocked. Fixes #75702 Signed-off-by: Daniel DeGrasse <daniel.degrasse@nxp.com>
Remove memory spaces not needed for the counter_basic_api test. Add frdm_rw612.overlay Signed-off-by: David Leach <david.leach@nxp.com>
Test was marked build only with filters in each scenario looking for sdl-dc which is only available on native_sim, so cut the chase and use platform_only to narrow things down to native_sim directly instead of building the world to get information we already know. reduces build/run time from 78s to 17s on invocation of twister with default options, saves a ton more when running twister with --all. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Make sure this expansion doesn't include `CONFIG_DCACHE_LINE_SIZE`, which would be undefined and produce a build error. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no>
Add information about CAN supported on these boards: - ek_ra8m1 - ek_ra8d1 - mck_ra8t1 Signed-off-by: The Nguyen <the.nguyen.yf@renesas.com>
As part of the deprecation process of TinyCrypt started in #79566, this commit set the TinyCrypt based crypto shim driver as deprecated. Signed-off-by: Valerio Setti <vsetti@baylibre.com>
As for the IMX SOCs all the lines removed in this commit were actually commented out so there's basically no change in code behavior expected here. The only affected SOCs family is therefore the Kinetis one. Signed-off-by: Valerio Setti <vsetti@baylibre.com>
Following the deprecation of TinyCrypt (#79566) we remove tags referring to it. Signed-off-by: Valerio Setti <vsetti@baylibre.com>
Following the deprecation of TinyCrypt (#79566) we remove TinyCrypt usage in random generators. This basically only affects the CTR-DRBG random generator which from now only will only make use of Mbed TLS. Signed-off-by: Valerio Setti <vsetti@baylibre.com>
As part of TinyCrypt deprecation process (#79566) this commit removes usage of this library from the JWT subsystem and its related tests. Signed-off-by: Valerio Setti <vsetti@baylibre.com>
Fix spurious ISO Sync Receiver stall due to uninitialised value accessed due to regression introduced by commit 64facee ("Bluetooth: controller: Stop Sync ISO ticker when establishment fails"). Signed-off-by: Vinayak Kariappa Chettimada <vich@nordicsemi.no>
Instead of checking the `enabled` flag, check if BT_MESH_SUSPENDED is set in the legacy advertiser thread. BT_MESH_SUSPENDED is set earlier than advertiser is stopped and will prevent the advertiser send anything earlier. Signed-off-by: Pavel Vasilyev <pavel.vasilyev@nordicsemi.no>
k_sleep may not be enough to let advertiser send the message. Instead we should rely on the bt_mesh_send_cb. Signed-off-by: Pavel Vasilyev <pavel.vasilyev@nordicsemi.no>
Fix the include guard mismatch. Found building with clang. Signed-off-by: Carles Cufi <carles.cufi@nordicsemi.no>
[1] was introduced to get more valuable answers from the PropertySpec.path API, which is supposed to tell in which file the property's specification was "last modfied". Further work on related issues [2] showed that the approach chosen in [1] is dead end: we need to first rethink how bindings (and especially child-bindings) are initialized. [1] edtlib: fix last modified semantic in included property specs [2] edtlib: Preserve paths of properties from included child bindings See also: #65221, #78095 This reverts commit b3b5ad8. Signed-off-by: Christophe Dufaza <chris@openmarl.org>
add missing switched_out trace point. Partially fixes #76057 Signed-off-by: Anas Nashif <anas.nashif@intel.com>
GAP PAST (and BAP) tests were also enabled since controller support is under review. This also adds bqw file which is exported draft project from Qualification Workspace. Signed-off-by: Szymon Janc <szymon.janc@codecoup.pl>
This completes the existing "Documentation" section of the release notes for 4.0 by documenting the various changes/improvements implemented during the last development cycle. Signed-off-by: Benjamin Cabé <benjamin@zephyrproject.org>
On nRF54H and nRF92, booting certain cores requires programming a UICR, which is normally generated using nrf-regtool. This should be considered an optional dependency, because we do not wish to force non-Nordic users to install it just to work with Zephyr, or just for build-only tests. When nrf-regtool is not installed, a CMake warning will be displayed, but people ignore warnings all the time. As the last line of defense, check for missing UICR in the nrfutil flash runner, to prevent our users from unintentionally programming unbootable firmware. Show a fatal error specifically if CONFIG_NRF_REGTOOL_GENERATE_UICR=y, yet no UICR exists. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no>
mdio_enet_nxp driver accesses the registers of its parent node Ethernet MAC This commit enables this node in mimxrt1062_fmurt6 board's device tree. This also fixes Issue #80881 Signed-off-by: Sumit Batra <sumit.batra@nxp.com>
This adds some bits about demand paging in release note for 4.0. Signed-off-by: Daniel Leung <daniel.leung@intel.com>
This adds some bits about serial/UART in the migration and release notes for 4.0. Signed-off-by: Daniel Leung <daniel.leung@intel.com>
Add entries for video driver contributed or enhanced. Add entries for video APIs introduced or modified. Signed-off-by: Josuah Demangeon <me@josuah.net>
Update board name to meet zephyr:board model. Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
Fix typo and re-phrase help description to improve it. Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
Adds notes on Espressif changes to this release. Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
fix wrong board target in build command snippets Signed-off-by: Benjamin Cabé <benjamin@zephyrproject.org>
Add notes about the new storage system ZMS Signed-off-by: Riadh Ghaddab <rghaddab@baylibre.com>
Add an extra "git show" step to the release process to verify that the key has been indeed signed. Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>
Update hal_nxp to fix hang when board reset Signed-off-by: Gaofeng Zhang <gaofeng.zhang@nxp.com>
Validates cmux frame length and drops it if its larger than the receive buffer Signed-off-by: Henrik Skreslet <henrik.skreslet@gmail.com>
Small mismatch between the actual procedure name and the name as described in the doxygen documentation for the bap set broadcast code procedure Signed-off-by: Andries Kruithof <andries.kruithof@nordicsemi.no>
As other targets in the LPC55xxx series, the LPC55S36 has a Flash controller that raises ECC errors when reading erased pages directly. To avoid this, there is special code for this platform that calls the HAL FLASH_IsFlashAreaReadable() function. However, this in turn calls a function at an hardcoded address in ROM that _always_ causes an instruction fault, making the situation worse. This patch reworks the read operation to use the FLASH_Read() HAL function for this target to gracefully handle error conditions and properly emulate accesses to erased pages. The preprocessor is required since some targets do not define the FLASH_Read() function. Fixes: #80325 Signed-off-by: Luca Burelli <l.burelli@arduino.cc>
The current alignment logic does not work as expected if given unaligned values, resulting in a skip of the first word. The length also has to take into account the starting address: for example, asking for 2 bytes at offset 3 should actually check 8 bytes. This patch adjusts the logic so that it always includes the first and the last word of the input area. Signed-off-by: Luca Burelli <l.burelli@arduino.cc>
The capture to LVGL sample is using an incorrect name for the weact_ov2640_cam_module shield Signed-off-by: Benjamin Cabé <benjamin@zephyrproject.org>
Add checks in the GPIO bitbang mode to avoid a fault for missing configuration in the devicetree. Fixes #80850 Signed-off-by: Benedikt Schmidt <benedikt.schmidt@embedded-solutions.at>
a typo in usb2 clock initialization which impact the function of usb2. fixes: #81027 Signed-off-by: Raymond Lei <raymond.lei@nxp.com>
The test had a few off-by-ones in the code, which caused access to invalid data. Fixed by setting the right buffer sizes and the right AD length fields. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
This patch fixes an issue where the reset pin is used even when it's not given. Signed-off-by: Maximilian Deubel <maximilian.deubel@nordicsemi.no>
`STM32_PERIPH_BUS_MIN` is not the minimum bus address in `stm32u0_clock.h` Signed-off-by: Arif Balik <arifbalik@outlook.com>
A call to the write API function was never returning as we were trapped into an infinite loop. This was caused by the pixel_count pointer not being incremented properly. Signed-off-by: Johan Lafon <johan.lafon@syslinbit.com>
In case more than one segment per pixel is required, only part of the segments were written resulting in low and uneven pixel brightness (at least on NHD-2.7-12864WDW3). Fix it by writing all required segments. Signed-off-by: Johan Lafon <johan.lafon@syslinbit.com>
Only the first chunk of the desired image is displayed repeatedly over the screen height (at least on an NHD-2.7-12864WDW3 display). Fix it by computing the internal mono01 to 4bit grayscale conversion buffer size correctly so it can fit the entire image. Signed-off-by: Johan Lafon <johan.lafon@syslinbit.com>
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.
No description provided.