-
Notifications
You must be signed in to change notification settings - Fork 370
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
tinyAVR and megaAVR 0/1/2-series support #463
Comments
I have also looked at this and even written some code. There are at least two other differences: My view is that the peripherals are sufficiently different that it makes sense to create new implementations, rather than trying to rewrite the existing ones. That should also minimise the risk of breakage. What I have so far: The interrupt changes have been used to fix some bugs with level-triggered interrupts on the supported devices, but also remove the interrupt-specific IRQs. They can be put back. These changes are unpublished as there is not enough there to test, other than for regressions. They are also tangled with my other changes. If you are interested it may be possible to pull them out onto a branch based on upstream. Otherwise, I have no plans to revisit before winter in Europe. |
Sorry for the late reply. I made a fork simavrxt to work on this eventually. I looked into peripherals and I agree that it would be best to rewrite them. This does feel like a major undertaking though. I'm just beginning to get familiar with simavr so I expect it will take even more time than I think. In any case, I'll probably won't have much time to look into this before at least November. |
The interrupt and register/memory map changes are now in the "main" branch of my fork. No peripheral code so far. All the tests and examples for existing cores seems to work. |
Thanks for working on it. Personally I gave up register level emulation for my project, I'm doing simulation at a higher level and it works fine. I didn't really have the time to work on simavr, sorry! |
I'm interested in adding support for the new AVR chips, 0/1/2-series tinyAVR and megaAVR, that have been released in the past few years. While overall the cores look a lot like classic AVR cores, there are a few major changes that I'm not sure will currently work with simavr. A few highlights:
LD
/ST
instructions.I won't go into the peripherals details because I don't know much about them, but I believe each peripheral have at least a few differences from classic AVR. More info at: https://github.com/SpenceKonde/megaTinyCore, or just the datasheets.
I don't know much about simavr, but it seems to me like there would be a lot of changes involved. There seems to be relatively few tests so I would be afraid to break something. Could this all be supported without major changes? Should it be supported, or would it be preferable if I forked the project? What do you think?
The text was updated successfully, but these errors were encountered: