[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