[riot-notifications] [RIOT-OS/RIOT] boards/common/nrf52: add openocd support for 'nordic_softdevice_ble' (#11470)

Alexandre Abadie notifications at github.com
Mon May 27 16:37:11 CEST 2019


aabadie commented on this pull request.



> @@ -0,0 +1,38 @@
+#!/usr/bin/env python3
+
+# Copyright (C) 2019 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.
+
+import sys
+from testrunner import run
+
+
+
+def _check_shell_is_working(child, tries=5):

I just rechecked the proposed change using `test_nordic_softdevice_openocd.sh` and it works like a charm. Maybe you missed the synchronization `child.expect_exact('All up, running the shell now')`, at line 18 ?

-- 
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/11470#discussion_r287813985
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190527/bc51113b/attachment.html>


More information about the notifications mailing list