Skip to content

Conversation

@israpps
Copy link
Contributor

@israpps israpps commented May 20, 2025

This variation of MCMAN offers the same funcionality that game bundled DONGLEMAN drivers.

It can freely access both mc0:/ and mc1:/ on arcade PS2s.

Also, the driver seems to work well with DAEMON.IRX based on some quick tests

@israpps
Copy link
Contributor Author

israpps commented Jun 7, 2025

Not sure why it now says int SecrAuthDongle(int, int, int); is implicitly declared... but I've locally compiled and tested this IRX and works as expected. It can be used to access mc0 and mc1 freely on arcade PS2s....

However, it's highly recommended to still avoid accessing SoulCalibur2 Conquest card with anything that is not the DONGLEMAN module from SC2

@israpps israpps marked this pull request as ready for review June 7, 2025 04:12
@israpps
Copy link
Contributor Author

israpps commented Jun 7, 2025

@uyjulian @rickgaiser

Do you guys have any idea why it complains of implicit decl of SecrAuthDongle but locally builds perfectly?

@uyjulian
Copy link
Member

uyjulian commented Jun 7, 2025

I think you want to default MCMAN_BUILDING_DONGLEMAN to 0 in iop/memorycard/mcman/Makefile

@israpps
Copy link
Contributor Author

israpps commented Jun 7, 2025

Yes, I realized while talking with @ ziemas

I will fix when I'm back at the PC to keep it in one commit

@uyjulian
Copy link
Member

uyjulian commented Jun 9, 2025

lgtm

@uyjulian uyjulian merged commit ef3e895 into ps2dev:master Jun 9, 2025
3 checks passed
} while (++retries < 100);

if (p[3] == 0x66)
HAKAMA_SIGNALSEMA(); //El_isra: Original impl has two semas (still commented there) replaced with one for simplicity
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you can't signal before as it could potentially change the value of p before you read in the following "ifs"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, you're right

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.

3 participants