[libcamera-devel] [PATCH] android: camera_capabilities: Add (1600x1200) and (1280x960) resolutions

[Jacopo Mondi]

Hi Han-Lin

On Tue, Apr 26, 2022 at 08:14:16PM +0800, Han-Lin Chen via libcamera-devel wrote:
> Although resolutions (1600x1200) and (1280x960) are not mandatory to be
> supported by the Android Camera3 specification, they are commonly used
> by Android devices as viewfinder streams for 4:3 still capture.
>
> Add them into stream resolution candidates.

Fine by me. I was concerned by very low-end devices that might not be
capable of producing those resolutions, but seems like there is no
enforcement on any resolution being mandatory when we build the
cameraResolutions list in initializeStreamConfigurations().

Reviewed-by: Jacopo Mondi <jacopo at jmondi.org>

Thanks
  j

>
> Signed-off-by: Han-Lin Chen <hanlinchen at chromium.org>
> ---
>  src/android/camera_capabilities.cpp | 15 +++++++++++----
>  1 file changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/src/android/camera_capabilities.cpp b/src/android/camera_capabilities.cpp
> index 5242055c..e0f23c87 100644
> --- a/src/android/camera_capabilities.cpp
> +++ b/src/android/camera_capabilities.cpp
> @@ -31,13 +31,20 @@ namespace {
>
>  /*
>   * \var camera3Resolutions
> - * \brief The list of image resolutions defined as mandatory to be supported by
> - * the Android Camera3 specification
> + * \brief The list of image resolutions commonly supported by Android
> + *
> + * The following are defined as mandatory to be supported by the Android
> + * Camera3 specification: (320x240), (640x480), (1280x720), (1920x1080).
> + *
> + * The following 4:3 resolutions are defined as optional, but commonly
> + * supported by Android devices: (1280x960), (1600x1200).
>   */
>  const std::vector<Size> camera3Resolutions = {
>  	{ 320, 240 },
>  	{ 640, 480 },
>  	{ 1280, 720 },
> +	{ 1280, 960 },
> +	{ 1600, 1200 },
>  	{ 1920, 1080 }
>  };
>
> @@ -492,8 +499,8 @@ int CameraCapabilities::initializeStreamConfigurations()
>  	/*
>  	 * Build the list of supported image resolutions.
>  	 *
> -	 * The resolutions listed in camera3Resolution are mandatory to be
> -	 * supported, up to the camera maximum resolution.
> +	 * The resolutions listed in camera3Resolution are to be supported,
> +	 * up to the camera maximum resolution.
>  	 *
>  	 * Augment the list by adding resolutions calculated from the camera
>  	 * maximum one.
> --
> 2.36.0.rc2.479.g8af0fa9b8e-goog
>