Update zephyr 3.4 #7
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
Add the description of each error code as defined by the core spec and the CSS. Signed-off-by: Emil Gydesen <[email protected]>
In case lwm2m_socket_start() internal error, it should only do cleanup on the socket, i. e. call lwm2m_socket_close(), not lwm2m_engine_stop(). The latter resets the entire lwm2m_context, which results in removal of active observations. This should not be done, as it collides with the RD client logic, where connection resumption may skip the full registration phase, in result not notifying the server that it should restart the observations. At the same time, the RD client should clean the lwm2m_context when it's done trying to update the registration and proceeds with regular registration/bootstrap in the network error handler. In that case, only the socket was closed, so the lwm2m_context needs to be reset separately. Signed-off-by: Robert Lubos <[email protected]>
This makes Kconfig logging options for this driver depending on whether the driver is enabled or not. Signed-off-by: Caspar Friedrich <[email protected]>
Add UTIL_INC, UTIL_DEC and UTIL_X2 macros Signed-off-by: Erwan Gouriou <[email protected]>
…alues Compiling previous code using LLVM reveals that accessing line_range values in runtime is not clean, as the following error is generated: "error: initializer element is not a compile-time constant" Replace runtime method by compile time using UTIL_INC and UTIL_x2 macros. Signed-off-by: Erwan Gouriou <[email protected]>
Based on interpretation of Clarification Errata ES-22876 Request for Clarification - Recombination actions when only padding unframed PDUs are received, it was decided that an SDU should be released as "Missing" when only padding PDUs are received for any SDU. - https://bluetooth.atlassian.net/browse/ES-22876 This change will also assist streaming in the Common Central design where the central was found to send only padding PDUs before streaming starts. Signed-off-by: Nirosharn Amarasinghe <[email protected]>
Testing: -- Receiving only padding PDUs for an SDU -- Receiving leading padding PDUs (not an expected use case) Signed-off-by: Nirosharn Amarasinghe <[email protected]>
Time wrapping was omitted at some points. These were corrected or clarified. Signed-off-by: Nirosharn Amarasinghe <[email protected]>
-- Implemented reporting of lost / missing SDUs for framed PDUs. This should result in SDUs being released when only framed padding PDUs or error PDUs are received. -- Removed unused configuration params structure, storage and linked APIs. -- Reduced default logging level to INF instead of DBG. -- Included debug logging messaged for framed consumption. -- Removed separate latency_unframed and latency_framed and replaced with a common sdu_sync_const as they are mutually exclusive for each sink. -- Restricted framed time stamp calculation only to situations where all dependent information is available. -- Corrected SDU sequence numbering to release starting at zero. Signed-off-by: Nirosharn Amarasinghe <[email protected]>
-- Updated test cases to include missing / lost SDUs. -- Included tests for receiving only framed padding PDUs. -- Removed tests linked to sink and source configuration params as these were unused and removed. Signed-off-by: Nirosharn Amarasinghe <[email protected]>
Path for cache directory has changed, adapt workflow accordingly. Signed-off-by: Anas Nashif <[email protected]>
This board as a dedicated console serial header right next to the stlink connector. This serial header is connected to usart1. This patch fixes the pinmux for usart1 and makes it the default console serial port. Signed-off-by: Sahaj Sarup <[email protected]>
This board has boot mode pins, 5 and 6 on the J3 header, that puts the board into dfu mode. This patch adds that as a supported flashing config, default still remains openocd with jlink. Signed-off-by: Sahaj Sarup <[email protected]>
In order to support external IP stacks that might have Connection ID support, the LwM2M engine should allow client to bypass default behaviour. New set_socketoptions() callback added into client context that allows overriding all socket opetions. This is called after a socket is opened, but before the connect() is called. This cannot be combined with load_credentials() callback on all platforms as for example nRF91 requires modem offline when credentials are written. This would cause socket to be closed as well. Second change is that we allow fine tuning of what we do with socket handle when QUEUE mode is enabled and engine enters idle state. First option would be to close the socket. That would cause TLS Alert(Close Notify) to be send. This is a band choice if LTE modem was already in PSM or eDRX power saving mode. Second option would be to delay socket closing until we are going to send LwM2M update. There TLS Alert is also send, but most probably lost due to NAT mapping timed out. This is a best choice for LTE modem with DTL session cache enabled. Two new options are to keep socket open, and either stop listening or just keep listening. Both of these options work fine when we have DTLS Connection ID support. Signed-off-by: Seppo Takalo <[email protected]>
`local_addr` would only be initialized if `context->local->sin*_addr` was non-null. However, since `context` is a fresh context object, `local_addr` always remains at its initial value of `INADDR_ANY`, which is propagated to the context by `net_context_bind()`. By populating `local_addr` using the TCP endpoint, `getsockname()` now returns the correct local address. Signed-off-by: Armin Brauns <[email protected]>
The port we are connecting to is stored in network byte order, thus, we need to convert it to the CPU's byte order before logging Signed-off-by: Mark Oude Elberink <[email protected]>
1. use 'BIT' macro instead of explicit left shift operations, since 'BIT' macro implicitly adds unsigned type suffix, avoiding possible [shiftTooManyBitsSigned] scenarios, and 2. add an explicit uint32_t cast to 'BIT' macro argument, complying with required [misra-c2012-10.1] rule which states; operands shall not be of an inappropriate essential type. Found as a coding guideline violation (Rule 10.1) by static code scanning tool. Note: Tested on STM32L5 Nucleo-144 board (stm32l552xx). Signed-off-by: ferar alashkar <[email protected]>
add an unconditional break to switch-clause's default case, complying with required [misra-c2012-16.3] rule which states; An unconditional break statement terminate every switch-clause. Found as a coding guideline violation (Rule 16.3) by static code scanning tool. Note: Tested on STM32L5 Nucleo-144 board (stm32l552xx). Signed-off-by: ferar alashkar <[email protected]>
…meters This fixes missing guard access to GATT read parameters. The code checks `busy` flag and returns an error in case there's ongoing GATT Read operation already. Fixes: zephyrproject-rtos#58425 Signed-off-by: Mariusz Skamra <[email protected]>
This fixes repeated attempt to bt_tbs_client_discover that failed because srv_inst->current_inst was not cleared during previous discovery process. The issue seen in case the GATT Read CCID returned an error. The code has been factored out to separate helper function. Signed-off-by: Mariusz Skamra <[email protected]>
…ction As long as we do not have NVS support in TBS, the subscriptions shall be removed once ACL is disconnected to avoid potential crash on reconnection, as the subscription parameters might be not valid anymore. Signed-off-by: Mariusz Skamra <[email protected]>
In case TCP connection is being closed from the TCP stack, due to for instance retransmission timeout, the stack should also switch the TCP state to CLOSED. Otherwise, there was a risk of dereferencing the TCP context twice, for example if the application was in active socket send(), and tried to reschedule data transmission. Additionally, make sure that the TCP_CLOSED state handling is a no-op state - otherwise, there is a risk that if packets keep incoming before the application dereferences the TCP context on its side, TCP stack will incorrectly dereference the context for the second time from within due to current TCP_CLOSED state logic. Signed-off-by: Robert Lubos <[email protected]>
Mass storage sample contains separate Kconfig options for RAM, Flash and SDMMC disk drivers. All disk drivers can be enabled simultaneously and therefore the maximum number of concurrent LUNs is 3. Configure Mass Storage class maximum LUNs to 3 to solve initialization failures on targets that enable more than one disk driver at a time. Each additional supported LUN increases the Mass Storage class memory usage by 48 bytes (struct scsi_ctx) on 32-bit targets. Allocating 3 SCSI context structures allows the sample to work regardless what disk drivers are concurrently enabled. Fixes: zephyrproject-rtos#57657 Signed-off-by: Tomasz Moń <[email protected]>
Create cache directory before moving ccache. Signed-off-by: Anas Nashif <[email protected]>
The field offset is incorrect, L1EXP is at bit 24 and L1ETP at bit 25. Fixes: cc6e9c0 ("soc/intel_adsp: Low level HDA driver and tests") Signed-off-by: Pierre-Louis Bossart <[email protected]>
By adding the link to an openAMP project reference Linux image targeted to the kv260 board which comes with openAMP support and remoteproc endpoints. Signed-off-by: Felipe Neves <[email protected]>
Move the two UART nodes so that they are under "soc" rather than "espi", leave only xec-espi-host-dev nodes there. The UART device can be used indepdently by the driver uart_mchp_xec.c and it's normally initialized before before the espi one. Moving the device node up a level so this does not trigger a false positive on the build time priority checking. Signed-off-by: Fabio Baltieri <[email protected]>
Promote Task Watchdog API from Experimental to Unstable. The API has proven to be suitable for its purpose. Signed-off-by: Martin Jäger <[email protected]>
Promote DAC API from Experimental to Unstable. The API is well adopted and used in more than 2 implementations. Signed-off-by: Martin Jäger <[email protected]>
add explicit boolean type to 'if' statement controlling expression, consolidating it with 'stm32_exti_is_pending' function return type, thus improving code readability and maintainability, complying with required [misra-c2012-14.4] rule which states; The controlling expression of an if statement and the controlling expression of an iteration-statement shall have essentially boolean type. Found as a coding guideline violation (Rule 14.4) by static code scanning tool. Note: Tested on STM32L5 Nucleo-144 board (stm32l552xx). Signed-off-by: ferar alashkar <[email protected]>
It doesn't make sense to select this option at SoC level. This feature is meant for subsystems/modules that need device handles to be modifiable at runtime. Signed-off-by: Gerard Marull-Paretas <[email protected]>
If CONFIG_MCUX_DCNANO_LCDIF_EXTERNAL_FB_MEM is not set and CONFIG_MCUX_DCNANO_LCDIF_FB_NUM is zero, a division by zero occurs. Use the same framebuffer size definition as in the external-framebuffer case to resolve this. Fixes zephyrproject-rtos#58908. Signed-off-by: Aedan Cullen <[email protected]>
Added information about CVEs fixed during 3.4 release. Signed-off-by: Flavio Ceolin <[email protected]>
Add a placeholder for CVE-2023-1901 and CVE-2023-1902. Signed-off-by: Flavio Ceolin <[email protected]>
Add PM release notes. Signed-off-by: Flavio Ceolin <[email protected]>
The prompt is referring to another existing driver(NXP S32), now the correct name is displayed. Signed-off-by: Thomas Stranger <[email protected]>
Update typo in the code comments. Signed-off-by: Divin Raj <[email protected]>
This use of offset 'i' should follow the range check. Signed-off-by: Mingjie Shen <[email protected]>
Cavs25 sram size should be 3MB instead of 2MB, thus update the correct value. Signed-off-by: Jaska Uimonen <[email protected]>
Fix an issue causing a wrong Bluetooth identity value to be stored. It was happening because the `bt_dev.id_count` was incremented after the settings being stored. To fix this, `bt_dev.id_count` is now incremented right before the ID creation and is decremented if the ID creation failed. Signed-off-by: Théo Battrel <[email protected]>
Add a new BabbleSim test checking that the value of the Bluetooth identity stored in the settings is correct. Signed-off-by: Théo Battrel <[email protected]>
Add release notes for ADC and PWM drivers, Nordic HAL and other things related to Nordic. Signed-off-by: Andrzej Głąbek <[email protected]>
Add release notes for ARC for 3.4 release Signed-off-by: Eugeniy Paltsev <[email protected]> Signed-off-by: Evgeniy Paltsev <[email protected]>
Without the extra spaces Sphinx does not render this correctly. Signed-off-by: Carles Cufi <[email protected]>
DSP AGU/XY extensions ARC EM processors are supported now in Zephyr. Signed-off-by: Eugeniy Paltsev <[email protected]> Signed-off-by: Evgeniy Paltsev <[email protected]>
Typo fix. Signed-off-by: Eugeniy Paltsev <[email protected]> Signed-off-by: Evgeniy Paltsev <[email protected]>
Remove 'Working Draft' from title. Signed-off-by: Anas Nashif <[email protected]>
Use past tense in some of the release notes. Signed-off-by: Anas Nashif <[email protected]>
Remove all placeholders and empty sections. Signed-off-by: Anas Nashif <[email protected]>
One more entry about ztest API being deprecated. Signed-off-by: Anas Nashif <[email protected]>
Add the new release to the doc index. Signed-off-by: Anas Nashif <[email protected]>
Remove all old releases from the index. Signed-off-by: Anas Nashif <[email protected]>
LTS is now 2.7.5. Signed-off-by: Anas Nashif <[email protected]>
Make this the final release. Signed-off-by: Anas Nashif <[email protected]>
Zephyr v3.4.0
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.