Skip to content

Conversation

@Wolfleader101
Copy link
Contributor

@Wolfleader101 Wolfleader101 commented Oct 22, 2025

Change error logging from error to warning for invalid framesize and set framesize to FRAMESIZE_5MP if it exceeds the limit.

Description

OV5640 Sensors actually supporty up to 5MP (2592x1944) - Currently the code limits it FRAMESIZE_QSXGA, // 2560x1920
image

image

This change is same as the code in mega_ccm.c


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

Change error logging from error to warning for invalid framesize and set framesize to FRAMESIZE_5MP if it exceeds the limit.
@Wolfleader101 Wolfleader101 changed the title Update framesize validation to check FRAMESIZE_5MP fix: ov5640 framesize validation to check FRAMESIZE_5MP Oct 22, 2025
@me-no-dev me-no-dev merged commit 12c147f into espressif:master Nov 6, 2025
@Wolfleader101 Wolfleader101 deleted the fix/ov5640-max-res branch November 6, 2025 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants