Skip to content

Conversation

arun-silabs
Copy link
Contributor

@arun-silabs arun-silabs commented Sep 30, 2025

Problem / Feature

This PR upstreams the clock-setting fixes for 917SoC from release_2.7-1.4 branch to main.

Change overview

After integrating the sl_main component into the Matter application, we observed a timing discrepancy in the device clock. Specifically, the clock was lagging by approximately 2%—for every 100 seconds of real time, the device clock advanced only 98 seconds. This deviation resulted in failure of time-sensitive operation. The deviation is seen as the tick rate is not being used while setting the systick configuration.

To fix this drift, the tick rate macro is leveraged while setting the systick configuration.

Below commits from release_2.7-1.4 branch are being cherry-picked :
#158
#165

Testing

Tested the timing related test-cases (like cadmin test-cases, where a commissioning window would be opened only for a particular amount of time) with lighting-app and lock-app and ensured that the device clock is functioning without any errors.

@arun-silabs arun-silabs requested a review from a team September 30, 2025 09:23
@Sarthak-Shaha
Copy link
Contributor

Should this be done on CSA master ?

@mkardous-silabs
Copy link
Contributor

@arun-silabs can the PR title be updated to reflect the changes being cherry-picked?

@arun-silabs arun-silabs changed the title [Cherry-Pick] Set the systick value based upon the configured clock rate [Cherry-Pick] Use configTICKRATE_HZ macro to set the systick configuration Oct 6, 2025
@arun-silabs
Copy link
Contributor Author

@arun-silabs can the PR title be updated to reflect the changes being cherry-picked?

Updated the PR title

@arun-silabs
Copy link
Contributor Author

Should this be done on CSA master ?

Yes, this change should be on CSA master.

Copy link
Contributor

@mkardous-silabs mkardous-silabs left a comment

Choose a reason for hiding this comment

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

PR should be on CSA and not matter_sdk.

@mkardous-silabs mkardous-silabs merged commit ecf8f0d into main Oct 6, 2025
2 checks passed
@mkardous-silabs mkardous-silabs deleted the clock_setting_fix branch October 6, 2025 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants