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

Update SMC bootloader tools #56

Conversation

stople
Copy link
Collaborator

@stople stople commented Oct 13, 2024

This PR updates the SMC bootloader tools and information. This is mostly to include changes made in the zip version added here #53 (comment) , but, the instructions are also reorganized and cleaned up a bit.

Main changes are that these tools are better together with bootloader v3, and, they add a warning if the user attempts to e.g. install bootloader v2 if bootloader v3 is installed, causing a significant risk of bricking the x16.

Also, I took the liberty to add the pre-built bootloader .BIN files from existing bootloaders to this repo, causing that files are in one place, and that there are fewer risks involved with e.g. renaming the files.

@stople stople marked this pull request as draft October 13, 2024 17:54
@stople stople changed the title Update SMC bootloader tools (instructions) Update SMC bootloader tools Oct 13, 2024
@stople stople force-pushed the update-smc-bootloader-tools-and-instructions branch from 73a5ed5 to 788aadf Compare October 13, 2024 19:00
@stople stople force-pushed the update-smc-bootloader-tools-and-instructions branch from 788aadf to 8ee8e8b Compare October 13, 2024 19:03
@stople stople marked this pull request as ready for review October 13, 2024 21:30
@stople
Copy link
Collaborator Author

stople commented Oct 13, 2024

@stefan-b-jakobsson ready for review. I did a quick test on my machine, seems like the tools still works.

@stefan-b-jakobsson
Copy link
Collaborator

There's a dead link to the Flash manipulation tools in doc/update-guide.md under heading "In-system update of the bootloader".

@stefan-b-jakobsson
Copy link
Collaborator

I tested the new flash tools. They work fine as far as I can tell. I also tested to do an SMC update after updating the bootloader.

One small thing I think I remember from some discussion. The dump tool prints the command you can use to save the bootloader to disk. If I remember correctly, the end address should be the actual end address + 1.

@stople stople marked this pull request as draft October 28, 2024 22:12
@stople stople marked this pull request as ready for review October 28, 2024 22:45
@stople stople added enhancement New feature or request and removed enhancement New feature or request labels Oct 28, 2024
@stople stople marked this pull request as draft October 28, 2024 22:58
@stople stople marked this pull request as ready for review October 28, 2024 23:05
@stople
Copy link
Collaborator Author

stople commented Oct 28, 2024

There's a dead link to the Flash manipulation tools in doc/update-guide.md under heading "In-system update of the bootloader".

Well spotted. Fixed.

@stople
Copy link
Collaborator Author

stople commented Oct 28, 2024

I tested the new flash tools. They work fine as far as I can tell. I also tested to do an SMC update after updating the bootloader.

One small thing I think I remember from some discussion. The dump tool prints the command you can use to save the bootloader to disk. If I remember correctly, the end address should be the actual end address + 1.

I have updated the BSAVE commands to include the last byte

@stefan-b-jakobsson
Copy link
Collaborator

I think there's still a dead link in update-guide.md, at line 124. It refers to the file flash-manipulation.md that has been renamed.

@stople
Copy link
Collaborator Author

stople commented Oct 29, 2024

Well spotted, again. Fixed.

@stefan-b-jakobsson stefan-b-jakobsson merged commit 6541fc0 into X16Community:main Oct 30, 2024
1 check 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.

2 participants