@@ -10,6 +10,7 @@
#include <memory>
#include <vector>
+#include <linux/drm_fourcc.h>
#include <linux/media-bus-format.h>
#include <libcamera/camera.h>
@@ -249,7 +250,7 @@ IPU3CameraConfiguration::IPU3CameraConfiguration(Camera *camera,
void IPU3CameraConfiguration::adjustStream(StreamConfiguration &cfg, bool scale)
{
/* The only pixel format the driver supports is NV12. */
- cfg.pixelFormat = V4L2_PIX_FMT_NV12;
+ cfg.pixelFormat = DRM_FORMAT_NV12;
if (scale) {
/*
@@ -404,7 +405,7 @@ CameraConfiguration *PipelineHandlerIPU3::generateConfiguration(Camera *camera,
StreamConfiguration cfg = {};
IPU3Stream *stream = nullptr;
- cfg.pixelFormat = V4L2_PIX_FMT_NV12;
+ cfg.pixelFormat = DRM_FORMAT_NV12;
switch (role) {
case StreamRole::StillCapture:
@@ -1121,7 +1122,7 @@ int ImgUDevice::configureOutput(ImgUOutput *output,
return 0;
V4L2DeviceFormat outputFormat = {};
- outputFormat.fourcc = V4L2_PIX_FMT_NV12;
+ outputFormat.fourcc = dev->toV4L2Fourcc(DRM_FORMAT_NV12);
outputFormat.size = cfg.size;
outputFormat.planesCount = 2;
@@ -11,6 +11,7 @@
#include <memory>
#include <queue>
+#include <linux/drm_fourcc.h>
#include <linux/media-bus-format.h>
#include <ipa/rkisp1.h>
@@ -434,14 +435,14 @@ RkISP1CameraConfiguration::RkISP1CameraConfiguration(Camera *camera,
CameraConfiguration::Status RkISP1CameraConfiguration::validate()
{
static const std::array<unsigned int, 8> formats{
- V4L2_PIX_FMT_YUYV,
- V4L2_PIX_FMT_YVYU,
- V4L2_PIX_FMT_VYUY,
- V4L2_PIX_FMT_NV16,
- V4L2_PIX_FMT_NV61,
- V4L2_PIX_FMT_NV21,
- V4L2_PIX_FMT_NV12,
- V4L2_PIX_FMT_GREY,
+ DRM_FORMAT_YUYV,
+ DRM_FORMAT_YVYU,
+ DRM_FORMAT_VYUY,
+ DRM_FORMAT_NV16,
+ DRM_FORMAT_NV61,
+ DRM_FORMAT_NV21,
+ DRM_FORMAT_NV12,
+ /* \todo Add support for 8-bit greyscale to DRM formats */
};
const CameraSensor *sensor = data_->sensor_;
@@ -462,7 +463,7 @@ CameraConfiguration::Status RkISP1CameraConfiguration::validate()
if (std::find(formats.begin(), formats.end(), cfg.pixelFormat) ==
formats.end()) {
LOG(RkISP1, Debug) << "Adjusting format to NV12";
- cfg.pixelFormat = V4L2_PIX_FMT_NV12;
+ cfg.pixelFormat = DRM_FORMAT_NV12,
status = Adjusted;
}
@@ -541,7 +542,7 @@ CameraConfiguration *PipelineHandlerRkISP1::generateConfiguration(Camera *camera
return config;
StreamConfiguration cfg{};
- cfg.pixelFormat = V4L2_PIX_FMT_NV12;
+ cfg.pixelFormat = DRM_FORMAT_NV12;
cfg.size = data->sensor_->resolution();
config->addConfiguration(cfg);
@@ -623,7 +624,7 @@ int PipelineHandlerRkISP1::configure(Camera *camera, CameraConfiguration *c)
LOG(RkISP1, Debug) << "ISP output pad configured with " << format.toString();
V4L2DeviceFormat outputFormat = {};
- outputFormat.fourcc = cfg.pixelFormat;
+ outputFormat.fourcc = video_->toV4L2Fourcc(cfg.pixelFormat);
outputFormat.size = cfg.size;
outputFormat.planesCount = 2;
@@ -632,7 +633,7 @@ int PipelineHandlerRkISP1::configure(Camera *camera, CameraConfiguration *c)
return ret;
if (outputFormat.size != cfg.size ||
- outputFormat.fourcc != cfg.pixelFormat) {
+ outputFormat.fourcc != video_->toV4L2Fourcc(cfg.pixelFormat)) {
LOG(RkISP1, Error)
<< "Unable to configure capture in " << cfg.toString();
return -EINVAL;
@@ -176,7 +176,7 @@ int PipelineHandlerUVC::configure(Camera *camera, CameraConfiguration *config)
int ret;
V4L2DeviceFormat format = {};
- format.fourcc = cfg.pixelFormat;
+ format.fourcc = data->video_->toV4L2Fourcc(cfg.pixelFormat);
format.size = cfg.size;
ret = data->video_->setFormat(&format);
@@ -184,7 +184,7 @@ int PipelineHandlerUVC::configure(Camera *camera, CameraConfiguration *config)
return ret;
if (format.size != cfg.size ||
- format.fourcc != cfg.pixelFormat)
+ format.fourcc != data->video_->toV4L2Fourcc(cfg.pixelFormat))
return -EINVAL;
cfg.setStream(&data->stream_);
@@ -10,6 +10,7 @@
#include <iomanip>
#include <tuple>
+#include <linux/drm_fourcc.h>
#include <linux/media-bus-format.h>
#include <ipa/ipa_interface.h>
@@ -106,9 +107,9 @@ private:
namespace {
constexpr std::array<unsigned int, 3> pixelformats{
- V4L2_PIX_FMT_BGR24,
- V4L2_PIX_FMT_RGB24,
- V4L2_PIX_FMT_ARGB32,
+ DRM_FORMAT_RGB888,
+ DRM_FORMAT_BGR888,
+ DRM_FORMAT_BGRA8888,
};
} /* namespace */
@@ -137,7 +138,7 @@ CameraConfiguration::Status VimcCameraConfiguration::validate()
if (std::find(pixelformats.begin(), pixelformats.end(), cfg.pixelFormat) ==
pixelformats.end()) {
LOG(VIMC, Debug) << "Adjusting format to RGB24";
- cfg.pixelFormat = V4L2_PIX_FMT_RGB24;
+ cfg.pixelFormat = DRM_FORMAT_BGR888;
status = Adjusted;
}
@@ -186,7 +187,7 @@ CameraConfiguration *PipelineHandlerVimc::generateConfiguration(Camera *camera,
StreamConfiguration cfg(formats.data());
- cfg.pixelFormat = V4L2_PIX_FMT_RGB24;
+ cfg.pixelFormat = DRM_FORMAT_BGR888;
cfg.size = { 1920, 1080 };
cfg.bufferCount = 4;
@@ -231,7 +232,7 @@ int PipelineHandlerVimc::configure(Camera *camera, CameraConfiguration *config)
return ret;
V4L2DeviceFormat format = {};
- format.fourcc = cfg.pixelFormat;
+ format.fourcc = data->video_->toV4L2Fourcc(cfg.pixelFormat);
format.size = cfg.size;
ret = data->video_->setFormat(&format);
@@ -239,7 +240,7 @@ int PipelineHandlerVimc::configure(Camera *camera, CameraConfiguration *config)
return ret;
if (format.size != cfg.size ||
- format.fourcc != cfg.pixelFormat)
+ format.fourcc != data->video_->toV4L2Fourcc(cfg.pixelFormat))
return -EINVAL;
/*
Use the newly defined PixelFormat, which uses the DRM_FORMAT_* fourcc for image formats provided to applications. Signed-off-by: Jacopo Mondi <jacopo@jmondi.org> --- src/libcamera/pipeline/ipu3/ipu3.cpp | 7 ++++--- src/libcamera/pipeline/rkisp1/rkisp1.cpp | 25 ++++++++++++------------ src/libcamera/pipeline/uvcvideo.cpp | 4 ++-- src/libcamera/pipeline/vimc.cpp | 15 +++++++------- 4 files changed, 27 insertions(+), 24 deletions(-)