fix(cdsp_remoteproc): gate SSR behind --ssr and log wait timeouts/poll interval

Signed-off-by: Srikanth Muppandam <smuppand@qti.qualcomm.com>

Author: smuppand

Runner/suites/Kernel/Baseport/cdsp_remoteproc/run.sh

@@ -44,7 +44,65 @@ STOP_TO="${STOP_TO:-10}"
 START_TO="${START_TO:-10}"
 POLL_I="${POLL_I:-1}"
 
-log_info "DEBUG: STOP_TO=$STOP_TO START_TO=$START_TO POLL_I=$POLL_I"
+# --- CLI ----------------------------------------------------------------------
+# Default: do NOT do SSR (stop/start). Enable explicitly with --ssr
+DO_SSR=0
+
+usage() {
+    echo "Usage: $0 [--ssr] [--stop-to SEC] [--start-to SEC] [--poll-i SEC]" >&2
+    echo " --ssr Perform CDSP stop/start (SSR). Default: OFF" >&2
+    echo " --stop-to SEC Stop timeout (default: $STOP_TO)" >&2
+    echo " --start-to SEC Start timeout (default: $START_TO)" >&2
+    echo " --poll-i SEC Poll interval (default: $POLL_I)" >&2
+}
+
+while [ $# -gt 0 ]; do
+    case "$1" in
+        --ssr)
+            DO_SSR=1
+            shift
+            ;;
+        --stop-to)
+            if [ $# -lt 2 ]; then
+                log_fail "Missing value for --stop-to"
+                usage
+                exit 2
+            fi
+            STOP_TO="$2"
+            shift 2
+            ;;
+        --start-to)
+            if [ $# -lt 2 ]; then
+                log_fail "Missing value for --start-to"
+                usage
+                exit 2
+            fi
+            START_TO="$2"
+            shift 2
+            ;;
+        --poll-i)
+            if [ $# -lt 2 ]; then
+                log_fail "Missing value for --poll-i"
+                usage
+                exit 2
+            fi
+            POLL_I="$2"
+            shift 2
+            ;;
+        -h|--help)
+            usage
+            exit 0
+            ;;
+        *)
+            log_warn "Unknown argument: $1"
+            usage
+            exit 2
+            ;;
+    esac
+done
+
+log_info "Tunables: STOP_TO=$STOP_TO START_TO=$START_TO POLL_I=$POLL_I"
+log_info "SSR control: DO_SSR=$DO_SSR (0=no stop/start, 1=do stop/start)"
 
 # --- Device Tree gate ----------------------------------------------------
 if dt_has_remoteproc_fw "$FW"; then
@@ -58,9 +116,11 @@ fi
 
 # ---------- Discover all matching remoteproc entries ----------
 # get_remoteproc_by_firmware prints: "path|state|firmware|name"
-entries="$(get_remoteproc_by_firmware "$FW" "" all)" || entries=""
+entries="$(get_remoteproc_by_firmware "$FW" "" all 2>/dev/null)" || entries=""
 if [ -z "$entries" ]; then
     log_fail "$FW present in DT but no /sys/class/remoteproc entry found"
+    log_info "Writing to $RES_FILE"
+    echo "$TESTNAME FAIL" > "$RES_FILE"
     exit 1
 fi
 
@@ -97,43 +157,51 @@ while IFS='|' read -r rpath rstate rfirm rname; do
         continue
     fi
 
-    # Stop
-    dump_rproc_logs "$rpath" before-stop
-    t0=$(date +%s)
-    log_info "$inst_id: stopping"
-    if stop_remoteproc "$rpath" && wait_remoteproc_state "$rpath" offline "$STOP_TO" "$POLL_I"; then
-        t1=$(date +%s)
-        log_pass "$inst_id: stop PASS ($((t1 - t0))s)"
-        stop_res="PASS"
-    else
-        dump_rproc_logs "$rpath" after-stop-fail
-        log_fail "$inst_id: stop FAIL"
-        stop_res="FAIL"
-        inst_fail=$((inst_fail + 1))
-        RESULT_LINES="$RESULT_LINES
+    if [ "$DO_SSR" -eq 1 ]; then
+        # Stop
+        dump_rproc_logs "$rpath" before-stop
+        t0=$(date +%s)
+        log_info "$inst_id: stopping"
+	log_info "$inst_id: waiting for state=offline with timeout=${STOP_TO}s poll=${POLL_I}s"
+        if stop_remoteproc "$rpath" && wait_remoteproc_state "$rpath" offline "$STOP_TO" "$POLL_I"; then
+            t1=$(date +%s)
+            log_pass "$inst_id: stop PASS ($((t1 - t0))s)"
+            stop_res="PASS"
+        else
+            dump_rproc_logs "$rpath" after-stop-fail
+            log_fail "$inst_id: stop FAIL"
+            stop_res="FAIL"
+            inst_fail=$((inst_fail + 1))
+            RESULT_LINES="$RESULT_LINES
  $inst_id: boot=$boot_res, stop=$stop_res, start=$start_res, ping=$ping_res"
-        continue
-    fi
-    dump_rproc_logs "$rpath" after-stop
-
-    # Start
-    dump_rproc_logs "$rpath" before-start
-    t2=$(date +%s)
-    log_info "$inst_id: starting"
-    if start_remoteproc "$rpath" && wait_remoteproc_state "$rpath" running "$START_TO" "$POLL_I"; then
-        t3=$(date +%s)
-        log_pass "$inst_id: start PASS ($((t3 - t2))s)"
-        start_res="PASS"
-    else
-        dump_rproc_logs "$rpath" after-start-fail
-        log_fail "$inst_id: start FAIL"
-        start_res="FAIL"
-        inst_fail=$((inst_fail + 1))
-        RESULT_LINES="$RESULT_LINES
+            continue
+        fi
+        dump_rproc_logs "$rpath" after-stop
+
+        # Start
+        dump_rproc_logs "$rpath" before-start
+        t2=$(date +%s)
+        log_info "$inst_id: starting"
+	log_info "$inst_id: waiting for state=running with timeout=${START_TO}s poll=${POLL_I}s"
+        if start_remoteproc "$rpath" && wait_remoteproc_state "$rpath" running "$START_TO" "$POLL_I"; then
+            t3=$(date +%s)
+            log_pass "$inst_id: start PASS ($((t3 - t2))s)"
+            start_res="PASS"
+        else
+            dump_rproc_logs "$rpath" after-start-fail
+            log_fail "$inst_id: start FAIL"
+            start_res="FAIL"
+            inst_fail=$((inst_fail + 1))
+            RESULT_LINES="$RESULT_LINES
  $inst_id: boot=$boot_res, stop=$stop_res, start=$start_res, ping=$ping_res"
-        continue
+            continue
+        fi
+        dump_rproc_logs "$rpath" after-start
+    else
+        log_info "$inst_id: SSR disabled (--ssr not set). Skipping stop/start."
+        stop_res="SKIPPED"
+        start_res="SKIPPED"
     fi
-    dump_rproc_logs "$rpath" after-start
 
     # Optional RPMsg ping
     if CTRL_DEV=$(find_rpmsg_ctrl_for "$FW"); then