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

⚠️ rework DMA and GPTMR #1194

Merged
merged 9 commits into from
Mar 2, 2025
Merged

⚠️ rework DMA and GPTMR #1194

merged 9 commits into from
Mar 2, 2025

Conversation

stnolting
Copy link
Owner

  • ⚠️ DMA: remove FIRQ-triggered auto-mode; re-arrange control register bist
  • ⚠️ GPTMR: remove mode configuration bit; timer now always operate in "continuous" mode (interval timer)

@stnolting stnolting added HW Hardware-related cleanup Clean-up the codebase labels Mar 2, 2025
@stnolting stnolting self-assigned this Mar 2, 2025
@stnolting stnolting marked this pull request as ready for review March 2, 2025 09:01
@stnolting stnolting merged commit 449dd87 into main Mar 2, 2025
7 checks passed
@stnolting stnolting deleted the dev020325 branch March 2, 2025 14:32
@jpf91
Copy link
Contributor

jpf91 commented Mar 8, 2025

@stnolting Hmm any reason why the FIRQ trigger mode was removed? I was planning to use it :-)

@stnolting
Copy link
Owner Author

That was another completely over-engineered feature... 🙈

The DMA only has one channel, which also only has a single descriptor. Event-driven triggering therefore seems to make little sense.

In addition, it does not require much overhead to start the (pre-programmed) DMA from an interrupt-driven service routine.

I was planning to use it :-)

Can you give us a few details? Perhaps I was too hasty and there are actually real-word applications that would benefit greatly from this. 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Clean-up the codebase HW Hardware-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants