[libcamera-devel] [PATCH 1/2] py: gen-py-controls: Remove SceneFlicker workaround
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Tue Jul 25 15:28:43 CEST 2023
Hi Kieran,
Thank you for the patch.
On Tue, Jul 25, 2023 at 01:56:40PM +0100, Kieran Bingham via libcamera-devel wrote:
> The python bindings layer has to parse the libcamera controls to ensure
> that they are converted to suitable names for the python layer.
>
> Part of this strips out common prefixes from control names, however the
> SceneFlicker control would end up using an illegal name if processed in
> the same way as the other controls.
>
> The SceneFlicker control has now been removed as part of the
> introduction of the AeFlickerMode and AeFlickerPeriod controls.
>
> Remove the workaround in the python layer.
>
> Fixes: 6fdbf3f38c31 ("libcamera: controls: Add controls for AEC/AGC flicker avoidance")
> Signed-off-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
> src/py/libcamera/gen-py-controls.py | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/src/py/libcamera/gen-py-controls.py b/src/py/libcamera/gen-py-controls.py
> index 99f3bbcf5b80..9948c41e42b1 100755
> --- a/src/py/libcamera/gen-py-controls.py
> +++ b/src/py/libcamera/gen-py-controls.py
> @@ -48,9 +48,6 @@ def generate_py(controls, mode):
> # Adjustments for controls
> if name == 'LensShadingMapMode':
> prefix = 'LensShadingMapMode'
> - elif name == 'SceneFlicker':
> - # If we strip the prefix, we would get '50Hz', which is illegal name
> - prefix = ''
> else:
> prefix = find_common_prefix([e['name'] for e in enum])
> else:
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list