summaryrefslogtreecommitdiff
path: root/Documentation/thread-safety.dox
diff options
context:
space:
mode:
authorPaul Elder <paul.elder@ideasonboard.com>2025-01-13 15:59:43 -0600
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2025-01-20 21:36:40 +0200
commit85cb179f289d29d05babf7b2a0fdcfed5d94ba54 (patch)
treefafcdbc26e8889e10e185e75ae409a3018752b08 /Documentation/thread-safety.dox
parenta533bd004bcebcfa484bdb96ee905694237c9afa (diff)
controls: Redefine AeEnable
In the redesign of the AE-related controls, the AeEnable control was intended to be removed in favor of more specific sub-controls for analogue gain and exposure time. However this will cause problems if aperture sub-controls are introduced, and an application from a pre-aperture era uses a camera that supports aperture. If there is no AeEnable control, then a pre-aperture era application might set analogue gain and exposure time to manual, while aperture silently stays auto since that's the default mode. Thus aperture would be uncontrollable by the application. With an AeEnable control, then a pre-aperture era application can set AeEnable to manual, and under the hood all three of analogue gain and exposure time and aperture will be set to manual. The application won't be able to set the manual aperture, however. Although the above scenario is expected to be rare, the scenario with an AeEnable control seems less detrimental. With an AeEnable control at least the aperture would be static at a reasonably usable value, whereas without an AeEnable the aperture would be more-or-less uncontrolable and could go to extreme values as the AEGC algorithm tries to compensate for the manual analogue gain and exposure time values. Thus we redefine the AeEnable control, available only as a control and not in metadata. It will be preprocessed by the Camera class so that the relevant sub-controls are set. No pipline handler nor IPA shall act on the AeEnable control. The IPA still has to report the control as available, however. Signed-off-by: Paul Elder <paul.elder@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Stefan Klug <stefan.klug@ideasonboard.com> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Diffstat (limited to 'Documentation/thread-safety.dox')
0 files changed, 0 insertions, 0 deletions