[riot-notifications] [RIOT-OS/RIOT] Atmega PWM implementation (#7542)

Robert Hartung notifications at github.com
Fri Sep 15 10:11:59 CEST 2017


> I thought how to integrate them and finally decided to create a new timer definition called mini_timer_t instead of mega_timer_t since they are 8-bit timers and registers are also different. Defining it within the same struct might lead some future developer to fiddle for a while because "xtimer won't work" as it requires at least 16-bit timers. This was the reasoning behind it.

ACK. This is the main problem (even though xtimer needs a rework anyway..). And defining another type is absolutely the right way.

> In any case I would leave PWM and RTT as they are right now, and start working on a timer refactorization afterwards since it's a major change.

+1: The timer really needs a rework. Should we create an issue to discuss architecture and work everything out, so we can showcase what we imagine?
I would really like to see a single timer interface that is common across all timers (if possible), so that the user has an easy life to use ANY timer. RTT and PWM can then use this library. Do we have also have to provide a way to make sure that when using RTT with Timer2, PWM cannot be used? (e.g. software-restriction or something)? Or can both be used at the same time? hmmm

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/7542#issuecomment-329713326
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20170915/974e061e/attachment-0001.html>


More information about the notifications mailing list