From b0d2afab949da48f4d624f7f24722ee151dd62e4 Mon Sep 17 00:00:00 2001
From: Ian Kelling <iank@fsf.org>
Date: Mon, 4 Mar 2019 10:46:53 -0500
Subject: [PATCH] improve documentation

---
 relay-cmd | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/relay-cmd b/relay-cmd
index 6ed7eae..0da382e 100755
--- a/relay-cmd
+++ b/relay-cmd
@@ -6,14 +6,19 @@ usage() {
   cat <<EOF
 Usage: relay-cmd poweroff|poweron|reboot HOSTNAME|CHANNEL [RELAY_ID]
 
-For CHANNEL and RELAY_ID, run hidusb-relay-cmd and hidusb-relay-cmd state.
-I recommend writing a config to use HOSTNAME instead. Config is /etc/relay-cmd.conf
-Example config:
-
-# This config is just bash that gets sourced.
+RELAY_ID is not needed if only 1 relay device is connected.  To
+understand CHANNEL and RELAY_ID, run hidusb-relay-cmd and
+hidusb-relay-cmd state.  I recommend writing a config in
+/etc/relay-cmd.conf , then using HOSTNAME instead of CHANNEL and
+RELAY_ID. It serves as documentation of what you connected the relay
+to. Example config:
+
+# This config is just sourced from bash, so make sure its valid bash.
 # relay_id is not needed if only 1 relay is connected.
 cephserver3_channel=1
 cephserver3_relay_id=HURTM
+cephserver2_channel=2
+cephserver2_relay_id=HURTM
 EOF
   exit $1
 }
@@ -29,6 +34,7 @@ if [[ -e /etc/relay-cmd.conf ]]; then
   source /etc/relay-cmd.conf
 fi
 
+# we know HOSTNAME is given if it doesn't start with a number.
 if [[ $chan != [0-9]* ]]; then
   chan_var=${chan}_channel
   chan=${!chan_var}
-- 
2.25.1