[libcamera-devel] [PATCH 1/3] libcamera: ipa_manager: Fix IPA module min/max version check

[paul.elder at ideasonboard.com]

Hi Laurent,

On Mon, Jul 12, 2021 at 02:15:45AM +0300, Laurent Pinchart wrote:
> The IPAManager::createIPA() function has its minVersion and maxVersion
> parameters inverted. This doesn't cause any issue at the moment as both
> the minimum and maximum version are set to 1 by all callers, but it's
> still a bug. Fix it.

Huh, idk why it was inverted in the first place :/

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

Reviewed-by: Paul Elder <paul.elder at ideasonboard.com>

> ---
>  include/libcamera/internal/ipa_manager.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/libcamera/internal/ipa_manager.h b/include/libcamera/internal/ipa_manager.h
> index 34224e330da7..42201839901b 100644
> --- a/include/libcamera/internal/ipa_manager.h
> +++ b/include/libcamera/internal/ipa_manager.h
> @@ -31,8 +31,8 @@ public:
>  
>  	template<typename T>
>  	static std::unique_ptr<T> createIPA(PipelineHandler *pipe,
> -					    uint32_t maxVersion,
> -					    uint32_t minVersion)
> +					    uint32_t minVersion,
> +					    uint32_t maxVersion)
>  	{
>  		IPAModule *m = nullptr;
>  
> -- 
> Regards,
> 
> Laurent Pinchart
>