lc-compliance: Drop return value from SimpleCapture::stop()

The return value is never checked and serves to real usage, drop it.

Signed-off-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

2 files changed, 2 insertions, 4 deletions

diff --git a/src/lc-compliance/simple_capture.cpp b/src/lc-compliance/simple_capture.cpp
index 8f714d3b..811a6220 100644
--- a/src/lc-compliance/simple_capture.cpp
+++ b/src/lc-compliance/simple_capture.cpp
@@ -50,7 +50,7 @@ Results::Result SimpleCapture::start()
 	return { Results::Pass, "Started camera" };
 }
 
-Results::Result SimpleCapture::stop()
+void SimpleCapture::stop()
 {
 	Stream *stream = config_->at(0).stream();
 
@@ -59,8 +59,6 @@ Results::Result SimpleCapture::stop()
 	camera_->requestCompleted.disconnect(this, &SimpleCapture::requestComplete);
 
 	allocator_->free(stream);
-
-	return { Results::Pass, "Stopped camera" };
 }
 
 /* SimpleCaptureBalanced */
diff --git a/src/lc-compliance/simple_capture.h b/src/lc-compliance/simple_capture.h
index 4693c134..d9de53fb 100644
--- a/src/lc-compliance/simple_capture.h
+++ b/src/lc-compliance/simple_capture.h
@@ -24,7 +24,7 @@ protected:
 	virtual ~SimpleCapture();
 
 	Results::Result start();
-	Results::Result stop();
+	void stop();
 
 	virtual void requestComplete(libcamera::Request *request) = 0;