[riot-notifications] [RIOT-OS/RIOT] cpu/atmega: WIP implementation of pin change interrupts (#7610)
notifications at github.com
Mon Sep 18 08:55:02 CEST 2017
> But they were working before. Why it doesn't work with this PR?
@kYc0o Probably this wasn't explained very well. The current state is there are *no* Pin Change Interrupts available at all in RIOT. However, the threading implementation uses a pin change interrupt for context swaps. The ISR however, is only triggering a context swap here! See https://github.com/RIOT-OS/RIOT/blob/master/cpu/atmega_common/thread_arch.c#L254, ideally with some overhead we would have to check if a PCINT is enabled for the corresponding port and call pcint_handler from gpio.c (see my implementation). Otherwise, we'd make a context swap. However, this has a little overhead (check if other pins are enabled).
Comments and thoughts much appreciated. I would be ok with a little overhead (2-3 instructions) for context swapping.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications