From c20846a202ff899bc0db30313b4cdb4592887b82 Mon Sep 17 00:00:00 2001
From: Thomas Perrot <thomas.perrot@bootlin.com>
Date: Fri, 24 Apr 2026 09:44:24 +0200
Subject: [PATCH] snagrecover: fix cleanup option silently dropping "Done"
 message

In the -c (cleanup) getopts branch, `echo "Done"` was passed as an
argument to cleanup() instead of running as a separate command. The
missing semicolon caused the message to never be printed.

Signed-off-by: Thomas Perrot <thomas.perrot@bootlin.com>
---
 src/snagrecover/am335x_usb_setup.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/snagrecover/am335x_usb_setup.sh b/src/snagrecover/am335x_usb_setup.sh
index df2e0ed..2fe8ccd 100755
--- a/src/snagrecover/am335x_usb_setup.sh
+++ b/src/snagrecover/am335x_usb_setup.sh
@@ -99,7 +99,7 @@ while getopts "p:r:s:n:ch:" opt; do
     r) ROMUSB=$OPTARG;;
     s) SPLUSB=$OPTARG;;
     n) NETNS_NAME=$OPTARG;;
-    c) cleanup echo "Done"
+    c) cleanup && echo "Done"
       exit 0
       ;;
     h) print_usage