[libcamera-devel] [PATCH] libcamera: camera_sensor: Add Sony IMX327 sensor properties

[Kieran Bingham]

Quoting Benjamin Bara via libcamera-devel (2023-05-23 18:12:43)
> From: Benjamin Bara <benjamin.bara at skidata.com>
> 
> Copy the sensor properties from Sony IMX290, as the IMX327 is a variant
> of it.
> 
> Signed-off-by: Benjamin Bara <benjamin.bara at skidata.com>
> ---
> Hi,
> 
> I forgot to add the properties when adding the helper last week.
> These could also be squashed :)
> ---
>  src/libcamera/camera_sensor_properties.cpp | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/src/libcamera/camera_sensor_properties.cpp b/src/libcamera/camera_sensor_properties.cpp
> index fefb8afc..d35055d4 100644
> --- a/src/libcamera/camera_sensor_properties.cpp
> +++ b/src/libcamera/camera_sensor_properties.cpp
> @@ -103,6 +103,10 @@ const CameraSensorProperties *CameraSensorProperties::get(const std::string &sen
>                         .unitCellSize = { 2900, 2900 },
>                         .testPatternModes = {},
>                 } },
> +               { "imx327", {

I think we'd probably keep these in sort order based on the string here.
I can move it while applying.


Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

> +                       .unitCellSize = { 2900, 2900 },
> +                       .testPatternModes = {},
> +               } },
>                 { "imx296", {
>                         .unitCellSize = { 3450, 3450 },
>                         .testPatternModes = {},
> 
> ---
> base-commit: e8fccaea46b9e545282cd37d54b1acb168608a46
> change-id: 20230523-imx327-properties-61e7af857ba1
> 
> Best regards,
> -- 
> Benjamin Bara <benjamin.bara at skidata.com>
>