From 237d89a3a84a7ab897e0f07cb32ff3c2573f0f4e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ho=C3=A0ng=20=C4=90=E1=BB=A9c=20Hi=E1=BA=BFu?=
 <code@hdhoang.space>
Date: Fri, 22 Feb 2019 14:29:45 +0700
Subject: [PATCH 1/3] create a Makefile target for semihosting

---
 .travis.yml | 2 +-
 Makefile    | 7 ++++++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index 0f0e5e7..ffc0a3e 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -32,7 +32,7 @@ install:
 
 script:
   - make
-  - cargo build --features use_semihosting
+  - make build-semihosting
   - "[[ ${TRAVIS_OS_NAME} != 'windows' ]] && make bloat"
 
 cache:
diff --git a/Makefile b/Makefile
index cd40d66..079295f 100644
--- a/Makefile
+++ b/Makefile
@@ -6,11 +6,16 @@ build:
 	rustup target add thumbv7m-none-eabi
 	cargo build --release
 
+build-semihosting:
+	rustup component add llvm-tools-preview
+	rustup target add thumbv7m-none-eabi
+	cargo build --release --features use_semihosting
+
 dfu: build
 	./scripts/generate_dfu.sh
 	ls -l anne-key.dfu
 
-debug: build
+debug: build-semihosting
 	arm-none-eabi-gdb target/thumbv7m-none-eabi/release/anne-key
 
 openocd:

From 4c7e4efddd376ec15ba7f2e681653ec0a46697de Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ho=C3=A0ng=20=C4=90=E1=BB=A9c=20Hi=E1=BA=BFu?=
 <code@hdhoang.space>
Date: Fri, 22 Feb 2019 14:31:13 +0700
Subject: [PATCH 2/3] expand on openocd & gdbserver

---
 docs/hardware.md | 25 ++++++++++++++++++++-----
 1 file changed, 20 insertions(+), 5 deletions(-)

diff --git a/docs/hardware.md b/docs/hardware.md
index 5b4c63b..09339c8 100644
--- a/docs/hardware.md
+++ b/docs/hardware.md
@@ -22,18 +22,33 @@ The [STM32L151 Reference Manual](http://www.st.com/content/ccc/resource/technica
 Flashing and debugging
 ----------------------
 
-To develop it's best to directly flash via the debug pins instead of DFU, so you get full debugging support and even working stdout to your host machine.
+To develop it's best to directly flash via the debug pins instead of
+DFU, so you get full debugging support and even working semihosting
+printing to your host machine.
 
-Any ST-Link v2 programmer will do. You can find them cheap on ebay, or if you already have a STM32 Nucleo board you can use the programmer of that.
+Any ST-Link v2 or v2.1 programmer will do. You can find them cheap on
+ebay, or if you already have an STM32 Discovery/Nucleo board you can
+use the programmer built into that.
 
-All the debug pins are exposed and easily accessible. Just solder on some wires and connect to your programmer:
+All the debug pins are exposed and easily accessible, however standard
+jumpers are too high for the case. You can solder on some wires and connect to
+your programmer:
 
 ![PCB](images/stlink-solder.jpg)
 
+The LED chip is the one closer to the edge. The pins order is the same
+for both chips, counting from the BLE shield: SWDIO, SWCLK, GND, 3V3.
+
 OpenOCD
 -------
 
-Once your programmer is connected start `make openocd`.
+For the best experience, use a new opencd with a unified
+`interface/stlink.cfg` (0.11, not yet released). Otherwise you will
+need to specify the precise ST-Link version in `openocd.cfg`.  Once
+your programmer is connected, start the gdbserver with `make openocd`.
+
+In another console, `make debug` will build a semihosting-enabled
+binary and run `arm-none-eabi-gdb` over the gdbserver connection.
 
 DFU
 ---
@@ -88,4 +103,4 @@ digit. The OUI of the address can be TI, or unassigned.
 Case
 ----
 
-The PCB's screw positions are incompatible with most 60% case.
+The PCB's screw positions are incompatible with most 60% cases.

From 8de8f214dd2d120fd12200c1c162511fd98b13ef Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ho=C3=A0ng=20=C4=90=E1=BB=A9c=20Hi=E1=BA=BFu?=
 <code@hdhoang.space>
Date: Fri, 22 Feb 2019 15:18:04 +0700
Subject: [PATCH 3/3] repeat the command to list modified files

the variable MODIFIED_FILES lost linebreaks, wrongly skipping doc+code PRs
---
 .travis.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index ffc0a3e..7b4be2c 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -20,9 +20,9 @@ before_install:
 - |
     set -e
     # fail loudly when force-pushed
-    MODIFIED_FILES=$(git diff --name-only $TRAVIS_COMMIT_RANGE)
+    git diff --name-only $TRAVIS_COMMIT_RANGE > /dev/null
     # waiting for native solution https://github.com/travis-ci/travis-ci/issues/6301
-    if ! echo ${MODIFIED_FILES} | grep -qvE '(\.md$)|(^docs)/'; then
+    if ! git diff --name-only $TRAVIS_COMMIT_RANGE | grep -qvE '(\.md$)|(^docs)/'; then
       echo "Only docs were updated, stopping build process."
       exit
     fi