[libcamera-devel] [PATCH v3 1/3] libcamera: controls: Define a default lens position behaviour

[Laurent Pinchart]

Hi Naush,

Thank you for the patch.

On Mon, Jun 05, 2023 at 10:14:04AM +0100, Naushir Patuck via libcamera-devel wrote:
> Update the AfMode control description to explicitly define a
> startup/default behaviour.
> 
> On startup, the camera will move the lens to the position given by the
> default value of the LensPosition control if operating in manual focus
> mode. Typically this would be the hyperfocal position of the lens.
> 
> Signed-off-by: Naushir Patuck <naush at raspberrypi.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> ---
>  src/libcamera/control_ids.yaml | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/src/libcamera/control_ids.yaml b/src/libcamera/control_ids.yaml
> index adea5f90acc5..765168e539a8 100644
> --- a/src/libcamera/control_ids.yaml
> +++ b/src/libcamera/control_ids.yaml
> @@ -408,6 +408,13 @@ controls:
>              LensPosition control.
>  
>              In this mode the AfState will always report AfStateIdle.
> +
> +            If the camera is started in AfModeManual, it will move the focus
> +            lens to the position specified by the LensPosition control.
> +
> +            This mode is the recommended default value for the AfMode control.
> +            External cameras (as reported by the Location property set to
> +            CameraLocationExternal) may use a different default value.
>          - name: AfModeAuto
>            value: 1
>            description: |

-- 
Regards,

Laurent Pinchart