[libcamera-devel] [PATCH 2/7] libcamera: camera_sensor: Set sensor model property

[Laurent Pinchart]

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.

> 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