Skip to content

Conversation

rghaddab
Copy link
Contributor

For devices that need an erase before a write, ZMS do not verify that the next available location in the open sector is filled with the erase_value.
Fix this by adding a check at init.

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
E Reliability Rating on New Code (required ≥ C)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Aug 13, 2025
@rghaddab
Copy link
Contributor Author

will be updated very soone, removing stale label

@github-actions github-actions bot removed the Stale label Aug 15, 2025
@tpambor
Copy link
Contributor

tpambor commented Sep 5, 2025

@rghaddab Any update on this one?

@rghaddab
Copy link
Contributor Author

@tpambor I will take care of this next week

@rghaddab rghaddab force-pushed the rghaddab/zms-handle-faulty-bit-injection branch 2 times, most recently from 55bc5ac to 9b8b94e Compare September 25, 2025 09:33
@rghaddab rghaddab marked this pull request as ready for review September 25, 2025 09:33
@rghaddab rghaddab requested a review from tpambor September 25, 2025 09:34
@Laczen Laczen removed their request for review September 25, 2025 09:38
For devices that need an erase before a write, ZMS do not verify that
the next available location in the open sector is filled with the
erase_value.
Fix this by adding a check at init.

Signed-off-by: Riadh Ghaddab <[email protected]>
@rghaddab rghaddab force-pushed the rghaddab/zms-handle-faulty-bit-injection branch from 9b8b94e to 417ecd6 Compare September 25, 2025 10:04
Copy link

@rghaddab
Copy link
Contributor Author

@tpambor I just noticed that this is an old commit and I already fixed the sonarqubecloud issue.
Can you give your approval if this fixes your issue ?

@cfriedt cfriedt merged commit a5f0c96 into zephyrproject-rtos:main Oct 18, 2025
26 checks passed
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.

5 participants