[riot-notifications] [RIOT-OS/RIOT] cpu/saml21: extend and fix exti configuration for saml21 variants (#10794)

Dylan Laduranty notifications at github.com
Thu Jan 17 10:40:29 CET 2019


dylad commented on this pull request.



> @@ -30,10 +30,29 @@ extern "C" {
  * @brief   Mapping of pins to EXTI lines, -1 means not EXTI possible
  */
 static const int8_t exti_config[2][32] = {
+#if defined(CPU_MODEL_SAML21G18A)

You're right but what about SAML21E18 ? and SAML21x16 / SAML21x17 ?
I won't oppose to this but I am not convinced. 
If we keep this, you also must add the B variant.

-- 
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/10794#discussion_r248597984
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190117/b3e0c16c/attachment.html>


More information about the notifications mailing list