linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] media: video: do not clear 'driver' from an i2c_client
@ 2010-11-10 14:14 Wolfram Sang
  2010-11-10 14:24 ` Guennadi Liakhovetski
  2010-11-15 22:28 ` Wolfram Sang
  0 siblings, 2 replies; 4+ messages in thread
From: Wolfram Sang @ 2010-11-10 14:14 UTC (permalink / raw)
  To: linux-i2c
  Cc: Wolfram Sang, Mauro Carvalho Chehab, Guennadi Liakhovetski,
	Hans Verkuil, Sergio Aguirre, Michael Grzeschik, Philipp Wiesner,
	Márton Németh, linux-media, linux-kernel

The i2c-core does this already.

Reported-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
---

Not sure if this should go via i2c or media?

 drivers/media/video/imx074.c     |    1 -
 drivers/media/video/mt9m001.c    |    1 -
 drivers/media/video/mt9m111.c    |    1 -
 drivers/media/video/mt9t031.c    |    1 -
 drivers/media/video/mt9v022.c    |    1 -
 drivers/media/video/rj54n1cb0c.c |    1 -
 6 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/drivers/media/video/imx074.c b/drivers/media/video/imx074.c
index 27b5dfd..1a11691 100644
--- a/drivers/media/video/imx074.c
+++ b/drivers/media/video/imx074.c
@@ -467,7 +467,6 @@ static int imx074_remove(struct i2c_client *client)
 	icd->ops = NULL;
 	if (icl->free_bus)
 		icl->free_bus(icl);
-	client->driver = NULL;
 	kfree(priv);
 
 	return 0;
diff --git a/drivers/media/video/mt9m001.c b/drivers/media/video/mt9m001.c
index fcb4cd9..f7fc88d 100644
--- a/drivers/media/video/mt9m001.c
+++ b/drivers/media/video/mt9m001.c
@@ -798,7 +798,6 @@ static int mt9m001_remove(struct i2c_client *client)
 
 	icd->ops = NULL;
 	mt9m001_video_remove(icd);
-	client->driver = NULL;
 	kfree(mt9m001);
 
 	return 0;
diff --git a/drivers/media/video/mt9m111.c b/drivers/media/video/mt9m111.c
index 525a16e..53fa2a7 100644
--- a/drivers/media/video/mt9m111.c
+++ b/drivers/media/video/mt9m111.c
@@ -1092,7 +1092,6 @@ static int mt9m111_remove(struct i2c_client *client)
 	struct soc_camera_device *icd = client->dev.platform_data;
 
 	icd->ops = NULL;
-	client->driver = NULL;
 	kfree(mt9m111);
 
 	return 0;
diff --git a/drivers/media/video/mt9t031.c b/drivers/media/video/mt9t031.c
index 9bd44a8..7ce279c 100644
--- a/drivers/media/video/mt9t031.c
+++ b/drivers/media/video/mt9t031.c
@@ -896,7 +896,6 @@ static int mt9t031_remove(struct i2c_client *client)
 
 	if (icd)
 		icd->ops = NULL;
-	client->driver = NULL;
 	kfree(mt9t031);
 
 	return 0;
diff --git a/drivers/media/video/mt9v022.c b/drivers/media/video/mt9v022.c
index b96171c..6a784c8 100644
--- a/drivers/media/video/mt9v022.c
+++ b/drivers/media/video/mt9v022.c
@@ -930,7 +930,6 @@ static int mt9v022_remove(struct i2c_client *client)
 
 	icd->ops = NULL;
 	mt9v022_video_remove(icd);
-	client->driver = NULL;
 	kfree(mt9v022);
 
 	return 0;
diff --git a/drivers/media/video/rj54n1cb0c.c b/drivers/media/video/rj54n1cb0c.c
index d2fa2d4..57e11b6 100644
--- a/drivers/media/video/rj54n1cb0c.c
+++ b/drivers/media/video/rj54n1cb0c.c
@@ -1460,7 +1460,6 @@ static int rj54n1_remove(struct i2c_client *client)
 	icd->ops = NULL;
 	if (icl->free_bus)
 		icl->free_bus(icl);
-	client->driver = NULL;
 	kfree(rj54n1);
 
 	return 0;
-- 
1.7.2.3


^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [PATCH] media: video: do not clear 'driver' from an i2c_client
  2010-11-10 14:14 [PATCH] media: video: do not clear 'driver' from an i2c_client Wolfram Sang
@ 2010-11-10 14:24 ` Guennadi Liakhovetski
  2010-11-15 22:28 ` Wolfram Sang
  1 sibling, 0 replies; 4+ messages in thread
From: Guennadi Liakhovetski @ 2010-11-10 14:24 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-i2c, Mauro Carvalho Chehab, Hans Verkuil, Sergio Aguirre,
	Michael Grzeschik, Philipp Wiesner, Márton Németh,
	linux-media, linux-kernel

On Wed, 10 Nov 2010, Wolfram Sang wrote:

> The i2c-core does this already.
> 
> Reported-by: Jean Delvare <khali@linux-fr.org>
> Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>

Acked-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>

> ---
> 
> Not sure if this should go via i2c or media?
> 
>  drivers/media/video/imx074.c     |    1 -
>  drivers/media/video/mt9m001.c    |    1 -
>  drivers/media/video/mt9m111.c    |    1 -
>  drivers/media/video/mt9t031.c    |    1 -
>  drivers/media/video/mt9v022.c    |    1 -
>  drivers/media/video/rj54n1cb0c.c |    1 -
>  6 files changed, 0 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/media/video/imx074.c b/drivers/media/video/imx074.c
> index 27b5dfd..1a11691 100644
> --- a/drivers/media/video/imx074.c
> +++ b/drivers/media/video/imx074.c
> @@ -467,7 +467,6 @@ static int imx074_remove(struct i2c_client *client)
>  	icd->ops = NULL;
>  	if (icl->free_bus)
>  		icl->free_bus(icl);
> -	client->driver = NULL;
>  	kfree(priv);
>  
>  	return 0;
> diff --git a/drivers/media/video/mt9m001.c b/drivers/media/video/mt9m001.c
> index fcb4cd9..f7fc88d 100644
> --- a/drivers/media/video/mt9m001.c
> +++ b/drivers/media/video/mt9m001.c
> @@ -798,7 +798,6 @@ static int mt9m001_remove(struct i2c_client *client)
>  
>  	icd->ops = NULL;
>  	mt9m001_video_remove(icd);
> -	client->driver = NULL;
>  	kfree(mt9m001);
>  
>  	return 0;
> diff --git a/drivers/media/video/mt9m111.c b/drivers/media/video/mt9m111.c
> index 525a16e..53fa2a7 100644
> --- a/drivers/media/video/mt9m111.c
> +++ b/drivers/media/video/mt9m111.c
> @@ -1092,7 +1092,6 @@ static int mt9m111_remove(struct i2c_client *client)
>  	struct soc_camera_device *icd = client->dev.platform_data;
>  
>  	icd->ops = NULL;
> -	client->driver = NULL;
>  	kfree(mt9m111);
>  
>  	return 0;
> diff --git a/drivers/media/video/mt9t031.c b/drivers/media/video/mt9t031.c
> index 9bd44a8..7ce279c 100644
> --- a/drivers/media/video/mt9t031.c
> +++ b/drivers/media/video/mt9t031.c
> @@ -896,7 +896,6 @@ static int mt9t031_remove(struct i2c_client *client)
>  
>  	if (icd)
>  		icd->ops = NULL;
> -	client->driver = NULL;
>  	kfree(mt9t031);
>  
>  	return 0;
> diff --git a/drivers/media/video/mt9v022.c b/drivers/media/video/mt9v022.c
> index b96171c..6a784c8 100644
> --- a/drivers/media/video/mt9v022.c
> +++ b/drivers/media/video/mt9v022.c
> @@ -930,7 +930,6 @@ static int mt9v022_remove(struct i2c_client *client)
>  
>  	icd->ops = NULL;
>  	mt9v022_video_remove(icd);
> -	client->driver = NULL;
>  	kfree(mt9v022);
>  
>  	return 0;
> diff --git a/drivers/media/video/rj54n1cb0c.c b/drivers/media/video/rj54n1cb0c.c
> index d2fa2d4..57e11b6 100644
> --- a/drivers/media/video/rj54n1cb0c.c
> +++ b/drivers/media/video/rj54n1cb0c.c
> @@ -1460,7 +1460,6 @@ static int rj54n1_remove(struct i2c_client *client)
>  	icd->ops = NULL;
>  	if (icl->free_bus)
>  		icl->free_bus(icl);
> -	client->driver = NULL;
>  	kfree(rj54n1);
>  
>  	return 0;
> -- 
> 1.7.2.3
> 
> 

---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [PATCH] media: video: do not clear 'driver' from an i2c_client
  2010-11-10 14:14 [PATCH] media: video: do not clear 'driver' from an i2c_client Wolfram Sang
  2010-11-10 14:24 ` Guennadi Liakhovetski
@ 2010-11-15 22:28 ` Wolfram Sang
  2010-11-16 10:34   ` Jean Delvare
  1 sibling, 1 reply; 4+ messages in thread
From: Wolfram Sang @ 2010-11-15 22:28 UTC (permalink / raw)
  To: linux-i2c
  Cc: Mauro Carvalho Chehab, Guennadi Liakhovetski, Hans Verkuil,
	Sergio Aguirre, Michael Grzeschik, Philipp Wiesner,
	Márton Németh, linux-media, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 572 bytes --]

On Wed, Nov 10, 2010 at 03:14:13PM +0100, Wolfram Sang wrote:
> The i2c-core does this already.
> 
> Reported-by: Jean Delvare <khali@linux-fr.org>
> Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
> ---
> 
> Not sure if this should go via i2c or media?

Okay, as Jean did not pick it up in his latest pull request, I guess this means
it shall go via the media-tree? :) Mauro, will you pick it up?

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [PATCH] media: video: do not clear 'driver' from an i2c_client
  2010-11-15 22:28 ` Wolfram Sang
@ 2010-11-16 10:34   ` Jean Delvare
  0 siblings, 0 replies; 4+ messages in thread
From: Jean Delvare @ 2010-11-16 10:34 UTC (permalink / raw)
  To: Wolfram Sang, Mauro Carvalho Chehab
  Cc: linux-i2c, Guennadi Liakhovetski, Hans Verkuil, Sergio Aguirre,
	Michael Grzeschik, Philipp Wiesner, Márton Németh,
	linux-media, linux-kernel

Hi Wolfram,

Sorry for the late answer.

On Mon, 15 Nov 2010 23:28:15 +0100, Wolfram Sang wrote:
> On Wed, Nov 10, 2010 at 03:14:13PM +0100, Wolfram Sang wrote:
> > The i2c-core does this already.
> > 
> > Reported-by: Jean Delvare <khali@linux-fr.org>
> > Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
> > ---
> > 
> > Not sure if this should go via i2c or media?
> 
> Okay, as Jean did not pick it up in his latest pull request, I guess this means
> it shall go via the media-tree? :) Mauro, will you pick it up?

Yes I think it should go through Mauro's tree. I was simply waiting for
your other patch to get merged first (which happened yesterday),
because this patch depended on that one.

So:

Acked-by: Jean Delvare <khali@linux-fr.org>

and Mauro please pick the patch in your tree.

Thanks,
-- 
Jean Delvare

^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-11-16 10:35 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-11-10 14:14 [PATCH] media: video: do not clear 'driver' from an i2c_client Wolfram Sang
2010-11-10 14:24 ` Guennadi Liakhovetski
2010-11-15 22:28 ` Wolfram Sang
2010-11-16 10:34   ` Jean Delvare

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).