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
CONFIG: Add CAMERA_CONTROL if PIN defined #12516
Conversation
This comment has been minimized.
This comment has been minimized.
3e0e5fd
to
0b2f294
Compare
This comment has been minimized.
This comment has been minimized.
AUTOMERGE: (FAIL)
|
src/main/target/common_post.h
Outdated
@@ -121,6 +121,10 @@ | |||
|
|||
#endif // defined(USE_RX_CC2500) | |||
|
|||
#if defined(CAMERA_CONTROL_PIN) && defined(USE_OSD_SD) && !defined(USE_CAMERA_CONTROL) |
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.
Curious as to why && defined(USE_OSD_SD)
is necessary?
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.
Added in last push as it would only be needed when using analog - so it should be USE_VTX instead?
0b2f294
to
9821bcb
Compare
Do you want to test this code? Here you have an automated build: |
As I understand it, the intent of this PR is that if the config.h file includes a defined camera control pin, and the default cloud build defines are used to build the firmware, then the firmware will contain the code to provide the camera control feature. That makes sense to me, especially since a manufacturer may otherwise get support requests about their camera control pin not working. Users of boards without defined camera control pins would have to manually enable the camera control build option, and probably would have to manually configure a suitable resource pin in the CLI to use for this purpose. |
No description provided.