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

Fix serial consoles and images #152

Merged
merged 3 commits into from
Jan 25, 2024
Merged

Conversation

jonmason
Copy link
Contributor

The current code no longer boots and serial consoles do not work (since nanbield and mickledore respectively). Fix those and compress the wic image for developer ease-of-use.

In upstream commit 9d053af1fb570b4e3483de4ecd6827e1e0be61b7
(mickledore), the SERIAL_CONSOLE variable was removed.  This prevented
the serial consoles from working.  Change to get it working again.

Signed-off-by: Jon Mason <jdmason@kudzu.us>
Upstream commit 26d97acc71379ab6702fa54a23b6542a3f51779c (nanbield)
contains a change which breaks linking of images without
IMAGE_NAME_SUFFIX set.  Add an empty string to get this working again.

Signed-off-by: Jon Mason <jdmason@kudzu.us>
bmaptool can handle compression.  Let's compress the wic file to save
space in the build tree.

Signed-off-by: Jon Mason <jdmason@kudzu.us>
Copy link
Owner

@superna9999 superna9999 left a comment

Choose a reason for hiding this comment

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

Thanks for the fixes!

@jonmason
Copy link
Contributor Author

I should probably add this was tested on a AML-S905X-CC board and verified to work (but only tested on that platform). Also, please cherry pick the relevant changes back to the mickledore and nanbield branches as those don't work without them.

@superna9999
Copy link
Owner

Yep, I wanted to wait for CI to finish, but it keeps failing

@superna9999 superna9999 merged commit 0d73fd3 into superna9999:master Jan 25, 2024
1 check failed
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.

2 participants