[libcamera-devel] [PATCH] libcamera: pipeline_handler: Fix typo in documentation

[Kieran Bingham]

Hi Laurent,

On 13/04/2021 19:12, Laurent Pinchart wrote:
> There's no CameraConfiguration::valid(), the correct function is
> CameraConfiguration::validate().
> 

Indeed.

Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>


> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
>  src/libcamera/pipeline_handler.cpp | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/libcamera/pipeline_handler.cpp b/src/libcamera/pipeline_handler.cpp
> index 433c05f60db0..03aa865f5b36 100644
> --- a/src/libcamera/pipeline_handler.cpp
> +++ b/src/libcamera/pipeline_handler.cpp
> @@ -293,8 +293,8 @@ const ControlList &PipelineHandler::properties(const Camera *camera) const
>   * application.
>   *
>   * The configuration is guaranteed to have been validated with
> - * CameraConfiguration::valid(). The pipeline handler implementation shall not
> - * perform further validation and may rely on any custom field stored in its
> + * CameraConfiguration::validate(). The pipeline handler implementation shall
> + * not perform further validation and may rely on any custom field stored in its
>   * custom CameraConfiguration derived class.
>   *
>   * When configuring the camera the pipeline handler shall associate a Stream
> 

-- 
Regards
--
Kieran