summaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorMilan Zamazal <mzamazal@redhat.com>2025-03-19 10:55:33 +0100
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2025-04-01 10:47:39 +0300
commitceea066fa23c780eed65efbb243b216c7f511db8 (patch)
treeb0c6b379516f73d53e0e68ce1f6a2b997c447ed9 /include
parent485a807dcb5845a8438d3d84ce25bc38098785c4 (diff)
libcamera: software_isp: Reset stored exposure in black level
Automatic black level setting in software ISP updates the determined black level value when exposure or gain change. It stores the last exposure and gain values to detect the change. BlackLevel::configure() resets the stored black level value but not the exposure and gain values. This can prevent updating the black value and cause bad image output, e.g. after suspending and resuming a camera, if exposure and gain remain unchanged. Let's store exposure and gain in IPAActiveState. Although the values are not supposed to be used outside BlackLevel class, storing them in the context has the advantage of their automatic reset together with the other context contents and having them in `blc' struct indicates their relationship to the black value computation. Bug: https://bugs.libcamera.org/show_bug.cgi?id=259 Signed-off-by: Milan Zamazal <mzamazal@redhat.com> Tested-by: Robert Mader <robert.mader@collabora.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions