Skip to content

Conversation

@kkasperczyk-no
Copy link
Owner

Currently in all Matter samples and applications main loop was
refreshed every 10 ms to update LEDs state and thus resulted
in frequent MCU wake ups. Introduced several changes in order
to wake up MCU only on events or when LED needs a state change:

  • Moved updating BLE connection and Thread state from main loop
    to Matter event handler
  • Moved updating LED state from main loop to event handler
  • Refactored LEDWidget module to use timer and change LEDs state
    on timer expiration.
  • Modified Matter weather station events handling to be consistent
    with other Matter samples

Signed-off-by: Kamil Kasperczyk [email protected]

Copy link
Collaborator

@Damian-Nordic Damian-Nordic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have some code-style comments, but looking good.

Currently in all Matter samples and applications main loop was
refreshed every 10 ms to update LEDs state and thus resulted
in frequent MCU wake ups. Introduced several changes in order
to wake up MCU only on events or when LED needs a state change:
* Moved updating BLE connection and Thread state from main loop
to Matter event handler
* Moved updating LED state from main loop to event handler
* Refactored LEDWidget module to use timer and change LEDs state
on timer expiration.
* Modified Matter weather station events handling to be consistent
with other Matter samples

Signed-off-by: Kamil Kasperczyk <[email protected]>
kkasperczyk-no pushed a commit that referenced this pull request May 13, 2024
-This manifest update and relevant commits in sdk-nrf is made to add
 support for nRF54H20 devices without a local build of the core
-This includes taking in sdk-mbedtls pull request #15 for better
 client support in PK and MD wrapper APIs
-This includes taking in sdk-oberon-psa-crypto pull request #10 for
 better client support as well as fixups for psa_util used for
 TLS/DTLS and X.509 testing

Ref: NCSDK-15632

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants