-
Notifications
You must be signed in to change notification settings - Fork 13
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
libnds - Fix swiSoftReset for DSi #254
Comments
Is this even fixable? That swi is just not available on DSi according to GBATEK: https://problemkaputt.de/gbatek.htm#biosresetfunctions Compare it to https://problemkaputt.de/gbatek.htm#biosmemorycopy with DSi7 and DSi9 in the list of supported platforms. |
The fix would be using this https://github.com/blocksds/libnds/blob/master/source/common/swiSoftReset.s when a DSi is detected. |
Funnily enough, this used to be correct. It was broken during a patch to optimize swi calls: blocksds/libnds@23a669f I've created this PR to revert the change to On the other hand, this explains #10. Note that exit to loader still works on NDS as usual. |
The main issue I see is that the libnds implementation of
|
Making the issue here, since libnds has issues blocked.
Hope it's fine!
Right now, swiSoftReset calls the corresponding swi. While this is fine on DSs, on DSis this results in hanging.
This function is also called from other pieces of code, like ResetARM7. So it's not just about telling the user not to use it on DSis.
The text was updated successfully, but these errors were encountered: