diff options
| author | Guennadi Liakhovetski <g.liakhovetski@gmx.de> | 2013-04-04 11:49:13 -0300 | 
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2013-06-21 15:45:06 -0300 | 
| commit | 6b417c897000515e308ff45687319f8ede1ccdd2 (patch) | |
| tree | 0bd4fa6913dbd78b8214ca35d68aff811887186f | |
| parent | 663ccaf4cd61f510e8a8e9bb913d2ee9b1e94932 (diff) | |
[media] mx2-camera: move interface activation and deactivation to clock callbacks
When adding and removing a client, the mx2-camera driver only activates
and deactivates its camera interface respectively, which doesn't include
any client-specific actions. Move this functionality into .clock_start()
and .clock_stop() callbacks.
Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
| -rw-r--r-- | drivers/media/platform/soc_camera/mx2_camera.c | 28 | 
1 files changed, 18 insertions, 10 deletions
| diff --git a/drivers/media/platform/soc_camera/mx2_camera.c b/drivers/media/platform/soc_camera/mx2_camera.c index 772e0710f59b..45a0276be4e5 100644 --- a/drivers/media/platform/soc_camera/mx2_camera.c +++ b/drivers/media/platform/soc_camera/mx2_camera.c @@ -412,13 +412,26 @@ static void mx2_camera_deactivate(struct mx2_camera_dev *pcdev)  	writel(0, pcdev->base_emma + PRP_CNTL);  } +static int mx2_camera_add_device(struct soc_camera_device *icd) +{ +	dev_info(icd->parent, "Camera driver attached to camera %d\n", +		 icd->devnum); + +	return 0; +} + +static void mx2_camera_remove_device(struct soc_camera_device *icd) +{ +	dev_info(icd->parent, "Camera driver detached from camera %d\n", +		 icd->devnum); +} +  /*   * The following two functions absolutely depend on the fact, that   * there can be only one camera on mx2 camera sensor interface   */ -static int mx2_camera_add_device(struct soc_camera_device *icd) +static int mx2_camera_clock_start(struct soc_camera_host *ici)  { -	struct soc_camera_host *ici = to_soc_camera_host(icd->parent);  	struct mx2_camera_dev *pcdev = ici->priv;  	int ret;  	u32 csicr1; @@ -439,9 +452,6 @@ static int mx2_camera_add_device(struct soc_camera_device *icd)  	pcdev->frame_count = 0; -	dev_info(icd->parent, "Camera driver attached to camera %d\n", -		 icd->devnum); -  	return 0;  exit_csi_ahb: @@ -450,14 +460,10 @@ exit_csi_ahb:  	return ret;  } -static void mx2_camera_remove_device(struct soc_camera_device *icd) +static void mx2_camera_clock_stop(struct soc_camera_host *ici)  { -	struct soc_camera_host *ici = to_soc_camera_host(icd->parent);  	struct mx2_camera_dev *pcdev = ici->priv; -	dev_info(icd->parent, "Camera driver detached from camera %d\n", -		 icd->devnum); -  	mx2_camera_deactivate(pcdev);  } @@ -1271,6 +1277,8 @@ static struct soc_camera_host_ops mx2_soc_camera_host_ops = {  	.owner		= THIS_MODULE,  	.add		= mx2_camera_add_device,  	.remove		= mx2_camera_remove_device, +	.clock_start	= mx2_camera_clock_start, +	.clock_stop	= mx2_camera_clock_stop,  	.set_fmt	= mx2_camera_set_fmt,  	.set_crop	= mx2_camera_set_crop,  	.get_formats	= mx2_camera_get_formats, | 
