update tcp_mdns from master #182
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
Owner
hnnajh
commented
Jan 20, 2024
hnnajh
commented
…t spec (#30955) * update valve configuration and control cluster according to latest spec * DefaultOpenDuration is writable and updated device type name to Water Valve * moved domain to be the first element
* DeviceConformance: Factor out conformance checks This will let us use conformance as a pre-requisite in other tools. * Generates a minimal representation of a device This representation gives all the top level optional elements that are implemented on the device. Anything that DOES NOT appear in this representation is either mandatory or disallowed based on the elements represented here. * Fix linter * Restyled by isort * address review comments --------- Co-authored-by: Restyled.io <commits@restyled.io>
* TC-ACL-2.2: Implement in python Lets us do the all endpoint check in a sane way. * flake8 * Apply suggestions from code review Co-authored-by: René Josefsen <69624991+ReneJosefsen@users.noreply.github.com> --------- Co-authored-by: René Josefsen <69624991+ReneJosefsen@users.noreply.github.com>
#31080) * Add Nonce validation and parsing unit tests * Add missing comments * Remove unused test signature * fix comment * remove include * fix comment
To save some FLASH and RAM memory space we can disable Read Client functionality in nrfconnect samples. This commit adds kconfig to enable or disable Read Client in the Interaction Model.
* Added just the xml and regenerated. No implementations yet as requested. * regen * Apply suggestions from code review Added suggestions for ARRAY vs array Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * regenerate --------- Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
* Fix #30665 (EVSE) - Changed to use amperage_mA, energy_mWh - removed max on epoch_s - removed access for operate - removed side for events * Fix #30665 updates to try to get further with ZAP and autogen, but still fails with some parts of regen_all * Added ember-compatibility-functions.cpp which was missing. * Made all types all lowercase to resolve regen_all issues. * Fixed lint issue (trailing whitespace). * Fixes #30727 - Added initial EVSE cluster and Example Energy Managament app. * Tidied up old comments. * Restyled by whitespace * Restyled by gn * Restyled by prettier-markdown * Added copy of files to all-clusters-app linux BUILD.gn and did basic test with chip-tool * Fixed lint error (Remove PRId64) * Fix for Documentation Build and publish checker. * Updated all-clusters-app.zap after merge and regen_all * Added Cluster to ESP32 CMakeLists.txt * Fixed ESP32 compile error caused by %d * Added missing source files to each build variant * Restyled by gn * Fixed incorrect uint64_t in EnableCharging/EnableDischarging command * Fixed more issues seen on different platforms * Removed unused mEndpointId * Add source files to shell standalone BUILD.gn, More %d fixes for different platforms * Restyled by gn * Removed unused mMinimumChargingCurrentLimitFromCommand * Removed yet more unused variables * Fixed missing semi-colon. How did the other compilers not pick this up? * Capitalise function names * PR comment - Moved PluginServerInitCallback to sdk. Capitalised more function names in energy-management-app. * Restyled by whitespace * Fixes #30805 Updated energy-evse-cluster.xml * Fixes #30805 zap_regen_all commit. * Made Fault Event allow a nullable SessionID * Updates based on review (use kMaximumChargeCurrent instead of duplicate #define). Add HwSetVehicleID implementation * Added RFID Event support. Removed more unnecessary chip:: * Added Feature flags, optional commands and optional attributes. * Made command handling conditional based on features * Added Feature support to all-clusters-app * Restyled by clang-format * Fix to Darwin compile error - not checking strcmp return * Attempt to fix Darwin errors (return after else) * Updated based on latest upstream master * Removed unnecessary mInstance and used 'this' instead. * Regen_all after merge to master. * Fix review comment. * Ensure Init() returns a failure if there is one. Aligned to mode-base-server.cpp * Backed out Read attr check based on features. * Fixed EnumerateAcceptedCommands to handle Loop::Break condition. * Had missed StartDiagnostic as an optional command in InvokeCommand * Removed extra chip:: in attr types. * Updated HwSetVehicleID to copy the value from callee * Fixed potential buffer overrun in HwSetVehicleID. * Fixed simple to address comments raised by Andrei in PR 30857 * Fixed simple to address comments raised by Andrei in PR 30857 * Check Delegate is initialized before calling functions. * Check Delegate is initialized before calling functions. * Added callbacks into Application code * Restyled by whitespace * Ensured that mVehicleID free's any malloc'd CharSpan in destructor * Sync EnergyEvseDelegateImpl.cpp from Example Energy Management * Ensured that mVehicleID free's any malloc'd CharSpan in destructor * Sync EnergyEvseDelegateImpl.cpp from Example Energy Management * Sync'd changes from example energy management app, and commits from #30857 & #30727 * Added namespace to avoid global namespace error in header file. * Re-write of ApplicationInit to handle potential errors * Re-write of ApplicationInit to handle potential errors * Removed unnecessary void in function decl. * Open and saved in ZAP, then regen_all * Updated Energy-management-app.zap / .matter after change to general-diagnostics.xml change to MS. * Updated Energy-management-app.zap / .matter after change to general-diagnostics.xml change to MS. * Restyled by whitespace * Fixed types to be signed=true * Fixed 31032 - revert removal of side="server". Also turned on Events. * PR comment fix - remove Localization Config and Time Format Localization cluster * Removed EVSE commands from ZAP to avoid emberAf linker errors since these are handled in the IM Commands handler * Regen_all to update energy-management.matter file to remove commands that cause linker errors. * Minor changes to align all-clusters and energy-management common. * Apply suggestions from code review Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Addressed comment and replicated into example energy management copies. * Added documentation to EVSE Callbacks as to which struct in the union is used. * Added Energy EVSE and Device Energy Management to config-data.yml under CommandHandlerInterfaceOnlyClusters. Then turned on EVSE commands in ZAP. --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
* Fix for cert test failures * Fix for channel test cases
... by using ConvertECDSASignatureRawToDER which directly integrates with the ASN.1 writer rather than EcdsaRawSignatureToAsn1 from CHIPCryptoPAL. Also make the same change in the chip-cert gen-cd command.
* Introduce a Status + Cluster Code abstraction - Existing code always has to split IM status and cluster-specific status codes, which causes clumsy implementation of clusters (see issue #31120). - This PR introduces a value that encapsulates both the IM status and cluster-specific status, in an easy-to-pass-around abstraction, which can be directly used in place of Status. - Subsequent PR will implement usage in IM with overloads for common cases, such as for AddStatus. Issue #31120 Testing done: - Added unit tests - Other tests still pass * Restyled by clang-format * Restyled by gn * Address review comments * Rename ClusterStatus to ClusterStatusCode * Fix build deps * Restyled by gn --------- Co-authored-by: Restyled.io <commits@restyled.io>
* TC-ACE_1.6: Rework test to use TH keys * Restyled by prettier-yaml * renumber steps, fix one missed key --------- Co-authored-by: Restyled.io <commits@restyled.io>
* Changed check for no nodes in pool * Update src/app/reporting/SynchronizedReportSchedulerImpl.cpp Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> --------- Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
* Renaming mTestNextReportTimestamp since we now use it outside of tests * Updated member description in .h * Update src/app/reporting/SynchronizedReportSchedulerImpl.h Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Restyled by clang-format --------- Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> Co-authored-by: Restyled.io <commits@restyled.io>
* Skip Get RegistrationInfo when param is already added * Remove onICDRegistrationInfoRequired callback in chip-tool * Restyled by clang-format * Add comment * Restyled by whitespace --------- Co-authored-by: Restyled.io <commits@restyled.io>
* Added OTA support for 917 SoC platform * Added SIWX_917 macro for OTAConfig.cpp * Addressed review comments * Remove commented code * Restyled by whitespace * Restyled by clang-format * Restyled by gn * Added fix for NotifyupdateApplied command and TA alone image upgrade reset issue * Modified config file as per slc_1.2 * Restyled by whitespace * Restyled by clang-format * Removed SLC file Specific changes --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: kirankha <kishor.rankhamb@silabs.com>
* Adds fix for wifi apps build failure * Restyled by whitespace --------- Co-authored-by: Restyled.io <commits@restyled.io>
* [app] Fix DeferredAttributePerister memory leak ScopedMemoryBuffer's Release() method was used instead of Free(). Add CHECK_RETURN_VALUE annotation to the Release() method to prevent from making such a mistake in the future. Signed-off-by: Damian Krolik <damian.krolik@nordicsemi.no> * Code review --------- Signed-off-by: Damian Krolik <damian.krolik@nordicsemi.no>
* Update TV&TV casting app per issue 30807 * ZAP Regen Script * Remove unused GetNodeId method Add provisional labels Updated ContentAppObserver to pass test cases
* [icd] read UserActiveModeTrigger during commissioning * Update CommissionerMain.cpp fix typo --------- Co-authored-by: yunhanw-google <yunhanw@google.com>
We were inconsistent about our checks for the "allowed to not have params" case: some places used "command has no fields" as the check, and some used "command has only optional fields". This caused some APIs (for invoking commands with no params) to disappear when commands that used to have no fields got optional fields added to them. The fix: 1. Fixes the checks to consistently be "command has only optional fields" so the no-params APIs continue to exist for the commands that had optional fields added to them. 2. Fixes availability annotations for pre-existing commands with only optional fields so that we will not claim the no-params API is available before it was actually available, by basically listing all such commands in config-data.yaml. 3. Avoids generating deprecated overloads for the thins listed in item 2. 4. Moves the Darwin-only bits from the config-data.yaml in src/app into the Darwin version we are adding, and starts using that config file.
…ation changes (#31104) * [ESP32] Added an API to set the CD in DAC providers * Usage of SetCertificationDeclaration() API in lighting-app/esp32 * [docs] Added some references for production workflows * fix the readme files * Add some docs for the lifetime of data * adjustment to the API documentation
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.