[libcamera-devel] [PATCH] libcamera: pipeline: rkisp1: don't fail if sensorInfo fails

[Laurent Pinchart]

Hi Dafna,

Thank you for the patch.

On Wed, Apr 29, 2020 at 11:44:50PM +0200, Dafna Hirschfeld wrote:
> When starting the streaming, it might be that the call
> to sensorInfo fails. This does not mean that the camera can't
> stream so set 'ret' to 0

Oops :-/ I've added

Fixes: fd554f9dba31 ("libcamera: ipa: Add support for CameraSensorInfo")

(and a period after the last sentence of the commit message) and pushed
the patch. Thank you.

> Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld at collabora.com>
> ---
>  src/libcamera/pipeline/rkisp1/rkisp1.cpp | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/libcamera/pipeline/rkisp1/rkisp1.cpp b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> index 1a34ffe..6aa3178 100644
> --- a/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> +++ b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> @@ -828,6 +828,7 @@ int PipelineHandlerRkISP1::start(Camera *camera)
>  		/* \todo Turn this in an hard failure. */
>  		LOG(RkISP1, Warning) << "Camera sensor information not available";
>  		sensorInfo = {};
> +		ret = 0;
>  	}
>  
>  	std::map<unsigned int, IPAStream> streamConfig;

-- 
Regards,

Laurent Pinchart