[PATCH v7 04/12] test: ipa_data_serialization: Use DebugMetadataEnable
Stefan Klug
stefan.klug at ideasonboard.com
Mon Jan 13 11:24:59 CET 2025
Hi Jacopo,
Thank you for the patch.
On Fri, Jan 10, 2025 at 05:57:29PM -0600, Paul Elder wrote:
> From: Jacopo Mondi <jacopo at jmondi.org>
>
> Replace the deprecated AeEnable control with DebugMetadataEnable
> in ipa_data_serialization test. We use DebugMetadataEnable instead of
> one of the controls replacing AeEnable as they are not boolean controls.
>
> Signed-off-by: Jacopo Mondi <jacopo at jmondi.org>
> Signed-off-by: Paul Elder <paul.elder at ideasonboard.com>
> Reviewed-by: Paul Elder <paul.elder at ideasonboard.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Stefan Klug <stefan.klug at ideasonboard.com>
Cheers,
Stefan
>
> ---
> No change in v7
>
> No change in v6
>
> No change in v5
>
> Changes in v4:
> - s/ExposureTimeMode/DebugMetadataEnable/
>
> No change in v3
> ---
> test/serialization/ipa_data_serializer_test.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/serialization/ipa_data_serializer_test.cpp b/test/serialization/ipa_data_serializer_test.cpp
> index aea63c731..afea93a6c 100644
> --- a/test/serialization/ipa_data_serializer_test.cpp
> +++ b/test/serialization/ipa_data_serializer_test.cpp
> @@ -29,7 +29,7 @@ using namespace std;
> using namespace libcamera;
>
> static const ControlInfoMap Controls = ControlInfoMap({
> - { &controls::AeEnable, ControlInfo(false, true) },
> + { &controls::DebugMetadataEnable, ControlInfo(false, true) },
> { &controls::ExposureTime, ControlInfo(0, 999999) },
> { &controls::AnalogueGain, ControlInfo(1.0f, 32.0f) },
> { &controls::ColourGains, ControlInfo(0.0f, 32.0f) },
> --
> 2.39.2
>
More information about the libcamera-devel
mailing list