[riot-notifications] [RIOT-OS/RIOT] tests : Add new test periph_gpio_arduino to allow testing of arduino pin numbering (#7392)

lebrush notifications at github.com
Fri Sep 15 09:24:23 CEST 2017


lebrush requested changes on this pull request.

Untested but looks good and trust @smlng test skills :-). For now is Ok except 2 small style & copyright nitpicks. It would be nice to have implementation for the A pins in the future.

> @@ -0,0 +1,195 @@
+/*
+ * Copyright (C) 2017 Tom Keddie
+ * Copyright (C) 2014 Freie Universit├Ąt Berlin

I guess this Copyright line should go away

> @@ -0,0 +1,195 @@
+/*
+ * Copyright (C) 2017 Tom Keddie
+ * Copyright (C) 2014 Freie Universit├Ąt Berlin
+ *
+ * This file is subject to the terms and conditions of the GNU Lesser General
+ * Public License v2.1. See the file LICENSE in the top level directory for more
+ * details.
+ */
+
+/**
+ * @ingroup     tests
+ * @{
+ *
+ * @file
+ * @brief       Manual test application for GPIO peripheral drivers using Arduino pin numbering

This line is really long, please trim to 80 characters

-- 
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/7392#pullrequestreview-62963178
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20170915/7251c46c/attachment.html>


More information about the notifications mailing list