[PATCH v4 3/9] libcamera: software_isp: lut: Remove maybe_unused on a used argument

Laurent Pinchart laurent.pinchart at ideasonboard.com
Mon Jan 27 00:04:33 CET 2025


Hi Milan,

Thank you for the patch.

On Mon, Jan 13, 2025 at 02:51:00PM +0100, Milan Zamazal wrote:
> `params' argument of Lut::prepare is actually used, let's remove
> maybe_unused from it.
> 
> Signed-off-by: Milan Zamazal <mzamazal at redhat.com>
> Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> ---
>  src/ipa/simple/algorithms/lut.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/ipa/simple/algorithms/lut.cpp b/src/ipa/simple/algorithms/lut.cpp
> index d75ff710..243f0818 100644
> --- a/src/ipa/simple/algorithms/lut.cpp
> +++ b/src/ipa/simple/algorithms/lut.cpp
> @@ -83,7 +83,7 @@ void Lut::updateGammaTable(IPAContext &context)
>  void Lut::prepare(IPAContext &context,
>  		  [[maybe_unused]] const uint32_t frame,
>  		  [[maybe_unused]] IPAFrameContext &frameContext,
> -		  [[maybe_unused]] DebayerParams *params)
> +		  DebayerParams *params)
>  {
>  	/*
>  	 * Update the gamma table if needed. This means if black level changes

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list