michaelh/zephyr
1
0
Fork 0
Code Releases Activity

scripts/dfuutil: support both "alternate setting" name and number

Browse source 
Initial assumption with Arduino 101 was that the "alternate setting
name of the DFU interface" would be simple enough to be used as the
grep pattern when attempting to find the board in DFU mode.
However, for boards such as the 96Boards Carbon, this name includes
grep meta-characters, so the DFUUTIL_ALT environment variable is
provided as a numeric value, instead of the previously expected name
string.
In order to avoid false positives, we now check whether DFUUTIL_ALT
is a numeric value, and the grep pattern is adjusted accordingly to
search for either "alt=" or "name=".
Also take into account that the name might contain spaces.

Change-Id: I81351d03bd727b35093fd1280e2b8b6904f3c89b
Signed-off-by: Patrice Buriez <patrice.buriez@intel.com>
This commit is contained in:
Patrice Buriez 2017-03-26 15:55:23 +02:00 • committed by Anas Nashif
commit fd8c59b996
1 changed files with 19 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

21
scripts/support/dfuutil.sh
View file

@ -4,9 +4,26 @@
# fwversion.sh, included in arduino101-factory_recovery-flashpack.tar.bz2,
# which is available from https://downloadcenter.intel.com/download/25470
# It requires the following environment variables to be defined, typically
# exported from Makefile.board:
# - DFUUTIL_PID: vendor_ID:product_ID of the board in DFU mode
# - DFUUTIL_ALT: alternate setting name or number of the DFU interface
# - DFUUTIL_IMG: path to the binary image sent to the board
# - DFUUTIL_DFUSE_ADDR: target address, for DfuSe devices
DFUUTIL_EXE=${DFUUTIL:-dfu-util}
DFUUTIL_CMD="$DFUUTIL_EXE -d,$DFUUTIL_PID"
# Is DFUUTIL_ALT a numeric value?
num=$(printf '%u' "$DFUUTIL_ALT" 2>/dev/null)
if [ $? -eq 0 -a "$num" = "$DFUUTIL_ALT" ]; then
# alternate setting number
pattern=", alt=$DFUUTIL_ALT,"
else
# alternate setting name
pattern=", name=\"$DFUUTIL_ALT\","
fi
test_exe() {
if ! which $DFUUTIL_EXE >/dev/null 2>&1; then
echo "Error: Unable to locate dfu-util executable: $DFUUTIL_EXE"
@ -22,7 +39,7 @@ test_img() {
}
find_dfu() {
$DFUUTIL_CMD -l |grep "$DFUUTIL_ALT" >/dev/null 2>&1
$DFUUTIL_CMD -l |grep "$pattern" >/dev/null 2>&1
}
do_flash() {
@ -46,7 +63,7 @@ do_flash() {
# Flash DFU device with specified image
# Do NOT reset with -R, to avoid random 'error resetting after download'
$DFUUTIL_CMD -a $DFUUTIL_ALT -D $DFUUTIL_IMG
$DFUUTIL_CMD -a "$DFUUTIL_ALT" -D $DFUUTIL_IMG
ok=$?
if [ $ok -eq 0 -a $reset_dfu -eq 1 ]; then
echo "Now reset your board again to switch back to runtime mode."