-
Notifications
You must be signed in to change notification settings - Fork 3k
Fix flash_program_page API in LPC boards. #7128
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.
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree. This operation is redundant as a check is present in the driver to ensure address is page aligned.
/morph build |
Build : SUCCESSBuild number : 2320 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 1942 |
Test : FAILUREBuild number : 2098 |
This API allocates a program buffer of 256 on the stack to ensure alignment. However, FlashIAP driver already ensures this alignment of the user data.
9950a2f
to
9e5efbc
Compare
@davidsaada The latest commit fixes the test failure (helps if you comment on any change in the PR to provide actual status) |
(@0xc0170 was just about to - you beat me up to it...) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to confirm - the headline is for LPC fix but I can see functional test case removal - is this because test is actually always passed or ?
It is because FlashIAP makes sure that user data is aligned, no need to take care of it in HAL layer (which is what the test checked). LPC allocated a lot of stack for this purpose (keeping user data aligned at HAL layer), which is exactly what this PR fixed. |
/morph build |
Build : SUCCESSBuild number : 2352 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 1976 |
Test : SUCCESSBuild number : 2138 |
Description
This PR fixes the flash_program_page API in LPC boards.
This API allocates a program buffer of 256 on the stack to ensure alignment.
However, FlashIAP driver already ensures this alignment of the user data, so this allocation is redundant.
Pull request type