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.

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

Upmerge 25-03-2024 #305

Merged
merged 39 commits into from
Apr 30, 2024
Merged

Upmerge 25-03-2024 #305

merged 39 commits into from
Apr 30, 2024

Conversation

gmarull
Copy link
Member

@gmarull gmarull commented Mar 25, 2024

No description provided.

lzala and others added 20 commits January 31, 2024 10:49
Currently imgtool --verify fails for hex files with:

Invalid image magic; is this an MCUboot image?

Added support for hex files by converting hex to bin
using IntelHex::tobinstr().

Reusing image.load() needs a bit of rework, maybe a
common load method will be done in the future,

Signed-off-by: Lucian Zala <zala.lucian@gmail.com>
Fix "align" possible script error, caused by #1833.
Set default value for the align parameter.

Signed-off-by: Andrej Butok <andrey.butok@nxp.com>
Adds default Kconfig configuration that allows to build MCUboot
for NRF54L.
Currently this configuration turns off WDT and FPROTECT,
which is TODO to fix.

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
Signed-off-by: Mateusz Michalek <mateusz.michalek@nordicsemi.no>
In multi image swap validation of images could fail due to
headers being incorrectly re-read from storage.

Fixes #1768

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
Renamed the DK config file to proper name.

Signed-off-by: Andrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no>
The defines have been taken from the Zephyr flash_map.h
but as they are provided there for MCUboot only, they can be just
defined here.

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
Match version with what is now used in upstream Mynewt. Also add
ARM CMSIS which is now taken from original repo.

Signed-off-by: Szymon Janc <szymon.janc@codecoup.pl>
Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.15.4 to 1.16.2.
- [Release notes](https://github.com/sparklemotion/nokogiri/releases)
- [Changelog](https://github.com/sparklemotion/nokogiri/blob/main/CHANGELOG.md)
- [Commits](sparklemotion/nokogiri@v1.15.4...v1.16.2)

---
updated-dependencies:
- dependency-name: nokogiri
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Updates board names as per hwmv2

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Removes configuration for boards that have not been in zephyr
for a long time

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Renames configuration files for board names that have changed
with hwmv2

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Updates board names as per hwmv2

Signed-off-by: David Leach <david.leach@nxp.com>
Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Update FAMILY reference as needed.

Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
Updates a Kconfig that was renamed

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
CONFIG_FPROTECT is defined only in NRF repository, it
should not be added here.

Signed-off-by: Grzegorz Chwierut <grzegorz.chwierut@nordicsemi.no>
Fixes an issue with the estimated size calculation which wrongly
used the maximum align size for some multiplications, this would
mean that in some instances the estimated maximum image size was
smaller than the actual allowed size

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
There have been duplicate functions:
 boot_verify_dependencies
 boot_verify_slot_dependencies
 boot_verify_slot_dependency
with, very similar internals, scattered around unit.
The commit have moved them on top and squashed where possible.

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
MX25UM flash on rd_rw612_bga is very large (8MB), so we must increase
the number of max sectors when targeting this board with MCUboot

Signed-off-by: Daniel DeGrasse <daniel.degrasse@nxp.com>
Fixes an issue whereby the instruction and data caches being
disabled before booting code had bit-rotted and no longer worked,
adds a new Kconfig that allows this option to be turned off if
wanted.

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Adds a note on the fix for zephyr cache support

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Copy link
Contributor

@de-nordic de-nordic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zero diff with ncs/main, which is OK.
Tested with smp_svr sample on nrf52840dk_nrf52840, works fine
Tested serial recovery on nrf52840dk_nrf5240, seems to work ok.

This reverts commit 622b84c.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
gmarull and others added 18 commits April 30, 2024 16:39
…lot"

This reverts commit 3b0eac8.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
This reverts commit d3f55fe.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
…_sysflash.h"

This reverts commit f013888.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
This reverts commit 22ed9fb.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
This reverts commit f577000.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
This reverts commit 5cf8423.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
…ate multi-image"

This reverts commit f82a032.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
…fig"

This reverts commit 6a9b75f.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
This reverts commit 27fc8d7.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
This reverts commit 4d7e568.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
Regular upmerge.

Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
Seems multi-image dependencies are not supported for multi-image in
NCS yet. This is a workaround which reverts some lines to restore
previous MCUboot behavior, so that Immutable bootloader + MCUBoot type
builds will work.

Ref. NCSDK-8681

Signed-off-by: Sigvart Hovland <sigvart.hovland@nordicsemi.no>
Signed-off-by: Torsten Rasmussen <Torsten.Rasmussen@nordicsemi.no>
Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
(cherry picked from commit e02925a)
Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
Fixes a missing PCD define check, an image might have the
network core partition layout set but if PCD support is not
enabled then it should not assume that PCD support is part
of mcuboot.

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
(cherry picked from commit 5932630)
(cherry picked from commit 6024d0a)
Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
This adds support for using both NSIB and the multi-image configuration
in MCUboot. Before this was not possible due to upgradable bootloader
support through NSIB was using the `UPDATEABLE_IMAGE_NUMBER`
configuration to update the updateable bootloader.

In this commit we change from using `FLASH_AREA_IMAGE_PRIMARY` to get
the flash area ID to using the bootloader state where we set the flash
area ID of the free updatable bootloader slot if the image is intended
for this slot.

Ref. NCSDK-19223
Ref. NCSDK-23305

Signed-off-by: Sigvart Hovland <sigvart.hovland@nordicsemi.no>
Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
(cherry picked from commit 03af90f)
(cherry picked from commit 0d43674)
(cherry picked from commit b4464ad)
Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
Making sysflash.h and pm_sysflash.h more readable.

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
(cherry picked from commit c3b33eb)
(cherry picked from commit 418b6fe)
Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
The commit modifies pm_sysflash.h to add support for three
application images.

Ref. NCSDK-19223

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
Signed-off-by: Sigvart Hovland <sigvart.hovland@nordicsemi.no>
(cherry picked from commit 77c7e25)
(cherry picked from commit d700515)
(cherry picked from commit 7b87633)
Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
Added procedure which clean-up content of all the secondary slot
which contains valid header but couldn't be assigned to any of
supported primary images.
This behavior is needed when configuration allows to use one secondary
slot for collecting image for multiple primary slots.

Signed-off-by: Andrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no>
(cherry picked from commit ed3a0c2)
Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
FPROTECT is not suppored yet for nrf54l15.

Signed-off-by: Grzegorz Chwierut <grzegorz.chwierut@nordicsemi.no>
Signed-off-by: Gerard Marull-Paretas <gerard@teslabs.com>
@gmarull gmarull merged commit 0b5810d into nrfconnect:main Apr 30, 2024
1 check passed
@nvlsianpu
Copy link
Contributor

Why #308 was reverted? @gmarull

@gmarull
Copy link
Member Author

gmarull commented May 6, 2024

Why #308 was reverted? @gmarull

maybe a mistake

@gmarull gmarull deleted the upmerge-25032024 branch May 6, 2024 09:35
@nvlsianpu
Copy link
Contributor

reapplying in #311

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.