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

Changed the MSP430 hwtimer implementation to use TimerB7 instead of Time... #171

Closed
wants to merge 1 commit into from
Closed

Conversation

rousselk
Copy link
Contributor

...rA3.

This allows us to have 7 hwtimer instances (instead of 3) on MSP430F16x MCU.

…imerA3.

This allows us to have 7 hwtimer instances (instead of 3) on MSP430F16x MCU.
@mmbabel
Copy link

mmbabel commented Aug 22, 2013

While this is true for the F16x models, some other MCU's like F12x, F11x, G2x, F21x, F22x, AFE2x, F41x, F42x, F51x and F643x will have no available hwtimers at all. I highly recommend to find a solution to use TimerA and TimerB instead of remove the use of TimerA, all MCU's have a TimerA but not all have a TimerB.

@rousselk
Copy link
Contributor Author

Indeed; however, since RIOT only support MSPF16x for now, I made this as a quick fix to improve the current situation.

Use of many timers is a very interesting solution, but it will probably require some changes in the hwtimer API. I'm gonna open a ticket about that.

@mehlis
Copy link
Contributor

mehlis commented Sep 14, 2013

please rebase onto master, there is at least one conflict

@rousselk
Copy link
Contributor Author

Actually, this PR won't be included.

Work has begun on a more portable solution, that uses both TimerA and TimerB where appropriate (as overflowed suggested in a previous comment), and will work on any MSP430 device. It will supersede and replace this branch.

@rousselk rousselk closed this Sep 16, 2013
eduazocar pushed a commit to eduazocar/RIOT that referenced this pull request Jul 1, 2022
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.

None yet

3 participants