Pad all but last sector in flash binary to write whole of sector #800

Merged
kilograham merged 2 commits from sparse_uf2 into develop 2022-05-10 13:03:44 -05:00
kilograham commented 2022-05-03 13:44:15 -05:00 (Migrated from github.com)

Additional changes on top of #763

Additional changes on top of #763
lurch (Migrated from github.com) reviewed 2022-05-04 04:27:10 -05:00
@ -296,0 +315,4 @@
// Create a dummy page, if it does not exist yet. note that all present pages are first
// zeroed before they are filled with any contents, so a dummy page will be all zeros.
auto &dummy = pages[page];
}
lurch (Migrated from github.com) commented 2022-05-04 04:27:10 -05:00

For this last sector, it makes sense that we don't want/need to add any zero-padding after the data in that sector; but do any holes before the data in the last sector still need to be filled with zeroes?

For this last sector, it makes sense that we don't want/need to add any zero-padding _after_ the data in that sector; but do any holes _before_ the data in the last sector still need to be filled with zeroes?
kilograham (Migrated from github.com) reviewed 2022-05-04 11:20:49 -05:00
@ -296,0 +315,4 @@
// Create a dummy page, if it does not exist yet. note that all present pages are first
// zeroed before they are filled with any contents, so a dummy page will be all zeros.
auto &dummy = pages[page];
}
kilograham (Migrated from github.com) commented 2022-05-04 11:20:49 -05:00

good point :-)

good point :-)
kilograham (Migrated from github.com) reviewed 2022-05-04 12:59:01 -05:00
@ -296,0 +315,4 @@
// Create a dummy page, if it does not exist yet. note that all present pages are first
// zeroed before they are filled with any contents, so a dummy page will be all zeros.
auto &dummy = pages[page];
}
kilograham (Migrated from github.com) commented 2022-05-04 12:59:01 -05:00

actually it doesn't matter from a correctness point of view (the actual data of the last sector will be written correctly), however as it stands a gap in the last sector will not be filled with zeroes.

actually it doesn't matter from a correctness point of view (the actual data of the last sector will be written correctly), however as it stands a gap in the last sector will not be filled with zeroes.
kilograham commented 2022-05-09 15:11:28 -05:00 (Migrated from github.com)

fixes #762

fixes #762
Wren6991 (Migrated from github.com) approved these changes 2022-05-10 04:38:04 -05:00
Sign in to join this conversation.
No reviewers
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
gh-raspberrypi/pico-sdk!800
No description provided.