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

Change the FMC frequency source to HCLK #995

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

Bexin3
Copy link

@Bexin3 Bexin3 commented Nov 10, 2024

Change the FMC frequency source to HCLK, freeing up PLL2 for other uses. On Portenta, the frequency will be same. On giga it will be higher but still within spec, and timings should get recalculated based on MICROPY_HW_SDRAM_FREQUENCY.

Change the FMC frequency source to HCLK, freeing up PLL2 for other uses. On Portenta, the frequency will be same. On giga it will be higher but still within spec, and I timings should get recalculated based on MICROPY_HW_SDRAM_FREQUENCY.
@CLAassistant
Copy link

CLAassistant commented Nov 10, 2024

CLA assistant check
All committers have signed the CLA.

@Bexin3
Copy link
Author

Bexin3 commented Nov 10, 2024

Had to do a small change, should be in a final form now though.

@megacct
Copy link

megacct commented Nov 10, 2024

Applied this PR to my GIGA and ran its SDRAM intensive workload for 90mins or so. Nil impact. I will continue running with these changes and report any issues

@Bexin3
Copy link
Author

Bexin3 commented Nov 10, 2024

Applied this PR to my GIGA and ran its SDRAM intensive workload for 90mins or so. Nil impact. I will continue running with these changes and report any issues

That's great to hear!!
I tried for a bit with video transfer from camera to ram to display and worked well, though didn't test it that long yet.

From my testing I was now able to set PLL2 to any frequency while using ram, which is great as I can now use it to have ADCs run as I want them to. (Though this will be useful for anyone who wants to use PLL2 at different frequencies)

Speed wise with transfers couldn't see any decrease.

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