* Re: [PATCH] drivers: remove all i2c_set_clientdata(client, NULL)
[not found] <1275310552-14685-1-git-send-email-w.sang@pengutronix.de>
@ 2010-05-31 13:01 ` Mark Brown
2010-05-31 16:54 ` Greg KH
` (2 subsequent siblings)
3 siblings, 0 replies; 7+ messages in thread
From: Mark Brown @ 2010-05-31 13:01 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-i2c, Jean Delvare, George Joseph, Riku Voipio,
Guillaume Ligneul, Ben Dooks (embedded platforms),
Dmitry Torokhov, Alessandro Rubini, Richard Purdie, Colin Leroy,
Benjamin Herrenschmidt, Mauro Carvalho Chehab, Samuel Ortiz,
David Woodhouse, Liam Girdwood, Paul Gortmaker, Alessandro Zummo,
Greg Kroah-Hartman, lm-sensors, linux-kernel, linux-input,
linuxppc-dev, linux-media, linux-mtd, rtc-linux, devel
On Mon, May 31, 2010 at 02:55:48PM +0200, Wolfram Sang wrote:
> I2C-drivers can use the clientdata-pointer to point to private data. As I2C
> devices are not really unregistered, but merely detached from their driver, it
> used to be the drivers obligation to clear this pointer during remove() or a
> failed probe(). As a couple of drivers forgot to do this, it was agreed that it
> was cleaner if the i2c-core does this clearance when appropriate, as there is
> no guarantee for the lifetime of the clientdata-pointer after remove() anyhow.
> This feature was added to the core with commit
> e4a7b9b04de15f6b63da5ccdd373ffa3057a3681 to fix the faulty drivers.
>
> As there is no need anymore to clear the clientdata-pointer, remove all current
> occurrences in the drivers to simplify the code and prevent confusion.
>
> Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] drivers: remove all i2c_set_clientdata(client, NULL)
[not found] <1275310552-14685-1-git-send-email-w.sang@pengutronix.de>
2010-05-31 13:01 ` [PATCH] drivers: remove all i2c_set_clientdata(client, NULL) Mark Brown
@ 2010-05-31 16:54 ` Greg KH
2010-05-31 19:09 ` Dmitry Torokhov
2010-06-01 13:53 ` Jean Delvare
3 siblings, 0 replies; 7+ messages in thread
From: Greg KH @ 2010-05-31 16:54 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-i2c, Jean Delvare, George Joseph, Riku Voipio,
Guillaume Ligneul, Ben Dooks (embedded platforms),
Dmitry Torokhov, Alessandro Rubini, Richard Purdie, Colin Leroy,
Benjamin Herrenschmidt, Mauro Carvalho Chehab, Samuel Ortiz,
Mark Brown, David Woodhouse, Liam Girdwood, Paul Gortmaker,
Alessandro Zummo, lm-sensors, linux-kernel, linux-input,
linuxppc-dev, linux-media, linux-mtd, rtc-linux, devel
On Mon, May 31, 2010 at 02:55:48PM +0200, Wolfram Sang wrote:
> I2C-drivers can use the clientdata-pointer to point to private data. As I2C
> devices are not really unregistered, but merely detached from their driver, it
> used to be the drivers obligation to clear this pointer during remove() or a
> failed probe(). As a couple of drivers forgot to do this, it was agreed that it
> was cleaner if the i2c-core does this clearance when appropriate, as there is
> no guarantee for the lifetime of the clientdata-pointer after remove() anyhow.
> This feature was added to the core with commit
> e4a7b9b04de15f6b63da5ccdd373ffa3057a3681 to fix the faulty drivers.
>
> As there is no need anymore to clear the clientdata-pointer, remove all current
> occurrences in the drivers to simplify the code and prevent confusion.
>
> Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
> Cc: Jean Delvare <khali@linux-fr.org>
Acked-by: Greg Kroah-Hartman <gregkh@suse.de>
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] drivers: remove all i2c_set_clientdata(client, NULL)
[not found] <1275310552-14685-1-git-send-email-w.sang@pengutronix.de>
2010-05-31 13:01 ` [PATCH] drivers: remove all i2c_set_clientdata(client, NULL) Mark Brown
2010-05-31 16:54 ` Greg KH
@ 2010-05-31 19:09 ` Dmitry Torokhov
2010-05-31 20:57 ` Jean Delvare
2010-05-31 21:48 ` Richard Purdie
2010-06-01 13:53 ` Jean Delvare
3 siblings, 2 replies; 7+ messages in thread
From: Dmitry Torokhov @ 2010-05-31 19:09 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-i2c, Jean Delvare, George Joseph, Riku Voipio,
Guillaume Ligneul, Ben Dooks (embedded platforms),
Alessandro Rubini, Richard Purdie, Colin Leroy,
Benjamin Herrenschmidt, Mauro Carvalho Chehab, Samuel Ortiz,
Mark Brown, David Woodhouse, Liam Girdwood, Paul Gortmaker,
Alessandro Zummo, Greg Kroah-Hartman, lm-sensors, linux-kernel,
linux-input, linuxppc-dev, linux-media, linux-mtd, rtc-linux,
devel
Hi Wolfram,
On Mon, May 31, 2010 at 02:55:48PM +0200, Wolfram Sang wrote:
> I2C-drivers can use the clientdata-pointer to point to private data. As I2C
> devices are not really unregistered, but merely detached from their driver, it
> used to be the drivers obligation to clear this pointer during remove() or a
> failed probe(). As a couple of drivers forgot to do this, it was agreed that it
> was cleaner if the i2c-core does this clearance when appropriate, as there is
> no guarantee for the lifetime of the clientdata-pointer after remove() anyhow.
> This feature was added to the core with commit
> e4a7b9b04de15f6b63da5ccdd373ffa3057a3681 to fix the faulty drivers.
>
> As there is no need anymore to clear the clientdata-pointer, remove all current
> occurrences in the drivers to simplify the code and prevent confusion.
>
> Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
> Cc: Jean Delvare <khali@linux-fr.org>
> ---
>
> Some more notes:
>
> I waited for rc1 as I knew there were some drivers/patches coming along which
> needed to be processed, too.
>
> I'd suggest that this goes via the i2c-tree, so we get rid of all occurences at
> once.
>
Frankly I'd prefer taking input stuff through my tree with the goal of
.36 merge window just to minimize potential merge issues. This is a
simple cleanup patch that has no dependencies, so there is little gain
from doing it all in one go.
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] drivers: remove all i2c_set_clientdata(client, NULL)
2010-05-31 19:09 ` Dmitry Torokhov
@ 2010-05-31 20:57 ` Jean Delvare
2010-05-31 21:48 ` Richard Purdie
1 sibling, 0 replies; 7+ messages in thread
From: Jean Delvare @ 2010-05-31 20:57 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: Wolfram Sang, linux-i2c, George Joseph, Riku Voipio,
Guillaume Ligneul, Ben Dooks (embedded platforms),
Alessandro Rubini, Richard Purdie, Colin Leroy,
Benjamin Herrenschmidt, Mauro Carvalho Chehab, Samuel Ortiz,
Mark Brown, David Woodhouse, Liam Girdwood, Paul Gortmaker,
Alessandro Zummo, Greg Kroah-Hartman, lm-sensors, linux-kernel,
linux-input, linuxppc-dev, linux-media, linux-mtd, rtc-linux,
devel
Hi Dmitry,
On Mon, 31 May 2010 12:09:12 -0700, Dmitry Torokhov wrote:
> Frankly I'd prefer taking input stuff through my tree with the goal of
> .36 merge window just to minimize potential merge issues. This is a
> simple cleanup patch that has no dependencies, so there is little gain
> from doing it all in one go.
If I take the patch in my i2c tree, the aim is to merge it upstream
immediately, so merge issues won't exist.
--
Jean Delvare
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drivers: remove all i2c_set_clientdata(client, NULL)
2010-05-31 19:09 ` Dmitry Torokhov
2010-05-31 20:57 ` Jean Delvare
@ 2010-05-31 21:48 ` Richard Purdie
2010-06-01 5:05 ` Dmitry Torokhov
1 sibling, 1 reply; 7+ messages in thread
From: Richard Purdie @ 2010-05-31 21:48 UTC (permalink / raw)
To: Dmitry Torokhov, Wolfram Sang
Cc: linux-i2c, Jean Delvare, George Joseph, Riku Voipio,
Guillaume Ligneul, Ben Dooks (embedded platforms),
Alessandro Rubini, Colin Leroy, Benjamin Herrenschmidt,
Mauro Carvalho Chehab, Samuel Ortiz, Mark Brown, David Woodhouse,
Liam Girdwood, Paul Gortmaker, Alessandro Zummo,
Greg Kroah-Hartman, lm-sensors, linux-kernel, linux-input,
linuxppc-dev, linux-media, linux-mtd, rtc-linux, devel
On Mon, 2010-05-31 at 12:09 -0700, Dmitry Torokhov wrote:
> On Mon, May 31, 2010 at 02:55:48PM +0200, Wolfram Sang wrote:
> > I2C-drivers can use the clientdata-pointer to point to private data. As I2C
> > devices are not really unregistered, but merely detached from their driver, it
> > used to be the drivers obligation to clear this pointer during remove() or a
> > failed probe(). As a couple of drivers forgot to do this, it was agreed that it
> > was cleaner if the i2c-core does this clearance when appropriate, as there is
> > no guarantee for the lifetime of the clientdata-pointer after remove() anyhow.
> > This feature was added to the core with commit
> > e4a7b9b04de15f6b63da5ccdd373ffa3057a3681 to fix the faulty drivers.
> >
> > As there is no need anymore to clear the clientdata-pointer, remove all current
> > occurrences in the drivers to simplify the code and prevent confusion.
> >
> > Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
> > Cc: Jean Delvare <khali@linux-fr.org>
> > ---
> >
> > Some more notes:
> >
> > I waited for rc1 as I knew there were some drivers/patches coming along which
> > needed to be processed, too.
> >
> > I'd suggest that this goes via the i2c-tree, so we get rid of all occurences at
> > once.
> >
>
> Frankly I'd prefer taking input stuff through my tree with the goal of
> .36 merge window just to minimize potential merge issues. This is a
> simple cleanup patch that has no dependencies, so there is little gain
> from doing it all in one go.
How about asking Linus to take this one now, then its done and we can
all move on rather than queuing up problems for the next merge window?
Acked-by: Richard Purdie <rpurdie@linux.intel.com>
Cheers,
Richard
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drivers: remove all i2c_set_clientdata(client, NULL)
2010-05-31 21:48 ` Richard Purdie
@ 2010-06-01 5:05 ` Dmitry Torokhov
0 siblings, 0 replies; 7+ messages in thread
From: Dmitry Torokhov @ 2010-06-01 5:05 UTC (permalink / raw)
To: Richard Purdie
Cc: Wolfram Sang, linux-i2c, Jean Delvare, George Joseph, Riku Voipio,
Guillaume Ligneul, Ben Dooks (embedded platforms),
Alessandro Rubini, Colin Leroy, Benjamin Herrenschmidt,
Mauro Carvalho Chehab, Samuel Ortiz, Mark Brown, David Woodhouse,
Liam Girdwood, Paul Gortmaker, Alessandro Zummo,
Greg Kroah-Hartman, lm-sensors, linux-kernel, linux-input,
linuxppc-dev, linux-media, linux-mtd, rtc-linux, devel
On Mon, May 31, 2010 at 10:48:32PM +0100, Richard Purdie wrote:
> On Mon, 2010-05-31 at 12:09 -0700, Dmitry Torokhov wrote:
> > On Mon, May 31, 2010 at 02:55:48PM +0200, Wolfram Sang wrote:
> > > I2C-drivers can use the clientdata-pointer to point to private data. As I2C
> > > devices are not really unregistered, but merely detached from their driver, it
> > > used to be the drivers obligation to clear this pointer during remove() or a
> > > failed probe(). As a couple of drivers forgot to do this, it was agreed that it
> > > was cleaner if the i2c-core does this clearance when appropriate, as there is
> > > no guarantee for the lifetime of the clientdata-pointer after remove() anyhow.
> > > This feature was added to the core with commit
> > > e4a7b9b04de15f6b63da5ccdd373ffa3057a3681 to fix the faulty drivers.
> > >
> > > As there is no need anymore to clear the clientdata-pointer, remove all current
> > > occurrences in the drivers to simplify the code and prevent confusion.
> > >
> > > Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
> > > Cc: Jean Delvare <khali@linux-fr.org>
> > > ---
> > >
> > > Some more notes:
> > >
> > > I waited for rc1 as I knew there were some drivers/patches coming along which
> > > needed to be processed, too.
> > >
> > > I'd suggest that this goes via the i2c-tree, so we get rid of all occurences at
> > > once.
> > >
> >
> > Frankly I'd prefer taking input stuff through my tree with the goal of
> > .36 merge window just to minimize potential merge issues. This is a
> > simple cleanup patch that has no dependencies, so there is little gain
> > from doing it all in one go.
>
> How about asking Linus to take this one now, then its done and we can
> all move on rather than queuing up problems for the next merge window?
>
That should work.
Acked-by: Dmitry Torokhov <dtor@mail.ru>
> Acked-by: Richard Purdie <rpurdie@linux.intel.com>
>
--
Dmitry
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drivers: remove all i2c_set_clientdata(client, NULL)
[not found] <1275310552-14685-1-git-send-email-w.sang@pengutronix.de>
` (2 preceding siblings ...)
2010-05-31 19:09 ` Dmitry Torokhov
@ 2010-06-01 13:53 ` Jean Delvare
3 siblings, 0 replies; 7+ messages in thread
From: Jean Delvare @ 2010-06-01 13:53 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-i2c, George Joseph, Riku Voipio, Guillaume Ligneul,
Ben Dooks (embedded platforms), Dmitry Torokhov,
Alessandro Rubini, Richard Purdie, Colin Leroy,
Benjamin Herrenschmidt, Mauro Carvalho Chehab, Samuel Ortiz,
Mark Brown, David Woodhouse, Liam Girdwood, Paul Gortmaker,
Alessandro Zummo, Greg Kroah-Hartman, lm-sensors, linux-kernel,
linux-input, linuxppc-dev, linux-media, linux-mtd, rtc-linux,
devel
On Mon, 31 May 2010 14:55:48 +0200, Wolfram Sang wrote:
> I2C-drivers can use the clientdata-pointer to point to private data. As I2C
> devices are not really unregistered, but merely detached from their driver, it
> used to be the drivers obligation to clear this pointer during remove() or a
> failed probe(). As a couple of drivers forgot to do this, it was agreed that it
> was cleaner if the i2c-core does this clearance when appropriate, as there is
> no guarantee for the lifetime of the clientdata-pointer after remove() anyhow.
> This feature was added to the core with commit
> e4a7b9b04de15f6b63da5ccdd373ffa3057a3681 to fix the faulty drivers.
>
> As there is no need anymore to clear the clientdata-pointer, remove all current
> occurrences in the drivers to simplify the code and prevent confusion.
>
> Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
> Cc: Jean Delvare <khali@linux-fr.org>
> (...)
Applied, thanks. Will go to Linus in the next few days.
--
Jean Delvare
^ permalink raw reply [flat|nested] 7+ messages in thread