Sync csa branch with main #282
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
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
…s as defunct (#37547)
…… (#37533) * For thread devices throttle the response to BlockQuery by an interval specified in kBdxThrottleIntervalInMsecs so that we don't overload the network with frequent BDX messages * Restyle * Apply suggestions from code review Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Add user defaults support to allow the BDX throttle interval for thread device to be configurable - Address review comments --------- Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
Log the state of a controller that gets notified about browse results (so we can tell whether it's suspended or not, for example).
Throttle by rounding up to the nearest multiple of our throttle interval, not by just comparing to that interval. This matches the old "poll at 50ms" behavior we had.
* Unify D-Bus interface include files naming * Update code after D-Bus namespace change * Optimize OpenThread D-Bus API stub by removing non-used props This optimization saves: .text -> 16980 bytes .data -> 4720 bytes * Simplify namespace for wpa_supplicant D-Bus codegen API * Optimize D-Bus API stub for wpa_supplicant This optimization saves: .text -> 69969 bytes .data -> 16384 bytes * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>
This is an initial cut from the current Device Library(excluding Scenes Mgmt). A future addendum might include slight modifications to the composition.
This makes it easier to edit platform code from within Xcode
|
The check for |
Add explicit CloseSession calls in TC_CGEN_2_8 and TC_CGEN_2_9 to ensure proper cleanup of commissioner library resources. This fixes an issue where subsequent commissioner sessions could not be established due to lingering resources from previous test runs. Bug: Commissioner session establishment failures in subsequent test runs Fix: Add explicit session cleanup after commissioning operations Testing: - Set PICS_USER_PROMPT=1 for manual intervention testing - Verified with terms-and-conditions-app test harness - Confirmed proper cleanup by: 1. Running initial commissioning 2. Performing factory reset (process kill + KVS cleanup) 3. Successfully re-establishing commissioner session - Test passes with manual intervention at factory reset and commissioning prompts
Just log whether we have one at all.
mkardous-silabs
approved these changes
Feb 14, 2025
* Adding missing ExitContainer calls and adding GetLength Checks * Adding Symbolic TLV Tags * Integrating Comments
This fixes some copy & paste typos from temporary development of size scripts. Also fixes the display of images for the tool examples (the gh pages upload did not have the corresponding png). Co-authored-by: Andrei Litvin <andreilitvin@google.com>
* PICS documentation: Initial draft NOTE: some sections have not yet been filled, and will be addressed after the 1.4.1 SVE. * spelling * add some fixes and clarifications based on feedback from Yinyi * spelling
* Implement WifiSleepManager * Add missing nogncheck to icd configuration include * Restyled by clang-format * Apply suggestions from code review Co-authored-by: Junior Martinez <67972863+jmartinez-silabs@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Ricardo Casallas <77841255+rcasallas-silabs@users.noreply.github.com> --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Junior Martinez <67972863+jmartinez-silabs@users.noreply.github.com> Co-authored-by: Ricardo Casallas <77841255+rcasallas-silabs@users.noreply.github.com>
* Make coverage seemingly work: set per test output raw profile, add scripting logic * Adding scripts and fixes * Restyle * use llvm cov to merge things * Attempt for better coverage merging. -object seems to do the trick * make lcov work on my machine as well * Restyled by gn --------- Co-authored-by: Restyled.io <commits@restyled.io>
jmartinez-silabs
approved these changes
Feb 14, 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.
This PR syncs the csa branch with the main branch.
PR MUST BE MERGED WITH MERGE COMMIT - ADMIN MUST ENABLE THE OPTION