[libcamera-devel] [PATCH 1/2] libcamera: camera_sensor: Add OV5675 sensor properties
Jacopo Mondi
jacopo at jmondi.org
Wed May 4 08:15:54 CEST 2022
Hi Quentin,
On Tue, May 03, 2022 at 05:57:24PM +0200, Quentin Schulz via libcamera-devel wrote:
> From: Quentin Schulz <quentin.schulz at theobroma-systems.com>
>
> Add an entry to the sensor properties for OmniVision OV5675. Only the
> first test pattern is included as the others that are exposed by the
> kernel aren't supported by libcamera control yet.
>
> Cc: Quentin Schulz <foss+libcamera at 0leil.net>
> Signed-off-by: Quentin Schulz <quentin.schulz at theobroma-systems.com>
Don't have a datasheet here, but the patch looks good.
Reviewed-by: Jacopo Mondi <jacopo at jmondi.org>
Looking at the driver these three additional test patterns are
available
"Top-Bottom Darker Color Bar",
"Right-Left Darker Color Bar",
"Bottom-Top Darker Color Bar"
I can see at least 4 other mainline drivers which features the same
test patterns, it might be worth adding them to the list of values
supported by the TestPatternMode control.
Thanks
j
> ---
> src/libcamera/camera_sensor_properties.cpp | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/src/libcamera/camera_sensor_properties.cpp b/src/libcamera/camera_sensor_properties.cpp
> index 7a349012..235edca1 100644
> --- a/src/libcamera/camera_sensor_properties.cpp
> +++ b/src/libcamera/camera_sensor_properties.cpp
> @@ -123,6 +123,13 @@ const CameraSensorProperties *CameraSensorProperties::get(const std::string &sen
> { controls::draft::TestPatternModeColorBars, 1 },
> },
> } },
> + { "ov5675", {
> + .unitCellSize = { 1120, 1120 },
> + .testPatternModes = {
> + { controls::draft::TestPatternModeOff, 0 },
> + { controls::draft::TestPatternModeColorBars, 1 },
> + },
> + } },
> { "ov5693", {
> .unitCellSize = { 1400, 1400 },
> .testPatternModes = {
> --
> 2.35.1
>
More information about the libcamera-devel
mailing list