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

doc: readme-zephyr: fix scratch partition #1951

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

butok
Copy link
Contributor

@butok butok commented Apr 30, 2024

It tells about adding the 24KB scratch partition,
not about storage partition.

@butok
Copy link
Contributor Author

butok commented May 2, 2024

Hi @nordicjm,
the CI fail is not caused by this commit.
Could you fix the CI?

@nordicjm
Copy link
Collaborator

nordicjm commented May 2, 2024

@sylvioalves you have a very broken module, please fix

@sylvioalves
Copy link
Contributor

Fixed by #72186. Please rebase.

@butok
Copy link
Contributor Author

butok commented May 2, 2024

Fixed by #72186. Please rebase.

This PR is already based on the latest mcuboot commit d4394c2
What rebase do you mean?

@sylvioalves
Copy link
Contributor

sylvioalves commented May 2, 2024

@nordicjm, does re-triggering this PR pipeline fetch latest zephyr main?

It tells about adding the 24KB scratch partition,
not about storage partition.
Updates the used path to HWMv2.

Signed-off-by: Andrej Butok <andrey.butok@nxp.com>
@butok
Copy link
Contributor Author

butok commented May 6, 2024

@nordicjm, does re-triggering this PR pipeline fetch latest zephyr main?

So amend & push has helped.

@butok
Copy link
Contributor Author

butok commented May 16, 2024

UP

1 similar comment
@butok
Copy link
Contributor Author

butok commented Jul 23, 2024

UP

@nordicjm nordicjm requested a review from de-nordic July 23, 2024 11:07
@de-nordic de-nordic merged commit 50f3514 into mcu-tools:main Jul 24, 2024
55 checks passed
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