[riot-notifications] [RIOT-OS/RIOT] cpu/atmega: WIP implementation of pin change interrupts (#7610)

Robert Hartung 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:
https://github.com/RIOT-OS/RIOT/pull/7610#issuecomment-330140929
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20170917/d883d6fe/attachment.html>


More information about the notifications mailing list