summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/libcamera/include/v4l2_subdevice.h4
-rw-r--r--src/libcamera/v4l2_subdevice.cpp4
-rw-r--r--test/v4l2_subdevice/list_formats.cpp6
3 files changed, 7 insertions, 7 deletions
diff --git a/src/libcamera/include/v4l2_subdevice.h b/src/libcamera/include/v4l2_subdevice.h
index 3ecf0851..f42cccab 100644
--- a/src/libcamera/include/v4l2_subdevice.h
+++ b/src/libcamera/include/v4l2_subdevice.h
@@ -37,8 +37,8 @@ public:
bool isOpen() const;
void close();
- std::string deviceNode() const { return entity_->deviceNode(); }
- std::string deviceName() const { return entity_->name(); }
+ const std::string &deviceNode() const { return entity_->deviceNode(); }
+ const std::string &entityName() const { return entity_->name(); }
int setCrop(unsigned int pad, Rectangle *rect);
int setCompose(unsigned int pad, Rectangle *rect);
diff --git a/src/libcamera/v4l2_subdevice.cpp b/src/libcamera/v4l2_subdevice.cpp
index 8260e6eb..10925f9f 100644
--- a/src/libcamera/v4l2_subdevice.cpp
+++ b/src/libcamera/v4l2_subdevice.cpp
@@ -166,7 +166,7 @@ void V4L2Subdevice::close()
*/
/**
- * \fn V4L2Subdevice::deviceName()
+ * \fn V4L2Subdevice::entityName()
* \brief Retrieve the name of the media entity associated with the subdevice
*
* \return The name of the media entity the subdevice is associated to
@@ -315,7 +315,7 @@ int V4L2Subdevice::setFormat(unsigned int pad, V4L2SubdeviceFormat *format)
std::string V4L2Subdevice::logPrefix() const
{
- return "'" + deviceName() + "'";
+ return "'" + entityName() + "'";
}
int V4L2Subdevice::enumPadSizes(unsigned int pad,unsigned int code,
diff --git a/test/v4l2_subdevice/list_formats.cpp b/test/v4l2_subdevice/list_formats.cpp
index bf859779..db486554 100644
--- a/test/v4l2_subdevice/list_formats.cpp
+++ b/test/v4l2_subdevice/list_formats.cpp
@@ -51,7 +51,7 @@ int ListFormatsTest::run()
formats = scaler_->formats(0);
if (formats.empty()) {
cerr << "Failed to list formats on pad 0 of subdevice "
- << scaler_->deviceName() << endl;
+ << scaler_->entityName() << endl;
return TestFail;
}
for (auto it = formats.begin(); it != formats.end(); ++it)
@@ -60,7 +60,7 @@ int ListFormatsTest::run()
formats = scaler_->formats(1);
if (formats.empty()) {
cerr << "Failed to list formats on pad 1 of subdevice "
- << scaler_->deviceName() << endl;
+ << scaler_->entityName() << endl;
return TestFail;
}
for (auto it = formats.begin(); it != formats.end(); ++it)
@@ -70,7 +70,7 @@ int ListFormatsTest::run()
formats = scaler_->formats(2);
if (!formats.empty()) {
cerr << "Listing formats on non-existing pad 2 of subdevice "
- << scaler_->deviceName()
+ << scaler_->entityName()
<< " should return an empty format list" << endl;
return TestFail;
}