[libcamera-devel] [PATCH v2 3/5] libcamera: camera_sensor: Register ColorFilterArrangement
paul.elder at ideasonboard.com
paul.elder at ideasonboard.com
Tue Dec 29 05:50:02 CET 2020
Hi Jacopo,
On Mon, Dec 28, 2020 at 05:40:01PM +0100, Jacopo Mondi wrote:
> Inspect the list of media bus codes supported by the camera sensor
> in order to deduce the color filter arrangement and register the
> ColorFilterArrangement draft property.
>
> Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> Signed-off-by: Jacopo Mondi <jacopo at jmondi.org>
Reviewed-by: Paul Elder <paul.elder at ideasonboard.com>
> ---
> src/libcamera/camera_sensor.cpp | 36 +++++++++++++++++++++++++++++----
> 1 file changed, 32 insertions(+), 4 deletions(-)
>
> diff --git a/src/libcamera/camera_sensor.cpp b/src/libcamera/camera_sensor.cpp
> index 1628ba9c892b..e786821d4ba2 100644
> --- a/src/libcamera/camera_sensor.cpp
> +++ b/src/libcamera/camera_sensor.cpp
> @@ -17,6 +17,7 @@
>
> #include <libcamera/property_ids.h>
>
> +#include "libcamera/internal/bayer_format.h"
> #include "libcamera/internal/formats.h"
> #include "libcamera/internal/sysfs.h"
> #include "libcamera/internal/utils.h"
> @@ -178,10 +179,6 @@ int CameraSensor::init()
> if (ret < 0)
> return ret;
>
> - ret = initProperties();
> - if (ret)
> - return ret;
> -
> /* Enumerate, sort and cache media bus codes and sizes. */
> formats_ = subdev_->formats(pad_);
> if (formats_.empty()) {
> @@ -210,6 +207,10 @@ int CameraSensor::init()
> */
> resolution_ = sizes_.back();
>
> + ret = initProperties();
> + if (ret)
> + return ret;
> +
> return 0;
> }
>
> @@ -307,6 +308,33 @@ int CameraSensor::initProperties()
> properties_.set(properties::PixelArrayActiveAreas, { crop });
> }
>
> + /* Color filter array pattern, register only for RAW sensors. */
> + for (const auto &format : formats_) {
> + unsigned int mbusCode = format.first;
> + BayerFormat bayerFormat = BayerFormat::fromMbusCode(mbusCode);
> + if (!bayerFormat.isValid())
> + continue;
> +
> + int32_t cfa;
> + switch (bayerFormat.order) {
> + case BayerFormat::BGGR:
> + cfa = properties::draft::BGGR;
> + break;
> + case BayerFormat::GBRG:
> + cfa = properties::draft::GBRG;
> + break;
> + case BayerFormat::GRBG:
> + cfa = properties::draft::GRBG;
> + break;
> + case BayerFormat::RGGB:
> + cfa = properties::draft::RGGB;
> + break;
> + }
> +
> + properties_.set(properties::draft::ColorFilterArrangement, cfa);
> + break;
> + }
> +
> return 0;
> }
>
> --
> 2.29.2
>
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel
More information about the libcamera-devel
mailing list