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
Add Datel AGP save commands for SRAM/FRAM. Improve EEPROM. #2045
Conversation
Yeah. Whoops. |
This PR fixes my GBA games in the Advance Game Port as I said on the forums. I guess flash memory is pretty rare compared to eeprom? |
@@ -191,7 +206,7 @@ u32 CEXIAgp::ImmRead(u32 _uSize) | |||
|
|||
void CEXIAgp::ImmWrite(u32 _uData, u32 _uSize) | |||
{ | |||
if ((_uSize == 1) && ((_uData & 0xFF000000) == 0)) | |||
if ((_uSize == 1) && ((_uData & 0xFF000000) == 0)) // Execute current command? |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
Would you please squash the commits down to one commit? |
{ | ||
// Handle endian write - could be done with byte access in 0xAE commands instead | ||
std::vector<u8> temp_eeprom; | ||
temp_eeprom.resize(m_eeprom_size); |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
I've seen someone on IRC say that with Fire Emblem (not sure which one?) it's not saving properly and zeroes out, but I'm not able to reproduce save problems right now. I did some pretty thorough testing, using the PR build instead of my custom build with the hanging fix. Test 1:
Test 2:
Test 3:
|
Sorry to be picky, but would you please squash the two commits? |
Handle possible endian file read issues at file access time instead of using Swap64 on every access. Code comments and clean up.
@dolphin-emu-bot rebuild |
Add Datel AGP save commands for SRAM/FRAM. Improve EEPROM.
This adds initial support for the 0xAE04 and 0xAE07 commands of the AGP (doesn't include Flash ROM).