Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Backport v3.3-branch] drivers: flash: spi_nor: select largest valid erase operation #61153

Merged
merged 1 commit into from
Sep 5, 2023
Merged

[Backport v3.3-branch] drivers: flash: spi_nor: select largest valid erase operation #61153

merged 1 commit into from
Sep 5, 2023

Conversation

zephyrbot
Copy link
Collaborator

@zephyrbot zephyrbot commented Aug 4, 2023 •
edited by fabiobaltieri

Backport ea2dd9f from #60943.

Fixes #60904

@Crzyrndm @github-actions
drivers: flash: spi_nor: select largest valid erase operation
011abb4 
The spi_nor erase op selection was based on the alignment of the end of
the region to be erased. This prevented larger erase operations being
selected in many cases

Closes #60904

Signed-off-by: Joshua Crawford <joshua.crawford@levno.com>
(cherry picked from commit ea2dd9f)
@zephyrbot zephyrbot added Backport Backport PR and backport failure issues area: Flash labels Aug 4, 2023
@fabiobaltieri fabiobaltieri merged commit a042ba2 into v3.3-branch Sep 5, 2023
24 of 25 checks passed
@nashif nashif deleted the backport-60943-to-v3.3-branch branch February 13, 2024 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Laczen Laczen Laczen approved these changes

@de-nordic de-nordic de-nordic approved these changes

@nashif nashif Awaiting requested review from nashif nashif is a code owner

@jgl-meta jgl-meta Awaiting requested review from jgl-meta

Assignees

@de-nordic de-nordic

Labels
area: Flash Backport Backport PR and backport failure issues
Projects
Backports
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@zephyrbot @Laczen @de-nordic @fabiobaltieri @Crzyrndm