[libcamera-devel] [PATCH] raspberrypi: Add the correct integer const postfix for FrameDurations
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Wed Jan 27 01:32:06 CET 2021
Hi Naush,
Thank you for the patch.
I've updated the subject to "ipa: raspberrypi: Cast FrameDurations
limits to 64-bit integer", and pushed the patch.
On Mon, Jan 25, 2021 at 08:19:32AM +0000, Naushir Patuck wrote:
> At startup, ControlInfoMap::generateIdmap() threw a log message warning
> that the controls::FrameDurations had a type mismatch based on the
> min/max values provided in libcamera::RPi::Controls initialiser.
>
> Fix this warning by forcing the integer constants to be 64-bit wide by
> using the INT64_C() macro.
>
> Signed-off-by: Naushir Patuck <naush at raspberrypi.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> Reviewed-by: Kieran Bingham <kieran.bingham at ideaasonboard.com>
> ---
> include/libcamera/ipa/raspberrypi.h | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/include/libcamera/ipa/raspberrypi.h b/include/libcamera/ipa/raspberrypi.h
> index 1de36039cee0..4038428bdba4 100644
> --- a/include/libcamera/ipa/raspberrypi.h
> +++ b/include/libcamera/ipa/raspberrypi.h
> @@ -7,6 +7,8 @@
> #ifndef __LIBCAMERA_IPA_INTERFACE_RASPBERRYPI_H__
> #define __LIBCAMERA_IPA_INTERFACE_RASPBERRYPI_H__
>
> +#include <stdint.h>
> +
> #include <libcamera/control_ids.h>
> #include <libcamera/controls.h>
>
> @@ -65,7 +67,7 @@ static const ControlInfoMap Controls = {
> { &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{}) },
> - { &controls::FrameDurations, ControlInfo(1000, 1000000000) },
> + { &controls::FrameDurations, ControlInfo(INT64_C(1000), INT64_C(1000000000)) },
> };
>
> } /* namespace RPi */
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list