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>

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 */

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;