[PATCH 3/3] ipa: rkisp1: cproc: Drop incorrect [[maybe_unused]]
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Thu Jun 13 13:00:28 CEST 2024
Hi Stefan,
Thank you for the patch.
On Thu, Jun 13, 2024 at 10:33:20AM +0200, Stefan Klug wrote:
> A few function parameters are marked with [[maybe_unused]] but are
> actually used. Drop the attribute.
>
> Signed-off-by: Stefan Klug <stefan.klug at ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
> src/ipa/rkisp1/algorithms/cproc.cpp | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/ipa/rkisp1/algorithms/cproc.cpp b/src/ipa/rkisp1/algorithms/cproc.cpp
> index 9f400a52dd96..ef0931b20453 100644
> --- a/src/ipa/rkisp1/algorithms/cproc.cpp
> +++ b/src/ipa/rkisp1/algorithms/cproc.cpp
> @@ -54,7 +54,7 @@ int convertContrastOrSaturation(const float v)
> /**
> * \copydoc libcamera::ipa::Algorithm::init
> */
> -int ColorProcessing::init([[maybe_unused]] IPAContext &context,
> +int ColorProcessing::init(IPAContext &context,
> [[maybe_unused]] const YamlObject &tuningData)
> {
> auto &cmap = context.ctrlMap;
> @@ -69,7 +69,7 @@ int ColorProcessing::init([[maybe_unused]] IPAContext &context,
> /**
> * \copydoc libcamera::ipa::Algorithm::configure
> */
> -int ColorProcessing::configure([[maybe_unused]] IPAContext &context,
> +int ColorProcessing::configure(IPAContext &context,
> [[maybe_unused]] const IPACameraSensorInfo &configInfo)
> {
> auto &cproc = context.activeState.cproc;
> @@ -85,7 +85,7 @@ int ColorProcessing::configure([[maybe_unused]] IPAContext &context,
> * \copydoc libcamera::ipa::Algorithm::queueRequest
> */
> void ColorProcessing::queueRequest(IPAContext &context,
> - [[maybe_unused]] const uint32_t frame,
> + const uint32_t frame,
> IPAFrameContext &frameContext,
> const ControlList &controls)
> {
> --
> 2.43.0
>
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list