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

AVR109: support for prodsig read and usersig read/write #1108

Merged
merged 2 commits into from
Oct 5, 2022

Conversation

MCUdude
Copy link
Collaborator

@MCUdude MCUdude commented Oct 2, 2022

This PR allows the AVR109 based Xboot bootloader to read prodsig and read/write to the usersig memory on any Xmega that Xboot supports. Tested on an ATxmega256A3BU.

This is an extension of the original AVR109 protocol, and I've not added any guards to prevent users from reading/writing to thee memories. Mostly because (I think) it's unlikely that Xmega users are running a different avr109 bootloader implementation.

Closes #200

@mcuee mcuee added the enhancement New feature or request label Oct 3, 2022
@stefanrueger
Copy link
Collaborator

Looks cool, if it works (presumably, you tried?) should be merged.

@MCUdude
Copy link
Collaborator Author

MCUdude commented Oct 3, 2022

Looks cool, if it works (presumably, you tried?) should be merged.

Thanks. I've tested it and can confirm that it works perfectly. It can perhaps be merged together with #1101 and #1102

@MCUdude
Copy link
Collaborator Author

MCUdude commented Oct 3, 2022

@mcuee it would be great if you could try this PR on your Xmega hardware as well before merging, just to be sure

@mcuee
Copy link
Collaborator

mcuee commented Oct 3, 2022

@mcuee it would be great if you could try this PR on your Xmega hardware as well before merging, just to be sure

Sorry this is a bit tricky for me to carry out quick testing as my xmega breakout board is really just a breakout board. Please proceed with the merge. I will try to test this later.

@MCUdude
Copy link
Collaborator Author

MCUdude commented Oct 3, 2022

Sorry this is a bit tricky for me to carry out quick testing as my xmega breakout board is really just a breakout board. Please proceed with the merge. I will try to test this later.

I think it's ready to be merged. I'll let Stefan do it together with the rest of the PRs that are also ready.
On the board I used for testing, an Xplained ATxmega256A3BU, I had to wire a USB to serial adapter directly to the UART0 pins and compile the code to match the board layout (start bootloader with SW0 switch, etc.)

@mcuee
Copy link
Collaborator

mcuee commented Oct 3, 2022

Please go ahead with the merge and not wait for my testting. Thanks.

@stefanrueger stefanrueger merged commit f2fb3b4 into avrdudes:main Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug #29896] Add support for XMEGA specific AVR109 protocol extensions used by XBoot
3 participants