Skip to content

Conversation

@trunghieulenxp
Copy link
Contributor

Do not enable the video sw generator if a real camera is present. This helps to save some spaces and to avoid unrelated code going into the built image.

If the real camera presents, there will be a warning which cannot be avoided:
`warning: VIDEO_SW_GENERATOR (defined at drivers/video/Kconfig.sw_generator:8) was assigned the value 'y' but got the value 'n'

@zephyrbot zephyrbot added the area: Video Video subsystem label Nov 21, 2024
Copy link
Contributor

@josuah josuah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes the software generator a default source that can be enabled by default by samples, since it goes a away when some board has a camera.

Great way to always make sure there is a "last resort" video source.

…sents

Do not enable the video sw generator if a real camera is present. This
helps to save some spaces and to avoid unrelated code going
into the built image.

Signed-off-by: Trung Hieu Le <[email protected]>
Signed-off-by: Phi Bang Nguyen <[email protected]>
Copy link
Contributor

@ngphibang ngphibang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commit titile should be changed to "drivers: video: sw_generator: Disable the driver if a real camera presents" as this affects only the video sw generator driver"

@trunghieulenxp trunghieulenxp force-pushed the fix/optimize_video_sw_generator_compilation branch from f397fe8 to 8319511 Compare November 21, 2024 16:06
@trunghieulenxp
Copy link
Contributor Author

Commit titile should be changed to "drivers: video: sw_generator: Disable the driver if a real camera presents" as this affects only the video sw generator driver"

Hi, I have modified it.

@kartben kartben merged commit 2ff984a into zephyrproject-rtos:main Nov 25, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Video Video subsystem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants