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

More DRX definitions #385

Merged
merged 2 commits into from
Jun 18, 2024
Merged

Conversation

team-orangeBlue
Copy link
Contributor

@team-orangeBlue team-orangeBlue commented Jun 13, 2024

Changed DRH state names, notably:
UNK0-> NORADIO (connected DRCs warn about poor connection and disconnect afterwards)
UNK1-> WII (connected DRCs enter wii mode without input)
UNK2-> [not renamed] (connected DRCs power off and when powered back on, enter wii mode)
UNK3-> NODRC (connected DRCs immediately power off; this mode can only be disabled by rebooting)
UNK7F-> [not renamed] (this mode is not recognized on latest DRH firmware, comment added)

DRC EEPROM now defined as data instead of being unknown.
More info yet to be added. What I can say is that the DRC does not know its' own serial.
Initial dumps seem to show that the size is 5E0h bytes total with a filler hex-string of "DEADBABE" at the end (when dumping as DRC0 - dumping as DRC1 shows strange results)

Changed DRH state names, notably:
UNK0-> NORADIO
UNK1-> WII
UNK2-> [not renamed]
UNK3-> NODRC
UNK7F-> [not renamed]

DRC EEPROM now defined as data instead of being unknown.
More info yet to be added.
Comment on lines 143 to 144
CCR_CDC_SYS_DRH_STATE_ECO = 0x04,
//! On latest firmware this state is not recognized
Copy link
Contributor

Choose a reason for hiding this comment

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

This state is definitely recognized on the latest firmware, even if it fails to enter it.
There are several references to it in the set and get state code.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The suggestion is for state 7F, not state 04.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, I'm talking about state 7F, I just put the comment one line too high.

@GaryOderNichts GaryOderNichts merged commit 263c036 into devkitPro:master Jun 18, 2024
1 check passed
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