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

EE SIO Support #164

Closed
wants to merge 3 commits into from
Closed

EE SIO Support #164

wants to merge 3 commits into from

Conversation

israpps
Copy link
Contributor

@israpps israpps commented Nov 10, 2023

Allows logging into the EE UART

h4570 and others added 3 commits September 4, 2022 15:25
* create x86 dockerfile

* remove tamtypes.h dependency

* fix copyright

* warning fix

* move code to shared/

* shared and ps2 compilation

* add run-tests action

* tests check

* tests check

* tests check

* tests check

* tests check

* tests check

* tests check

* tests check

* tests check

* tests check

* tests check

* tests check

* tests check

* tests check

* codacy fixes

* fix codacy error
An error happened if either IRX ID is less than 0 (error reported by MODLOAD itself, eg: missing IRX dependency), or if return value is 1 (aka: module returned NO_RESIDENT_END, aka: module willingly asked to be removed from RAM )
Enhanced IRX loader conditions
@Wolf3s
Copy link
Contributor

Wolf3s commented Nov 10, 2023

isra clang-format the code please.

@h4570
Copy link
Owner

h4570 commented Nov 11, 2023

Pretty cool feature.
Thank you for contribution.

Did you tested it already?
Proofs are appreciated.

@israpps
Copy link
Contributor Author

israpps commented Nov 11, 2023

isra clang-format the code please.

Not at the PC until tomorrow night.


Pretty cool feature. Thank you for contribution.

Did you tested it already? Proofs are appreciated.

Haven't tested it yet, but I made the siocookie library, and I've used EESIO before, so it should work...
But I'll try as soon as I can.

It can also be tested on emulator, EE SIO printf is printed on blue text

@h4570
Copy link
Owner

h4570 commented Nov 13, 2023

Please merge latest master to this branch.
Thanks!

@Wolf3s
Copy link
Contributor

Wolf3s commented Nov 14, 2023

@israpps why close this?

@israpps
Copy link
Contributor Author

israpps commented Nov 14, 2023

@israpps why close this?

Will make it from scratch
The PR was based on your fork, not upstream

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