Merge pull request qualcomm-linux#139 from smuppand/camera_rdi

Camera RDI Pipeline: Modularize and Robustify Frame Capture with Dynamic Python Parsing

Author: abbajaj806

Runner/suites/Multimedia/Camera/Camera_RDI_FrameCapture/README_Camera_RDI_FrameCapture.md

@@ -0,0 +1,98 @@
+# Camera RDI Frame Capture Test
+
+This test validates functional camera RDI (Raw Dump Interface) pipelines by:
+
+- Dynamically detecting all camera pipelines using `media-ctl`
+- Parsing valid RDI pipelines with a Python helper script
+- Streaming frames using `yavta` from detected working pipelines
+- Supporting manual override of video format and frame count
+
+## 📁 Test Directory Structure
+
+```
+Camera_RDI_FrameCapture/
+├── run.sh
+├── README_Camera_RDI_FrameCapture.md
+
+```
+
+## 🧠 How It Works
+
+1. Detects media device node dynamically
+2. Dumps the topology to a temporary file
+3. Parses pipeline details using `parse_media_topology.py`
+4. For each detected pipeline:
+   - Applies correct media-ctl `-V` and `-l` configuration
+   - Sets V4L2 controls pre-/post-streaming via `yavta`
+   - Attempts frame capture using `yavta`
+5. Logs PASS/FAIL/SKIP per pipeline
+6. Generates a `.res` file with final test result
+
+## ⚙️ Dependencies
+
+Make sure the following tools are available in the target filesystem:
+
+- `media-ctl`
+- `yavta`
+- `v4l2-ctl`
+- `python3`
+- Python camera pipeline parser (see `utils/camera/parse_media_topology.py`)
+- Kernel module: `qcom_camss`
+- Required DT nodes for `camss`, `isp`, or `camera` compatible strings
+
+## 🧪 Usage
+
+```sh
+./run.sh [--format <fmt1,fmt2,...>] [--frames <count>] [--help]
+```
+
+### Examples:
+
+- Auto-detect and capture 10 frames per working RDI pipeline:
+  ```sh
+  ./run.sh
+  ```
+
+- Force UYVY format and capture 5 frames:
+  ```sh
+  ./run.sh --format UYVY --frames 5
+  ```
+
+- Comma-seperated list of V4L2 formats to attempt per pipeline
+  ```sh
+  ./run.sh --format SRGGB10P,YUYV,UYVY --frames 5
+  ```
+
+## 📦 Output
+
+- Captured frame files: `frame-#.bin` in current directory
+- Result summary: `Camera_RDI_FrameCapture.res`
+- Detailed logs through `functestlib.sh`-based `log_info`, `log_pass`, etc.
+
+## ✅ Pass Criteria
+
+- At least one pipeline successfully captures frames
+- Logs include `"Captured <n> frames"` for at least one working video node
+
+## ❌ Fail/Skip Criteria
+
+- If pipeline configuration fails or no frames captured, it is marked FAIL
+- If no working pipelines are found or prerequisites are unmet, test is SKIPPED
+
+## 🧼 Cleanup
+
+Temporary files created:
+- `/tmp/v4l2_camera_RDI_dump_topo.*`
+- `/tmp/v4l2_camera_RDI_dump_pipelines.*`
+
+They are auto-removed at the end of the test.
+
+## 📝 Notes
+
+- The test is dynamic and supports multiple pipelines per board
+- Python script only outputs **valid** working pipelines (validated via `v4l2-ctl`)
+- `run.sh` is robust, CI-ready, and skips flaky or unsupported configurations gracefully
+
+---
+
+© Qualcomm Technologies, Inc. – All rights reserved

Runner/suites/Multimedia/Camera/Camera_RDI_FrameCapture/run.sh

@@ -0,0 +1,211 @@
+#!/bin/sh
+# Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
+# SPDX-License-Identifier: BSD-3-Clause-Clear
+
+SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
+INIT_ENV=""
+SEARCH="$SCRIPT_DIR"
+while [ "$SEARCH" != "/" ]; do
+    if [ -f "$SEARCH/init_env" ]; then
+        INIT_ENV="$SEARCH/init_env"
+        break
+    fi
+    SEARCH=$(dirname "$SEARCH")
+done
+
+if [ -z "$INIT_ENV" ]; then
+    echo "[ERROR] Could not find init_env (starting at $SCRIPT_DIR)" >&2
+    exit 1
+fi
+
+if [ -z "$__INIT_ENV_LOADED" ]; then
+    # shellcheck disable=SC1090
+    . "$INIT_ENV"
+fi
+# shellcheck disable=SC1090,SC1091
+. "$TOOLS/functestlib.sh"
+
+TESTNAME="Camera_RDI_FrameCapture"
+RES_FILE="./$TESTNAME.res"
+test_path=$(find_test_case_by_name "$TESTNAME")
+cd "$test_path" || exit 1
+
+print_usage() {
+    cat <<EOF
+Usage: $0 [--format <v4l2_fmt1,v4l2_fmt2,...>] [--frames <count>] [--help]
+
+Options:
+  --format <v4l2_fmt1,v4l2_fmt2,...> Test one or more comma-separated formats (e.g., UYVY,NV12)
+  --frames <count> Number of frames to capture per pipeline (default: 10)
+  --help Show this help message
+EOF
+}
+
+log_info "----------------------------------------------------------------------"
+log_info "------------------- Starting $TESTNAME Testcase ----------------------"
+log_info "=== Test Initialization ==="
+
+# --------- Argument Parsing ---------
+USER_FORMAT=""
+FRAMES=10
+while [ $# -gt 0 ]; do
+    case "$1" in
+        --format)
+            shift
+            USER_FORMAT="$1"
+            ;;
+        --frames)
+            shift
+            FRAMES="$1"
+            ;;
+        --help)
+            print_usage
+            exit 0
+            ;;
+        *)
+            log_error "Unknown argument: $1"
+            print_usage
+            exit 1
+            ;;
+    esac
+    shift
+done
+
+# --------- Prechecks ---------
+if ! dt_confirm_node_or_compatible "isp" "cam" "camss"; then
+    log_skip "$TESTNAME SKIP – No ISP/camera node/compatible found in DT"
+    echo "$TESTNAME SKIP" >"$RES_FILE"
+    exit 0
+fi
+
+DRIVER_MOD="qcom_camss"
+DMESG_MODULES='qcom_camss|camss|isp'
+DMESG_EXCLUDE='dummy regulator|supply [^ ]+ not found|using dummy regulator|Failed to create device link|reboot-mode.*-EEXIST|can.t register reboot mode'
+
+if ! is_module_loaded "$DRIVER_MOD"; then
+    log_skip "$TESTNAME SKIP – Driver module $DRIVER_MOD not loaded"
+    echo "$TESTNAME SKIP" >"$RES_FILE"
+    exit 0
+fi
+
+if scan_dmesg_errors "$SCRIPT_DIR" "$DMESG_MODULES" "$DMESG_EXCLUDE"; then
+    log_skip "$TESTNAME SKIP – $DRIVER_MOD probe errors detected in dmesg"
+    echo "$TESTNAME SKIP" >"$RES_FILE"
+    exit 0
+fi
+
+# --------- Dependency Checks ---------
+check_dependencies media-ctl yavta python3 v4l2-ctl || {
+    log_skip "$TESTNAME SKIP – Required tools missing"
+    echo "$TESTNAME SKIP" >"$RES_FILE"
+    exit 0
+}
+
+# --------- Media Node Detection ---------
+MEDIA_NODE=$(detect_media_node)
+if [ -z "$MEDIA_NODE" ]; then
+    log_skip "$TESTNAME SKIP – Media node not found"
+    echo "$TESTNAME SKIP" >"$RES_FILE"
+    exit 0
+fi
+log_info "Detected media node: $MEDIA_NODE"
+
+# --------- Pipeline Detection ---------
+TOPO_FILE=$(mktemp "/tmp/${TESTNAME}_topo.XXXXXX")
+TMP_PIPELINES_FILE=$(mktemp "/tmp/${TESTNAME}_blocks.XXXXXX")
+trap 'rm -f "$TOPO_FILE" "$TMP_PIPELINES_FILE"' EXIT
+
+media-ctl -p -d "$MEDIA_NODE" >"$TOPO_FILE" 2>/dev/null
+PYTHON_PIPELINES=$(run_camera_pipeline_parser "$TOPO_FILE")
+if [ -z "$PYTHON_PIPELINES" ]; then
+    log_skip "$TESTNAME SKIP – No valid pipelines found"
+    echo "$TESTNAME SKIP" >"$RES_FILE"
+    exit 0
+fi
+
+printf "%s\n" "$PYTHON_PIPELINES" > "$TMP_PIPELINES_FILE"
+
+log_info "User format override: ${USER_FORMAT:-<none>}"
+log_info "Frame count per pipeline: $FRAMES"
+
+# --------- Pipeline Processing ---------
+PASS=0; FAIL=0; SKIP=0; COUNT=0
+block=""
+
+while IFS= read -r line || [ -n "$line" ]; do
+    if [ "$line" = "--" ]; then
+        COUNT=$((COUNT+1))
+        TMP="/tmp/cam_block.$$.$COUNT"
+        printf "%s\n" "$block" > "$TMP"
+
+        # Parses block and sets SENSOR, VIDEO, YAVTA_DEV, FMT, etc
+        parse_pipeline_block "$TMP"
+        rm -f "$TMP"
+
+        # -------- Multi-format support (POSIX style, no arrays) --------
+        FORMATS_LIST="$USER_FORMAT"
+        if [ -z "$FORMATS_LIST" ]; then
+            # No user override, use detected format for this pipeline only
+            FORMATS_LIST="$YAVTA_FMT"
+        fi
+
+        OLD_IFS="$IFS"
+        IFS=','
+        for FMT_OVERRIDE in $FORMATS_LIST; do
+            FMT_OVERRIDE=$(printf '%s' "$FMT_OVERRIDE" | sed 's/^ *//;s/ *$//')
+            TARGET_FORMAT="$FMT_OVERRIDE"
+            [ -z "$TARGET_FORMAT" ] && TARGET_FORMAT="$YAVTA_FMT"
+
+            log_info "----- Pipeline $COUNT: ${SENSOR:-unknown} $VIDEO $TARGET_FORMAT -----"
+
+            if [ -z "$VIDEO" ] || [ "$VIDEO" = "None" ] || [ -z "$YAVTA_DEV" ]; then
+                log_skip "$SENSOR: Invalid pipeline – skipping"
+                SKIP=$((SKIP+1))
+                continue
+            fi
+
+            configure_pipeline_block "$MEDIA_NODE" "$TARGET_FORMAT"
+            execute_capture_block "$FRAMES" "$TARGET_FORMAT"
+            RET=$?
+
+            case "$RET" in
+                0)
+                    log_pass "$SENSOR $VIDEO $TARGET_FORMAT PASS"
+                    PASS=$((PASS+1))
+                    ;;
+                1)
+                    log_fail "$SENSOR $VIDEO $TARGET_FORMAT FAIL (capture failed)"
+                    FAIL=$((FAIL+1))
+                    ;;
+                2)
+                    log_skip "$SENSOR $VIDEO $TARGET_FORMAT SKIP (unsupported format)"
+                    SKIP=$((SKIP+1))
+                    ;;
+                3)
+                    log_skip "$SENSOR $VIDEO missing data – skipping"
+                    SKIP=$((SKIP+1))
+                    ;;
+            esac
+        done
+        IFS="$OLD_IFS"
+        block=""
+    else
+        if [ -z "$block" ]; then
+            block="$line"
+        else
+            block="$block
+$line"
+        fi
+    fi
+done < "$TMP_PIPELINES_FILE"
+
+log_info "Test Summary: Passed: $PASS, Failed: $FAIL, Skipped: $SKIP"
+if [ "$PASS" -gt 0 ]; then
+    echo "$TESTNAME PASS" >"$RES_FILE"
+elif [ "$FAIL" -gt 0 ]; then
+    echo "$TESTNAME FAIL" >"$RES_FILE"
+else
+    echo "$TESTNAME SKIP" >"$RES_FILE"
+fi
+
+exit 0