Skip to content

Conversation

smuppand
Copy link
Contributor

@smuppand smuppand commented Aug 8, 2025

What

  • Refactored Camera_RDI_FrameCapture/run.sh to be fully modular, sourcing all pipeline detection, parsing, and block execution logic from functestlib.sh and utils/camera/parse_media_topology.py.
  • The Python parser now dynamically handles sensor renaming and pad variations, ensuring robust camera block discovery on any Qualcomm-based platform.
  • Improved error handling, logging, and result file emission for CI/CD and test automation.
  • Run.sh logic matches the proven manual media-ctl/yavta flow, ensuring pipeline works even if entity order or naming varies.

Why

  • Existing logic was failing on device/sensor changes and required manual intervention.
  • Needed a robust, sensor-agnostic solution that can adapt to topology or driver changes.
  • Modularization enables reuse, easier maintenance, and faster debugging across the camera validation suite.

How

  • All core camera pipeline steps now use reusable shell/Python helpers.
  • Output is strictly structured for automated result parsing.
  • Format override and pad handling is correct for all known camera entity topologies.

Tested on:

  • Multiple RB3Gen2 devices and camera sensors, with verified frame capture and result emission.

@smuppand smuppand requested review from abbajaj806 and vnarapar August 8, 2025 11:21
@smuppand smuppand force-pushed the camera_yavta branch 2 times, most recently from ec0125a to 1a3f91f Compare August 8, 2025 11:32
IPA & RMNET tests: robust module handling, /dev node checks, dmesg scan, ShellCheck fixes
@smuppand
Copy link
Contributor Author

smuppand commented Aug 8, 2025

I need to move some functionality to another branch, so I'll create a new PR.

@smuppand smuppand closed this Aug 8, 2025
@smuppand smuppand deleted the camera_yavta branch September 12, 2025 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants