[ESP32] Fix incorrect RTC log format in SystemTimeSupport #37534
+105,732
−52,302
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
- Working backend with metric, trace and counter diagnostics - Diagnostic interface implementation with ring buffer storage - Added option ENABLE_ESP_DIAGNOSTICS_TRACE in chip KConfig - Added required options for enabling matter diagnostic trace in project Kconfig
- Resolve buffer issues - Use single buffer for store and retrieve of diagnostics - Resolve data loss issue
…essary comments, format files, namespace changes
- backend: Replace linkedlist with map for counter diagnostics - backend: Pass diagnostic storage as a parameter to backend - esp32_diagnostic_trace: Return actual data size
- esp32_diagnostic_trace: improve storage class design - add extra tlv closing bytes check before copying diagnostic - unify diagnostic entries into a single type
- Add related logic in temperature-measurement-app
- Add private CircularTLVReader and CircularTLVWriter - Remove redundant outer container for tlv's from Retrieve method - Maintain TAG's for diagnostic entry elements - Make diagnostic entry as a constant param to store method - Move Murmurhash to utils namespace
…gnostics - move diagnostic storage buffer to main
…ter to LogProvider - add kconfig option for retrieval buffer size
- pass diagnostic storage instance as a pointer to backend - replace ESPLog statement with ChipLog for logging diagnostic-storage: update IsEmptyBuffer method to IsBufferEmpty - change diagnostic TAG enum class
* Create basic Unit Test for Verhoff * Change the build script for Accessor class * Change structure and remove private accessors * Change dependencies in build file * Reduce dependencies list * Remove empty sources Editorial commit: remove extra `sources = []`, I expect our templates to handle this one missing. --------- Co-authored-by: Andrei Litvin <[email protected]>
Added package libpcsclite-dev to start-sysroot-vm.sh. This package is needed for the cross compilation of chip-tool for arm64.
… dependencies and unify initialization function (project-chip#37135) * Changes the WiFi initialization and adds it to platfrom init flow * Rename API based on refactor * Add debug log for failed sl_matter_wifi_init * Refactor platform-specific includes in MatterConfig.cpp * Refactor WiFi initialization to remove platform-specific dependencies and unify initialization function * Refactor InitWiFiStack to return CHIP_ERROR and improve error handling across implementations * Improve documentation for wfx_bus_start function * Remove error log from wfx_bus_start and update documentation for memory allocation failure * Apply suggestions from code review Co-authored-by: Mathieu Kardous <[email protected]> * Refactor InitWiFiStack error handling to use appropriate CHIP_ERROR codes and update documentation * Update src/platform/silabs/wifi/WifiInterface.h Co-authored-by: Mathieu Kardous <[email protected]> --------- Co-authored-by: Mathieu Kardous <[email protected]>
* [Zephyr] Add ZMS to Zephyr port Added a possibility to use ZMS fs backend in Zephyr. NVS fs backend imply is now controlled by the nrfconnect platform configuration directly. All other platforms use NVS fs backend by default. Signed-off-by: Arkadiusz Balys <[email protected]> [nrf toup] Fix defines for ZMS It can be squashed with the 0c49058 commit. Signed-off-by: Arkadiusz Balys <[email protected]> * [Zephyr] Fix ICD LIT connected configurations According to specification, if ICD LIT is supported, UAT and LITS features must be supported too. If `CHIP_ICD_LIT_SUPPORT` is selected, select CIP and UAT configs instead of implying them. Signed-off-by: Maciej Baczmanski <[email protected]> * [nrfconnect] Introduced Kconfigs to configure few params Introduced new Kconfig options to be able to configure the parameters related to persistent subscriptions re-establishment. Signed-off-by: Kamil Kasperczyk <[email protected]> [nrfconnect] Added module to synchronize DFU processes Introduced a module that can be used to synchronize DFU processes and make it mutual exclusive (prevent concurrent DFU runs using different protocols, e.g Matter OTA and DFU over BT SMP). Signed-off-by: Kamil Kasperczyk <[email protected]> [nrfconnect] Implement IPv6 agnostic L2 network state getters * Added new API for generic IPv6 connectivity checks * This patch allows the application layer to limit the amount of pre-processor logic (CONFIG_NET_L2_OPENTHREAD/CONFIG_CHIP_WIFI) needed to configure the Signed-off-by: Marcin Kajor <[email protected]> [nrfconnect] Added support for settings shell in mem profiling Enabled settings shell configuration and increased shell command buffer size when `CHIP_MEMORY_PROFILING` is enabled to allow reading and writing settings using shell. Signed-off-by: Maciej Baczmanski <[email protected]> [nrfconnect] Introduced Kconfig to configure CHIP stack size There is not Kconfig that would allow to set the CHIP task size, what is not convenient, as other stacks can be configured using Kconfig. [nrfconnect] Added log for the WiFi connection failure This might help when debugging the WiFi connection failures. Signed-off-by: Marcin Kajor <[email protected]> [nrfconnect] Refactor ExternalFlashManager Make `ExternalFlashManager` a singleton Signed-off-by: Maciej Baczmanski <[email protected]> [nrfconnect] Increase workqueue stack size for LIT Whien LIT is enabled, higher workqueue stack size is required when entering active mode. Signed-off-by: Maciej Baczmanski <[email protected]> [nrfconnect] Switch to nRF54L15 DK - Removed support for nRF54L15 PDK and added for nRF54L15 DK [nrfconnect] wifi: Migrate to nRF70 upstream nRF70 support moved upstream, update the names. Signed-off-by: Chaitanya Tata <[email protected]> [nrfconnect] matter: crypto: Use psa_crypto_config -Previously there was a library called mbedcrypto_common which was used to pass build-related configurations to matter. This is changed to instead reference to the following: -psa_crypto_config (for PSA and Mbed TLS configurations) -psa_interface (for PSA crypto include files) -The commit also removes passing the PSA crypto driver config file as this is no longer needed for an interface build (call from matter) Signed-off-by: Frank Audun Kvamtrø <[email protected]> [nrfconnect] Fixed mbedtls configs selection The updated way of handling mbedtls config file is done correctly only for TF-M builds, but without it some of the files are missing Signed-off-by: Kamil Kasperczyk <[email protected]> [nrfconnect] Disable unsupported SHELL_WILDCARD SHELL_WILDCARD is not supported for Matter devices in the newest Zephyr version because it selects POSIX_C_LIB_EXT, so we need to disable it. [nrfconnect] Enable SPI for nRF54L15DK NS build Added nRF54L15 DK NS to configuration. Signed-off-by: Arkadiusz Balys <[email protected]> [nrfconnect] config: align the MCUmgr Bluetooth transport Kconfig Aligned the MCUmgr Bluetooth transport Kconfig with the latest changes from Zephyr. Ref: NCSDK-29061 Signed-off-by: Kamil Piszczek <[email protected]> [nrfconnect] Enable MLDv2 join procedure for IPv6 Multicasts This commit updates the IPv6 multicast subscription mechanism by explicitly using the MLDv2 join procedure. This change ensures proper registration for multicast addresses, improving reliability in multicast communication. Signed-off-by: Łukasz Duda <[email protected]> [nrfconnect] Implicitly enable PSA crypto driver for given platform This commit makes in clear which crypto driver is enabled by default. Additionally, stack overflow has been fixed for CC3XX variant, and mbedTLS Heap is disabled for TF-M variant. Signed-off-by: Łukasz Duda <[email protected]> [nrfconnect] wifi: Fix nRF70 heap allocation The upstream WPA supplicant uses k_heap instead of libc_heap, so, move the heap to k_heap. Signed-off-by: Chaitanya Tata <[email protected]> [nrfconnect] wifi: Fix the heap allocation warning By default nRF70 driver allocates a higher heap to handle most cases, but as Matter doesn't use much heap, enable the ignore minimum warning configuration. Signed-off-by: Chaitanya Tata <[email protected]> [nrfconnect] Increased stack size for the OpenThread with nRF54L When using nRF54L15 the stack size required for crypto operations by OpenThread is bigger and must be increased. Signed-off-by: Kamil Kasperczyk <[email protected]> [nrfconnect] Enable ZMS in nrfconnect if RRAM is in use. Disabled NVS and enabled ZMS fs backend for all devices that uses RRAM. Signed-off-by: Arkadiusz Balys <[email protected]> [nrfconnect] Increased CHIP stack size for cracen based targets Targets using cracen crypto backend requires bigger CHIP stack size. Signed-off-by: Kamil Kasperczyk <[email protected]> [nrfconnect] disable `FPROTECT` for nRF54L15 `FPROTECT` should be disabled for nRF54L15 in app to allow correctly protecting whole region of mcuboot Signed-off-by: Maciej Baczmanski <[email protected]> [nrfconnect] Factory data partition location change - factory data can be placed before or after settings partition Signed-off-by: Konrad Grucel <[email protected]> [nrfconnect] Added config to enable report on active mode Introduced Kconfig option that allows to enable an ICD to send data report on entering the active mode. Signed-off-by: Kamil Kasperczyk <[email protected]> [nrfconnect] Increased system workqueue size for LIT The LIT devices seem to need bigger size of workqueue on 54L. Signed-off-by: Kamil Kasperczyk <[email protected]> [nrfconnect] Change `CHIP_SPI_NOR` config dependency Set `CHIP_SPI_NOR` to y for all targets that are built for nRF54L15DK, regardless of target SoC. Signed-off-by: Maciej Baczmanski <[email protected]> [nrfconnect] Disable chip_build_tools Disable chip_build_tools Signed-off-by: Adrian Gielniewski <[email protected]> * [Crypto] Stop compiling legacy SPAKE2P if PSA is available If the CHIP_CRYPTO_PSA_SPAKE2P is enabled the build system should not compile Legacy Spake2p implementation, because the PSA one is available in the PSASpake2p.cpp file. * VerifyOrDie logging for constrained devices 1. Add the configuration to enable logging the location of a failed VerifyOrDie() without logging the condition to reduce the code size impact but still be able to debug failing VerifyOrDie() conditions. 2. Allow to override __FILE__ macro with __FILE_NAME__ by setting the warn_builtin_macro_redefined GN arg to false to further reduce the code size increase. 3. Add Kconfigs for nRF Connect platform for enabling both features. Signed-off-by: Damian Krolik <[email protected]> * Include EXTRAVERSION in Matter software version string When a Matter application provides the VERSION file, the Matter software version string is set to 4-number string, such as "2.7.99+0" and the EXTRAVERSION field, which is typically used to denote the branch or tag name, such as "dev" or "rc1", is ignored. If the EXTRAVERSION field is non-empty, set the Matter software version string to a string such as "2.7.99-dev+0". * Clear Thread Network after last fabric is removed Dataset has to be cleared to allow commissioning to different network when CONFIG_CHIP_LAST_FABRIC_REMOVED_ERASE_ONLY is used. Signed-off-by: Adrian Gielniewski <[email protected]> * [workflows] Update docker images to version 98 Update docker image version to 98. Signed-off-by: Adrian Gielniewski <[email protected]> * [nrfconnect] Change the recommended version to 2.9.0 Change the recommended version of nRF Connected SDK to 2.9.0. Signed-off-by: Adrian Gielniewski <[email protected]> * [nrfconnect] Align examples to NCS 2.9.0 Signed-off-by: Adrian Gielniewski <[email protected]> * [nrfconnect] Fix FLASH overflow Disable some logs in lighting-app and lock-app due to flash overflow. Signed-off-by: Adrian Gielniewski <[email protected]> * [nrfconnect] Align test_driver configuration to NCS 2.9.0 Signed-off-by: Adrian Gielniewski <[email protected]> * [nrfconnect] Fix nrf builder Use correct path when running ctest. Signed-off-by: Adrian Gielniewski <[email protected]> * [nrfconnect] Disable native build Disable Zephyr native build until next release due to Zephyr regression. Signed-off-by: Adrian Gielniewski <[email protected]> --------- Signed-off-by: Arkadiusz Balys <[email protected]> Signed-off-by: Maciej Baczmanski <[email protected]> Signed-off-by: Adrian Gielniewski <[email protected]> Signed-off-by: Damian Krolik <[email protected]> Co-authored-by: Arkadiusz Balys <[email protected]> Co-authored-by: Maciej Baczmanski <[email protected]> Co-authored-by: Kamil Kasperczyk <[email protected]> Co-authored-by: Damian Krolik <[email protected]>
* Run dos2unix on all DM XML files in 1.4 * Add newlines at end of all DM XML files for 1.4 sed -i -e '$a\' data_model/1.4/device_types/* sed -i -e '$a\' data_model/1.4/clusters/* * DM XMLs: switch 1.4 DM XMLs to alchemy * Generator updates to alchemy * Update 1.4 using alchemy 1.5.0 * Restyled by autopep8 * Add the in-progress defines to match the 1.5 0.7 spec ballot * Fix define processing for alchemy, add tag file * Fix linter, add check for non-existent tag file * Remove a leftover file - this isn't generated by alchemy * Remove file from gni list --------- Co-authored-by: Restyled.io <[email protected]>
…-chip#37302) Signed-off-by: Adrian Gielniewski <[email protected]>
* Changed Dockerfile for crosscompile image. Now pointing to a new sysroot. Changed needed for NFC Commissioning. * Bumped version --------- Co-authored-by: Andrei Litvin <[email protected]>
github-actions
bot
added
platform
darwin
esp32
linux
nrf connect
zephyr
lib
crypto
controller
app
config
github
workflows
gn
telink
shell
tests
support
core
test driver
tizen
|
PR #37534: Size comparison from c81b3d4 to e899391 Full report (83 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
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.
Problem
Fixes espressif/esp-matter#1269
The
tm_yearandtm_monvalues were being logged incorrectly, resulting in an invalid date format.Changes
tm_yearby adding 1900.tm_monby adding 1 to convert from zero-based indexing.Testing
chip-tooland checking the log output.