[PATCH 1/7] libcamera: mali-c55: Limit max size to sensor resolution
Kieran Bingham
kieran.bingham at ideasonboard.com
Thu Jun 13 21:22:55 CEST 2024
Quoting Daniel Scally (2024-06-13 16:59:43)
> From: Jacopo Mondi <jacopo.mondi at ideasonboard.com>
>
> The Mali C55 ISP cannot upscale. The maximum allowed output size
> is the sensor's resolution.
>
> For RAW streams this is already handled in adjustRawSizes(), while
> for processed streams the maximum allowed resolution was wrongly
> set to the ISP maximum output size (8192x8192).
>
> Signed-off-by: Jacopo Mondi <jacopo.mondi at ideasonboard.com>
When posting a patch authored by someone else, you still need to add a
Signed-off-by: tag.
git-send-email -s does this for you I think.
> ---
> src/libcamera/pipeline/mali-c55/mali-c55.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/libcamera/pipeline/mali-c55/mali-c55.cpp b/src/libcamera/pipeline/mali-c55/mali-c55.cpp
> index 45c71c1d..9442d17c 100644
> --- a/src/libcamera/pipeline/mali-c55/mali-c55.cpp
> +++ b/src/libcamera/pipeline/mali-c55/mali-c55.cpp
> @@ -342,7 +342,7 @@ CameraConfiguration::Status MaliC55CameraConfiguration::validate()
> rawConfig = &config;
> }
>
given the commit message, this looks like a good place to say:
/*
* The C55 can not upscale. Limit the configuration to the ISP
* capabilities and the sensor resolution.
*/
But either way:
Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> - Size maxSize = kMaliC55MaxSize;
> + Size maxSize = std::min(kMaliC55MaxSize, data_->resolution());
> if (rawConfig) {
> /*
> * \todo Take into account the Bayer components ordering once
> --
> 2.30.2
>
More information about the libcamera-devel
mailing list