adafruit / adafruit/Adafruit-VC0706-Serial-Camera-Library

the variable "baud", a uint16_t, is too small to hold values > 65535 preventing 115200 bps

Open
#16 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
139
Forks
108
PR merge metrics
No merged PRs in 30d

Description

the variable "baud", a uint16_t, is too small to hold values > 65535 preventing 115200 bps

because of that, a call to cam.begin(115200) will not correctly set the variable "baud".
The uint16_t variable "baud" will only hold values up to 65535 (2^16-1).
I recommend converting that variable to a uint32_t.

Now if only I could get the actual baud rate to change! The camera is slow at 38400.

Contributor guide

No contributing guide indexed for this repository

Research direction

Inspect the declaration and uses of the uint16_t variable "baud", especially the cam.begin(115200) path. Confirm that the value can represent 115200 and verify whether the camera's configured baud rate changes as expected.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
embedded-iot
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.