[libcamera-devel] [PATCH] ipa: raspberrypi: Correct some of the ControlInfo ranges and defaults
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Wed Apr 6 01:12:36 CEST 2022
On Sat, Apr 02, 2022 at 02:21:25PM +0100, Kieran Bingham via libcamera-devel wrote:
> Quoting David Plowman via libcamera-devel (2022-03-28 11:05:44)
> > Some of the values were listed incorrectly. Specifically:
> >
> > ExposureValue: the range is now centred correctly on zero
> > Brightness: the default value (0.0) is made explicit
> > Contrast: the default value is corrected to be 1.0
> > Saturation: the default value is corrected to be 1.0
>
> Looks reasonable to me - but I think a review from Naush is probably
> more important than me.
Naush, do you plan to review this ? Otherwise I'll merge it as-is.
> Either way:
>
> Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
>
> > Signed-off-by: David Plowman <david.plowman at raspberrypi.com>
> > ---
> > include/libcamera/ipa/raspberrypi.h | 8 ++++----
> > 1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/include/libcamera/ipa/raspberrypi.h b/include/libcamera/ipa/raspberrypi.h
> > index 7f705e49..6a56b008 100644
> > --- a/include/libcamera/ipa/raspberrypi.h
> > +++ b/include/libcamera/ipa/raspberrypi.h
> > @@ -34,13 +34,13 @@ static const ControlInfoMap Controls({
> > { &controls::AeMeteringMode, ControlInfo(controls::AeMeteringModeValues) },
> > { &controls::AeConstraintMode, ControlInfo(controls::AeConstraintModeValues) },
> > { &controls::AeExposureMode, ControlInfo(controls::AeExposureModeValues) },
> > - { &controls::ExposureValue, ControlInfo(0.0f, 16.0f) },
> > + { &controls::ExposureValue, ControlInfo(-8.0f, 8.0f, 0.0f) },
> > { &controls::AwbEnable, ControlInfo(false, true) },
> > { &controls::ColourGains, ControlInfo(0.0f, 32.0f) },
> > { &controls::AwbMode, ControlInfo(controls::AwbModeValues) },
> > - { &controls::Brightness, ControlInfo(-1.0f, 1.0f) },
> > - { &controls::Contrast, ControlInfo(0.0f, 32.0f) },
> > - { &controls::Saturation, ControlInfo(0.0f, 32.0f) },
> > + { &controls::Brightness, ControlInfo(-1.0f, 1.0f, 0.0f) },
> > + { &controls::Contrast, ControlInfo(0.0f, 32.0f, 1.0f) },
> > + { &controls::Saturation, ControlInfo(0.0f, 32.0f, 1.0f) },
> > { &controls::Sharpness, ControlInfo(0.0f, 16.0f, 1.0f) },
> > { &controls::ColourCorrectionMatrix, ControlInfo(-16.0f, 16.0f) },
> > { &controls::ScalerCrop, ControlInfo(Rectangle{}, Rectangle(65535, 65535, 65535, 65535), Rectangle{}) },
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list