[libcamera-devel] [PATCH v2 01/13] libcamera: pipeline: rkisp1: Set number of planes based on format
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Tue Sep 15 02:27:10 CEST 2020
Hi Niklas,
Thank you for the patch.
On Mon, Sep 14, 2020 at 04:21:37PM +0200, Niklas Söderlund wrote:
> The RkISP1 pipeline originally only supported NV formats which have 2
> planes. When support for YUV formats was added the plane count on the
> output format was not made to reflect this. Instead of hard coding the
> plane count to 2 fetch the number of planes from the format information.
>
> Reported-by: Jacopo Mondi <jacopo at jmondi.org>
> Fixes: 2b1a908b5222e263 ("libcamera: camera: Add a validation API to the CameraConfiguration class")
> Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
> src/libcamera/pipeline/rkisp1/rkisp1.cpp | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/libcamera/pipeline/rkisp1/rkisp1.cpp b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> index a6fc3b8e36f3b00a..009d190d3ec828f0 100644
> --- a/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> +++ b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> @@ -667,10 +667,11 @@ int PipelineHandlerRkISP1::configure(Camera *camera, CameraConfiguration *c)
>
> LOG(RkISP1, Debug) << "Resizer output pad configured with " << format.toString();
>
> + const PixelFormatInfo &info = PixelFormatInfo::info(cfg.pixelFormat);
> V4L2DeviceFormat outputFormat = {};
> outputFormat.fourcc = video_->toV4L2PixelFormat(cfg.pixelFormat);
> outputFormat.size = cfg.size;
> - outputFormat.planesCount = 2;
> + outputFormat.planesCount = info.numPlanes();
>
> ret = video_->setFormat(&outputFormat);
> if (ret)
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list