Skip to content

boot: zephyr: decompression: reflect pad value - #704

Merged
nvlsianpu merged 1 commit into
nrfconnect:mainfrom
michalek-no:fix_zero_pad
Jul 30, 2026
Merged

boot: zephyr: decompression: reflect pad value#704
nvlsianpu merged 1 commit into
nrfconnect:mainfrom
michalek-no:fix_zero_pad

Conversation

@michalek-no

Copy link
Copy Markdown
Contributor

No description provided.

@michalek-no
michalek-no requested a review from a team as a code owner July 7, 2026 11:35
@michalek-no michalek-no added the DNM label Jul 7, 2026
NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Jul 7, 2026
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-mcuboot#704

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Jul 7, 2026
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-mcuboot#704

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
@michalek-no michalek-no changed the title Fix zero pad boot: zephyr: decompression: reflect pad value Jul 8, 2026
@michalek-no
michalek-no requested a review from tomchy July 8, 2026 07:19
@de-nordic

Copy link
Copy Markdown
Contributor

I am not sure this is proper solution. Looks like a hack where we are trying to fill in information that has not correctly been applied to a header by copying from other header.
The proper way should be to parametrize imgtool with pad value. We are giving header size to imgtool anyway, so we, as well, request to fill it with value we want.

move 0x00 or 0xFF from secondary slot
during decompression

Signed-off-by: Mateusz Michalek <mateusz.michalek@nordicsemi.no>
NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Jul 28, 2026
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-mcuboot#704

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
@sonarqubecloud

Copy link
Copy Markdown

@michalek-no michalek-no removed the DNM label Jul 30, 2026
@nvlsianpu
nvlsianpu merged commit e2f01b8 into nrfconnect:main Jul 30, 2026
4 checks passed
NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Jul 30, 2026
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-mcuboot#704

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
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