[libcamera-devel] [PATCH 10/11] test: ipa: ipa_interface: Pass configuration file to IPA init()

[Kieran Bingham]

Hi Laurent,

On 27/04/2020 04:17, Laurent Pinchart wrote:
> Pass the vimc IPA dummy configuration file to the IPA init() function.
> This will be used by the IPA to validate the init() call.

Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
>  test/ipa/ipa_interface_test.cpp | 11 ++++++++---
>  1 file changed, 8 insertions(+), 3 deletions(-)
> 
> diff --git a/test/ipa/ipa_interface_test.cpp b/test/ipa/ipa_interface_test.cpp
> index 2e2dfb8d1ebd..7fa88efee089 100644
> --- a/test/ipa/ipa_interface_test.cpp
> +++ b/test/ipa/ipa_interface_test.cpp
> @@ -98,8 +98,13 @@ protected:
>  		}
>  
>  		/* Test initialization of IPA module. */
> -		IPASettings settings;
> -		ipa_->init(settings);
> +		std::string conf = ipa_->configurationFile("vimc.conf");
> +		int ret = ipa_->init(IPASettings{ conf });
> +		if (ret < 0) {
> +			cerr << "IPA interface init() failed" << endl;
> +			return TestFail;
> +		}
> +
>  		timer.start(1000);
>  		while (timer.isRunning() && trace_ != IPAOperationInit)
>  			dispatcher->processEvents();
> @@ -155,7 +160,7 @@ private:
>  	}
>  
>  	std::shared_ptr<PipelineHandler> pipe_;
> -	std::unique_ptr<IPAInterface> ipa_;
> +	std::unique_ptr<IPAProxy> ipa_;
>  	enum IPAOperationCode trace_;
>  	EventNotifier *notifier_;
>  	int fd_;
> 

-- 
Regards
--
Kieran