| Message ID | 20210216122624.27940-2-dafna.hirschfeld@collabora.com |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series |
|
| Related | show |
Hi Dafna, On Tue, Feb 16, 2021 at 01:26:22PM +0100, Dafna Hirschfeld wrote: > Add a method 'hwRevision' to return the > info.hw_version reported by the driver. > > Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> Looks good to me. Reviewed-by: Paul Elder <paul.elder@ideasonboard.com> > --- > include/libcamera/internal/media_device.h | 2 ++ > src/libcamera/media_device.cpp | 1 + > 2 files changed, 3 insertions(+) > > diff --git a/include/libcamera/internal/media_device.h b/include/libcamera/internal/media_device.h > index d0459cde..c3292508 100644 > --- a/include/libcamera/internal/media_device.h > +++ b/include/libcamera/internal/media_device.h > @@ -41,6 +41,7 @@ public: > const std::string deviceNode() const { return deviceNode_; } > const std::string model() const { return model_; } > unsigned int version() const { return version_; } > + unsigned int hwRevision() const { return hwRevision_; } > > const std::vector<MediaEntity *> &entities() const { return entities_; } > MediaEntity *getEntityByName(const std::string &name) const; > @@ -79,6 +80,7 @@ private: > std::string deviceNode_; > std::string model_; > unsigned int version_; > + unsigned int hwRevision_; > > int fd_; > bool valid_; > diff --git a/src/libcamera/media_device.cpp b/src/libcamera/media_device.cpp > index de18d572..ec964948 100644 > --- a/src/libcamera/media_device.cpp > +++ b/src/libcamera/media_device.cpp > @@ -231,6 +231,7 @@ int MediaDevice::populate() > driver_ = info.driver; > model_ = info.model; > version_ = info.media_version; > + hwRevision_ = info.hw_revision; > > /* > * Keep calling G_TOPOLOGY until the version number stays stable. > -- > 2.17.1 >
diff --git a/include/libcamera/internal/media_device.h b/include/libcamera/internal/media_device.h index d0459cde..c3292508 100644 --- a/include/libcamera/internal/media_device.h +++ b/include/libcamera/internal/media_device.h @@ -41,6 +41,7 @@ public: const std::string deviceNode() const { return deviceNode_; } const std::string model() const { return model_; } unsigned int version() const { return version_; } + unsigned int hwRevision() const { return hwRevision_; } const std::vector<MediaEntity *> &entities() const { return entities_; } MediaEntity *getEntityByName(const std::string &name) const; @@ -79,6 +80,7 @@ private: std::string deviceNode_; std::string model_; unsigned int version_; + unsigned int hwRevision_; int fd_; bool valid_; diff --git a/src/libcamera/media_device.cpp b/src/libcamera/media_device.cpp index de18d572..ec964948 100644 --- a/src/libcamera/media_device.cpp +++ b/src/libcamera/media_device.cpp @@ -231,6 +231,7 @@ int MediaDevice::populate() driver_ = info.driver; model_ = info.model; version_ = info.media_version; + hwRevision_ = info.hw_revision; /* * Keep calling G_TOPOLOGY until the version number stays stable.
Add a method 'hwRevision' to return the info.hw_version reported by the driver. Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> --- include/libcamera/internal/media_device.h | 2 ++ src/libcamera/media_device.cpp | 1 + 2 files changed, 3 insertions(+)