[libcamera-devel] [PATCH 2/7] libcamera: camera_sensor: Set sensor model property
Niklas Söderlund
niklas.soderlund at ragnatech.se
Sat Aug 8 23:02:16 CEST 2020
Hi Laurent,
Thanks for your feedback.
On 2020-08-08 23:20:25 +0300, Laurent Pinchart wrote:
> Hi Niklas,
>
> Thank you for the patch.
>
> On Thu, Aug 06, 2020 at 03:09:32PM +0200, Niklas Söderlund wrote:
> > Set the sensor model property.
>
> Patch 1/7 documents a camera model property, while this is a sensor
> model. All this needs to be clarified. Same for patch 4/7.
We already do this with Location and Rotation, what makes Model
different?
>
> > Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
> > ---
> > src/libcamera/camera_sensor.cpp | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/src/libcamera/camera_sensor.cpp b/src/libcamera/camera_sensor.cpp
> > index d2679a4b80ce0d42..731ccf29de0ad442 100644
> > --- a/src/libcamera/camera_sensor.cpp
> > +++ b/src/libcamera/camera_sensor.cpp
> > @@ -200,6 +200,8 @@ int CameraSensor::init()
> > else
> > model_ = entityName;
> >
> > + properties_.set(properties::Model, model_);
> > +
> > /* Create and open the subdev. */
> > subdev_ = std::make_unique<V4L2Subdevice>(entity_);
> > int ret = subdev_->open();
>
> --
> Regards,
>
> Laurent Pinchart
--
Regards,
Niklas Söderlund
More information about the libcamera-devel
mailing list