[libcamera-devel] [PATCH v2 21/21] qcam: viewfinder: Add support for more native formats

[Laurent Pinchart]

Qt supports more 24-bit and 32-bit RGB formats for native painting.
Disable the converter and create a QImage in the appropriate format.

Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
---
Changes since v1:

- Rename formats to nativeFormats
---
 src/qcam/viewfinder.cpp | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/src/qcam/viewfinder.cpp b/src/qcam/viewfinder.cpp
index 45e226b58135..5aa0daebfb2d 100644
--- a/src/qcam/viewfinder.cpp
+++ b/src/qcam/viewfinder.cpp
@@ -7,16 +7,26 @@
 
 #include "viewfinder.h"
 
+#include <stdint.h>
 #include <utility>
 
 #include <QImage>
 #include <QImageWriter>
+#include <QMap>
 #include <QMutexLocker>
 #include <QPainter>
 #include <QtDebug>
 
 #include "format_converter.h"
 
+static const QMap<uint32_t, QImage::Format> nativeFormats{
+	{ DRM_FORMAT_ARGB8888, QImage::Format_RGB32 },
+	{ DRM_FORMAT_RGB888, QImage::Format_RGB888 },
+#if (QT_VERSION >= QT_VERSION_CHECK(5, 14, 0))
+	{ DRM_FORMAT_BGR888, QImage::Format_BGR888 },
+#endif
+};
+
 ViewFinder::ViewFinder(QWidget *parent)
 	: QWidget(parent), buffer_(nullptr)
 {
@@ -36,7 +46,7 @@ int ViewFinder::setFormat(const libcamera::PixelFormat &format,
 	 * If format conversion is needed, configure the converter and allocate
 	 * the destination image.
 	 */
-	if (format != DRM_FORMAT_ARGB8888) {
+	if (!nativeFormats.contains(format)) {
 		int ret = converter_.configure(format, size);
 		if (ret < 0)
 			return ret;
@@ -64,7 +74,7 @@ void ViewFinder::render(libcamera::FrameBuffer *buffer, MappedBuffer *map)
 	{
 		QMutexLocker locker(&mutex_);
 
-		if (format_ == DRM_FORMAT_ARGB8888) {
+		if (nativeFormats.contains(format_)) {
 			/*
 			 * If the frame format is identical to the display
 			 * format, create a QImage that references the frame
@@ -76,7 +86,8 @@ void ViewFinder::render(libcamera::FrameBuffer *buffer, MappedBuffer *map)
 			 * computing it naively
 			 */
 			image_ = QImage(memory, size_.width(), size_.height(),
-					size / size_.height(), QImage::Format_RGB32);
+					size / size_.height(),
+					nativeFormats[format_]);
 			std::swap(buffer, buffer_);
 		} else {
 			/*
-- 
Regards,

Laurent Pinchart