aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLorenz Meier <lm@inf.ethz.ch>2014-04-15 19:25:07 +0200
committerLorenz Meier <lm@inf.ethz.ch>2014-04-15 19:25:07 +0200
commit282f40d16247db6ae2598e7f28cd5cc42c4600d0 (patch)
treeea3db1d12d8412e4f063177a7379de06c4ec471f
parentc77c5c878b05928d0817123e935af4941f653759 (diff)
downloadpx4-firmware-282f40d16247db6ae2598e7f28cd5cc42c4600d0.tar.gz
px4-firmware-282f40d16247db6ae2598e7f28cd5cc42c4600d0.tar.bz2
px4-firmware-282f40d16247db6ae2598e7f28cd5cc42c4600d0.zip
Hotfix to PX4IO uploader. There are no known mishaps due to it, but very clearly the IO firmware flashing process should be verified after an upload.
-rw-r--r--src/drivers/px4io/px4io_uploader.cpp9
1 files changed, 7 insertions, 2 deletions
diff --git a/src/drivers/px4io/px4io_uploader.cpp b/src/drivers/px4io/px4io_uploader.cpp
index dd8abbac5..28ec62356 100644
--- a/src/drivers/px4io/px4io_uploader.cpp
+++ b/src/drivers/px4io/px4io_uploader.cpp
@@ -201,9 +201,14 @@ PX4IO_Uploader::upload(const char *filenames[])
continue;
}
- if (bl_rev <= 2)
+ if (bl_rev <= 2) {
ret = verify_rev2(fw_size);
- else if(bl_rev == 3) {
+ } else if(bl_rev == 3) {
+ ret = verify_rev3(fw_size);
+ } else {
+ /* verify rev 4 and higher still uses the same approach and
+ * every version *needs* to be verified.
+ */
ret = verify_rev3(fw_size);
}