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

Marian Buschsieweke notifications at github.com
Tue Apr 30 10:39:13 CEST 2019


Of the second PCINT Port (PCINT8 ... PCINT15) the pins PE0 (PCINT8, RXD0), PJ0 (PCINT9, RXD3) and PJ1 (PCINT10/TXD3) are available on the Arduino Mega. PJ2 - PJ6 (PCINT11 - PCINT15) are not mapped.

I did not test PE0, as UART0 is used for stdio. I did expect PJ0 and PJ1 to work, as only UART 0 and UART 1 are configured in `boards/common/atmega/include/periph_conf_atmega_common.h`, and UART 2 and UART 3 remain unused. Maybe there is indeed a conflict with UART 3 due to something I did overlook. I'll try to investigate.

I would personally keep the pcint configuration in the CPU and keep it fully populated in order to keep the maintenance burden low. (E.g. `gpio_init()` does also not test for GPIO pins that are available on the MCU, but not on the board.)

May point was that I could not confirm that the pcint implementation on port 2 of the ATmega2560 is working, which blocked this PR. Disabling that port for now (with the possibility of adding it later on) would allow me to set testing on green.

But a conflict with UART seems to be indeed the likely cause for the issue here. I hope that I can have a look at that again soon.

-- 
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-487865001
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190430/a1869523/attachment.html>


More information about the notifications mailing list