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

🐛 [BUG] mt32-pi / MiSTer I²C comms broken with recent Raspberry Pi firmware #292

Closed
dwhinham opened this issue Jun 14, 2022 · 3 comments
Labels

Comments

@dwhinham
Copy link
Owner

Issues #289 and #291 described problems with mt32-pi v0.12.0 when used with MiSTer FPGA.

The problem has been narrowed-down as being related to the Raspberry Pi firmware version used (start_cd.elf / fixup_cd.dat).

Circle Step 44.5 (as shipped in mt32-pi v0.12.0) recommends firmware from March 24.

With firmware versions later than March 22, I²C communications between mt32-pi and MiSTer are broken (corrupt OLED screen, "Unknown mode" errors, etc).

This has been temporarily worked-around in v0.12.1 by reverting to the previous recommended firmware version (January 20th).

This issue is to track investigation into what the underlying issue is. The firmware commit history suggests changes related to clocks, hence there could be an issue with how Circle's mailbox driver is retrieving clocks via the firmware (the I²C master driver appears to rely on the core clock).

@dwhinham dwhinham added the bug label Jun 14, 2022
@nikitalita
Copy link
Contributor

I can confirm the clock is wrong when using the March 24th firmware.

With a configuration file that has i2c bus speed set to 400khz.

Mt32-pi 0.12.1 (January 20th firmware), Clock is at correct frequency of 400khz
image

Mt32-pi 0.12.0 (March 24th firmware), Clock is at an incorrect frequency of ~641khz
image

dwhinham added a commit that referenced this issue Jun 15, 2022
@nikitalita
Copy link
Contributor

I think this can be closed now?

@dwhinham
Copy link
Owner Author

Yes, it can - I was being lazy because the commit message would have closed it when it hit master 🙂

dwhinham added a commit that referenced this issue Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants