Skip to content

docs(accessories): use single-line command for HDMI audio capture test - #2132

Merged
Milir-Radxa merged 1 commit into
radxa-docs:mainfrom
tzz-radxa:docs/hdmi-in-4k-audio-single-line-20260920
Sep 20, 2026
Merged

Milir-Radxa merged 1 commit into
radxa-docs:mainfrom
tzz-radxa:docs/hdmi-in-4k-audio-single-line-20260920

Conversation

@tzz-radxa

Copy link
Copy Markdown
Contributor

What this PR does

Follow-up to #2130 (merged): the HDMI audio capture test on the HK628 page is now a single-line command that can be copied and run directly.

bash -c 'v4l2-ctl -d /dev/video0 --set-fmt-video=width=1920,height=1080,pixelformat=NV12 --stream-mmap=3 --stream-poll --stream-count=480 --stream-to=/dev/null >/dev/null 2>&1 & video_pid=$!; sleep 1; arecord -D hw:CARD=sai2capture,DEV=0 -f S16_LE -r 48000 -c 2 -d 5 -t wav rk628-audio.wav; wait "$video_pid"'

Previously the same steps were written as a multi-line shell snippet with \ continuations and a blank-line separated VIDEO_PID=$! / wait "$VIDEO_PID". Copy-pasting it was error-prone; the single-line form keeps the same behaviour (video pipeline stays alive while arecord records) but is safe to paste in one go.

The change is made in the shared template docs/common/accessories/_hdmi-in-to-mipi-csi2-usage.mdx, and the board name / video node / sound card are still template parameters, so other boards inherit it automatically.

Scope

Docs only — Chinese (docs/) and English (i18n/en/) updated together. Requested during review of the adapter usage tutorial.

Combine the video capture and audio recording steps into one
'bash -c' command so users can copy and run it directly, and
align the Chinese and English versions.
@tzz-radxa
tzz-radxa requested a review from a team as a code owner September 20, 2026 09:59
@Milir-Radxa
Milir-Radxa merged commit a987603 into radxa-docs:main Sep 20, 2026
5 of 6 checks passed
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