Skip to content

[ZELDA] GB games resets retro-go config (and sometimes BSOD) #59

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

Closed
DSkywalk opened this issue Oct 9, 2023 · 6 comments
Closed

[ZELDA] GB games resets retro-go config (and sometimes BSOD) #59

DSkywalk opened this issue Oct 9, 2023 · 6 comments

Comments

@DSkywalk
Copy link

DSkywalk commented Oct 9, 2023

Hello @BrianPugh, I'm experimenting to take a little more advantage of the stock zelda, I hope you still have the disassembly at hand 👯

I have seen that if you run GB zeldas several times, changing the language, running and resetting the game. It deletes a code area supposedly available. We should not allow to run the gameboy emulator, to avoid problems if --no-la is used.

Sometimes its corrupt the retro-go config and when you load any rom... BSODs 💥

imagen

If you tell me some NOP to avoid it I'll upload it as PR.

Thanks for your great work 👍

@BrianPugh
Copy link
Owner

I'm not sure if I have the ghidra files still laying around; I'd also be a little hesitant to share them (to be sure that no proprietary code leaks).

That said, a lot of my notes (and commented out code of stuff i was working on) is in the zelda file.

As for fixing the presented issue, I'm open to PRs. I might be able to get around to it at some point, but unfortunately it's not very high on my priority list (I've got a lot of projects/work going on atm).

@DSkywalk
Copy link
Author

DSkywalk commented Oct 10, 2023

I wasn't commenting to share them, but just in case you could look at the menu area and prevent the 3rd option of the zelda menu from being able to be executed adding some NOP. And then I can prepare a patch for python as a PR 👍

@BenjaminSoelberg
Copy link
Contributor

I'll give it a go.

@BenjaminSoelberg
Copy link
Contributor

BenjaminSoelberg commented Nov 22, 2023

I have a solution, and I'll push a PR soon. @DSkywalk would you be so kind and do you have the time to help me test it ?
Please note that my changes merely disables the loading of LA, and I have not been able to reproduce the RG settings corruptions. I hope that you will be able to verify that.

BenjaminSoelberg added a commit to BenjaminSoelberg/game-and-watch-patch that referenced this issue Nov 22, 2023
…g (and sometimes BSOD)

It does so by ignoring any LA menu selection if --no-la was given.
BenjaminSoelberg added a commit to BenjaminSoelberg/game-and-watch-patch that referenced this issue Nov 23, 2023
…g (and sometimes BSOD)

It does so by ignoring any LA menu selection if --no-la was given.
@BrianPugh
Copy link
Owner

thanks @BenjaminSoelberg for the PR! Love it!

Closing this issue.

@DSkywalk
Copy link
Author

really thanks @BenjaminSoelberg ❤️

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

No branches or pull requests

3 participants