Skip to content
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

OV2640: Brightness, contrast + saturation not working anymore #3381

Closed
Slider0007 opened this issue Nov 11, 2024 · 4 comments · Fixed by #3417
Closed

OV2640: Brightness, contrast + saturation not working anymore #3381

Slider0007 opened this issue Nov 11, 2024 · 4 comments · Fixed by #3417
Assignees
Labels
bug Something isn't working Release16.0.0

Comments

@Slider0007
Copy link
Collaborator

Slider0007 commented Nov 11, 2024

The Problem

OV2640: Brightness, contrast and saturation parameter are not functional anymore.

This workaround to enable functionality is somewhere lost between 15.7 and 16.0:

/* Workaround - bug in cam library - enable bits are set without using bitwise OR logic -> only latest enable setting is used */

Workaround is still required for OV2640 camera even with newest camera driver.
For OV5640 it's not required, it's functional without this workaround.

Version

16.0.0RC4

Logfile

n.a.

Expected Behavior

No response

Screenshots

No response

Additional Context

No response

@Slider0007 Slider0007 added the bug Something isn't working label Nov 11, 2024
@Slider0007
Copy link
Collaborator Author

@SybexX: FYI

@Slider0007 Slider0007 changed the title OV2640: Contrast + Saturation not working anymore OV2640: Brightness, contrast + saturation not working anymore Nov 11, 2024
@caco3
Copy link
Collaborator

caco3 commented Dec 1, 2024

@SybexX or @Slider0007 Does one of you have such HW and can add and test a fix?

@SybexX
Copy link
Collaborator

SybexX commented Dec 2, 2024

Please test whether this fixes the problem (no changes are necessary for the OV5640): https://github.com/SybexX/AI-on-the-edge-device/actions/runs/12131765400

@Slider0007
Copy link
Collaborator Author

Slider0007 commented Dec 2, 2024

Using the provided test firmware, brightness, contrast and saturation adaption is working again with OV2640.

Be ware: I've only tested the part which is related to this issue and not the additional changes which are also included in this branch. (I'm not sure if this branch is intended to be merged directly or split up again to only address this issue here. IMHO it usually doesn't make sense to test a branch which contains multiple topics.)

@Slider0007 Slider0007 removed their assignment Dec 2, 2024
@caco3 caco3 linked a pull request Dec 3, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Release16.0.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants