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

Make IOP Reset configurable #93

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Make IOP Reset configurable #93

wants to merge 2 commits into from

Conversation

israpps
Copy link
Owner

@israpps israpps commented Feb 18, 2025

Under startup settings, a new configuration named Reboot IOP when loading ELF appears

This configuration will influence a new additional parameter passed to wLaunchELF ELF loader subprogram.

Based on the value of the config, the elf loader will reboot (or not) the PS1 CPU after SifLoadElf successful execution, and before the actual execution of the elf file (ExecPS2())

Additional changes:

  • Config has been simplified on the screen draw routine by using ternary operators to eliminate additional usage of sprintf().

  • ELF loader subprogram will print to the EE UART the following events:

    • IOP reboot issued
    • SifLoadElf failure
    • if code execution continued after ExecPS2() (maybe an err?)

    I know EE SIO is not available to everyone, but it may help with quick troubleshooting on PCSX2 or with EE SIO prepared console

@israpps
Copy link
Owner Author

israpps commented Feb 18, 2025

WLE-ETH=1-MMCE=0-EXFAT=0-DS34=0- (1).zip
Summoning @NathanNeurotic for testing.

It should work as is, but I just wanna b sure

@NathanNeurotic
Copy link

I just saw this - do you still need test?

@israpps
Copy link
Owner Author

israpps commented Feb 28, 2025

I just saw this - do you still need test?

Yeah, this is related to the issue you reported back then. When I made IOP reset always happen on ELF loading

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