[libcamera-devel] [PATCH 10/15] libcamera: ipu3: Remove camera_ from IPU3CameraConfiguration

[Niklas Söderlund]

Hi Jacopo,

Thanks for your work.

On 2020-07-01 14:30:31 +0200, Jacopo Mondi wrote:
> The IPU3CameraConfiguration::camera_ shared pointer is not used.
> Remove it.
> 
> Signed-off-by: Jacopo Mondi <jacopo at jmondi.org>

Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

> ---
>  src/libcamera/pipeline/ipu3/ipu3.cpp | 14 +++-----------
>  1 file changed, 3 insertions(+), 11 deletions(-)
> 
> diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp
> index 4d70f9e3a30c..16bda97a3cc3 100644
> --- a/src/libcamera/pipeline/ipu3/ipu3.cpp
> +++ b/src/libcamera/pipeline/ipu3/ipu3.cpp
> @@ -59,19 +59,13 @@ public:
>  class IPU3CameraConfiguration : public CameraConfiguration
>  {
>  public:
> -	IPU3CameraConfiguration(Camera *camera, IPU3CameraData *data);
> +	IPU3CameraConfiguration(IPU3CameraData *data);
>  
>  	Status validate() override;
>  
>  	const StreamConfiguration &cio2Format() const { return cio2Configuration_; };
>  
>  private:
> -	/*
> -	 * The IPU3CameraData instance is guaranteed to be valid as long as the
> -	 * corresponding Camera instance is valid. In order to borrow a
> -	 * reference to the camera data, store a new reference to the camera.
> -	 */
> -	std::shared_ptr<Camera> camera_;
>  	const IPU3CameraData *data_;
>  
>  	StreamConfiguration cio2Configuration_;
> @@ -121,11 +115,9 @@ private:
>  	MediaDevice *imguMediaDev_;
>  };
>  
> -IPU3CameraConfiguration::IPU3CameraConfiguration(Camera *camera,
> -						 IPU3CameraData *data)
> +IPU3CameraConfiguration::IPU3CameraConfiguration(IPU3CameraData *data)
>  	: CameraConfiguration()
>  {
> -	camera_ = camera->shared_from_this();
>  	data_ = data;
>  }
>  
> @@ -234,7 +226,7 @@ CameraConfiguration *PipelineHandlerIPU3::generateConfiguration(Camera *camera,
>  								const StreamRoles &roles)
>  {
>  	IPU3CameraData *data = cameraData(camera);
> -	IPU3CameraConfiguration *config = new IPU3CameraConfiguration(camera, data);
> +	IPU3CameraConfiguration *config = new IPU3CameraConfiguration(data);
>  	CIO2Device *cio2 = &data->cio2_;
>  
>  	if (roles.empty())
> -- 
> 2.27.0
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund