Skip to content

Improve video sw generator #69817

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

Merged
merged 4 commits into from
Mar 15, 2024
Merged

Improve video sw generator #69817

merged 4 commits into from
Mar 15, 2024

Conversation

ngphibang
Copy link
Collaborator

This PR add support for XRGB32 format to the video sw generator as this is useful to test camera and display pipeline on i.MX RT11XX. It also contains some minor fixes.

The first commit (which adds XRGB fourcc) is part of another PR and could be dropped if that PR is merged first :

include: video: Add XRGB32 pixel format

@zephyrbot zephyrbot added the area: Video Video subsystem label Mar 5, 2024
@zephyrbot zephyrbot requested a review from loicpoulain March 5, 2024 15:38
VIDEO_PATTERN_FPS should be coherently used across the entire code file.

Signed-off-by: Phi Bang Nguyen <[email protected]>
Add XRGB32 pixel format

Signed-off-by: Phi Bang Nguyen <[email protected]>
Add support for XRGB32 pixel format as this is useful to test camera
and display drivers on i.MX RT11xx

Signed-off-by: Phi Bang Nguyen <[email protected]>
@ngphibang
Copy link
Collaborator Author

Hi @loicpoulain ,
Could you help to review this one ?
Thanks

loicpoulain
loicpoulain previously approved these changes Mar 14, 2024
Copy link
Collaborator

@loicpoulain loicpoulain left a comment

Choose a reason for hiding this comment

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

neat!

Add code to handle invalid formats when setting format

Signed-off-by: Phi Bang Nguyen <[email protected]>
@ngphibang
Copy link
Collaborator Author

@loicpoulain I made a mistake in the commit "drivers: video: sw_generator: Fix set_format

Instead of moving static const struct video_format_cap fmts[], I dupplicated it. I 've just pushed force to fix this.
Could you please approve the PR again ? Sorry for this !

@loicpoulain loicpoulain self-requested a review March 14, 2024 12:40
@dleach02 dleach02 self-assigned this Mar 14, 2024
@dleach02 dleach02 requested a review from danieldegrasse March 14, 2024 18:20
@dleach02 dleach02 assigned loicpoulain and unassigned dleach02 Mar 14, 2024
@dleach02
Copy link
Member

Added @loicpoulain as the assignee as he is the maintainer of the drivers\video

@danieldegrasse can you give this a review?

@aescolar aescolar merged commit 4b8d3db into zephyrproject-rtos:main Mar 15, 2024
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.

5 participants