[riot-notifications] [RIOT-OS/RIOT] [WIP] cpu/atmega_common: pseudomodule-based pin change interrupt implementation (#11122)

Marian Buschsieweke notifications at github.com
Fri Mar 15 17:36:28 CET 2019


Currently @kaspar030 request for changes is blocking this PR from being merged. We should wait for him to review the changes and confirm that his comments are addressed. (As his comments targeted the bug that prevented Murdock from building successfully and Murdock now did build successfully, I would say those comments have been addressed.)

Also: As this feature was extremely controversial, I marked this PR to need at least two ACKs. I think the reason why this feature was controversial have been addressed (as I [explained in detail a few comments above](https://github.com/RIOT-OS/RIOT/pull/11122#issuecomment-471729008)). But it makes still sense to me that someone else agrees on that before merging.

-- 
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/11122#issuecomment-473356635
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190315/dcf7b721/attachment.html>


More information about the notifications mailing list