-
Notifications
You must be signed in to change notification settings - Fork 2k
Updates for synchronous and externally triggered cameras #4089
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
If you use the |
It's a bit messy, but hard to make it simpler without omitting required information (indeed it does omit repeating the "timeout-defeat" instructions for GS external trigger, but presumably people will infer that it needs to be done as in the previous section). I feel this stuff would be better as a separate Application Note rather than part of the generic camera documentation? Conspicuous by is absence is any mention of the new S/W synchronization method! |
documentation/asciidoc/accessories/camera/external_trigger.adoc
Outdated
Show resolved
Hide resolved
documentation/asciidoc/accessories/camera/external_trigger.adoc
Outdated
Show resolved
Hide resolved
documentation/asciidoc/accessories/camera/external_trigger.adoc
Outdated
Show resolved
Hide resolved
documentation/asciidoc/accessories/camera/synchronous_cameras.adoc
Outdated
Show resolved
Hide resolved
documentation/asciidoc/accessories/camera/synchronous_cameras.adoc
Outdated
Show resolved
Hide resolved
documentation/asciidoc/accessories/camera/synchronous_cameras.adoc
Outdated
Show resolved
Hide resolved
documentation/asciidoc/accessories/camera/synchronous_cameras.adoc
Outdated
Show resolved
Hide resolved
documentation/asciidoc/accessories/camera/synchronous_cameras.adoc
Outdated
Show resolved
Hide resolved
documentation/asciidoc/accessories/camera/synchronous_cameras.adoc
Outdated
Show resolved
Hide resolved
Ah, I've belatedly noticed that we do already document Software Camera Synchronisation, although it's well hidden under rpicam-apps and could maybe do with being bumped a bit up the hierarchy. I should definitely link there from here. |
documentation/asciidoc/accessories/camera/synchronous_cameras.adoc
Outdated
Show resolved
Hide resolved
9a6431a
to
a504223
Compare
Update synchronized cameras documentation to: - Suggest software synchronisation instead - Deprecate the XVS sync method for GS Cam, in favour of XTRIG - Recommend "always-on" property instead of using pull-ups - Recommend that trigger-mode for IMX477 be set using the new dtoverlay parameters "sync-source" and "sync-sink", while noting that these parameters mean something quite different for IMX296. - Document the need to edit the pipeline config file for timeout - Some attempt to tidy the structure while covering all the options
a504223
to
abfe8ee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Update synchronized cameras documentation to:
Deprecate the XVS sync method for GS Cam, in favour of XTRIG
Recommend "always-on" property instead of using pull-ups
Recommend "trigger-mode" for IMX477 be set using the new OF property (better for source and sink on same Pi) while noting that the module parameter still works and that "trigger-mode" means something quite different for IMX296!
Document the need to edit the pipeline config file for timeout
Some attempt to tidy the structure while covering all the options
Remove $ prompts from scripts/commands, so they can be copy-pasted