Skip to content

GIGA: Remove Arduino pins 11-13 from PWM list to fix SPI1 #65

New issue

Have a question about this project? No Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “No 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? No Sign in to your account

Merged
merged 2 commits into from
Mar 6, 2025

Conversation

KurtE
Copy link

@KurtE KurtE commented Feb 2, 2025

resolves: #64

Removed the pins logical Arduino pins 11-13 from the PWM pin list, plus their defines
for the timers.

With these defines in place, the pins Alternate Function settings were set to that of the timers
versus the SPI value (5)

Note: This was done by @mjs513 and myself.

@facchinm
Copy link
Member

facchinm commented Mar 5, 2025

I'd rather comment the offending PWMs so they can be restored when the dynamic loading will be supported. Also, Since I merged #62, it would be cool to rebase the patch on main 😉

resolves: arduino#64

Removed the pins logical Arduino pins 11-13 from the PWM pin list, plus their defines
for the timers.

With these defines in place, the pins Alternate Function settings were set to that of the timers
versus the SPI value (5)

Note: This was done by @mjs513 and myself.
@KurtE KurtE force-pushed the GIGA_M7_fix_spi5 branch from 871d871 to e831260 Compare March 5, 2025 19:54
@KurtE
Copy link
Author

KurtE commented Mar 6, 2025

I added the lines back in as comments, plus I rebased. Did a quick test on GIGA and my ILI9341 code started up and did
fillScreens...

@facchinm facchinm merged commit 587b322 into arduino:arduino Mar 6, 2025
2 of 4 checks passed
@KurtE KurtE deleted the GIGA_M7_fix_spi5 branch March 6, 2025 16:39
No Sign up for free to join this conversation on GitHub. Already have an account? No Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GIGA: after merging previous changes: SPI1.transfer hangs board
2 participants