| Message ID | 20081016171253.GB5515@oksana.dev.rtsoft.ru (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Commit | 11f1f2afd6b07729b12aaba479344d7f12d88ff9 |
| Headers | show |
Hi Anton, On Thu, 16 Oct 2008 21:12:53 +0400, Anton Vorontsov wrote: > If present the info->archdata is copied into the dev->archdata. > Some (OpenFirmware) platforms need it. > > Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com> > --- > drivers/i2c/i2c-core.c | 3 +++ > include/linux/i2c.h | 2 ++ > 2 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c > index 42e852d..5a485c2 100644 > --- a/drivers/i2c/i2c-core.c > +++ b/drivers/i2c/i2c-core.c > @@ -266,6 +266,9 @@ i2c_new_device(struct i2c_adapter *adap, struct i2c_board_info const *info) > > client->dev.platform_data = info->platform_data; > > + if (info->archdata) > + client->dev.archdata = *info->archdata; > + > client->flags = info->flags; > client->addr = info->addr; > client->irq = info->irq; > diff --git a/include/linux/i2c.h b/include/linux/i2c.h > index 0611512..3e358d3 100644 > --- a/include/linux/i2c.h > +++ b/include/linux/i2c.h > @@ -240,6 +240,7 @@ static inline void i2c_set_clientdata (struct i2c_client *dev, void *data) > * @flags: to initialize i2c_client.flags > * @addr: stored in i2c_client.addr > * @platform_data: stored in i2c_client.dev.platform_data > + * @archdata: copied into i2c_client.dev.archdata > * @irq: stored in i2c_client.irq > * > * I2C doesn't actually support hardware probing, although controllers and > @@ -259,6 +260,7 @@ struct i2c_board_info { > unsigned short flags; > unsigned short addr; > void *platform_data; > + struct dev_archdata *archdata; > int irq; > }; > No objection from me: Acked-by: Jean Delvare <khali@linux-fr.org> Let me know if I should push this patch upstream myself.
On Fri, 2008-10-17 at 11:21 +0200, Jean Delvare wrote: > Hi Anton, > > On Thu, 16 Oct 2008 21:12:53 +0400, Anton Vorontsov wrote: > > If present the info->archdata is copied into the dev->archdata. > > Some (OpenFirmware) platforms need it. > > > > Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com> > > --- So who's pushing this one ? Jean ? Cheers, Ben.
On Wed, 22 Oct 2008 11:27:34 +1100, Benjamin Herrenschmidt wrote: > On Fri, 2008-10-17 at 11:21 +0200, Jean Delvare wrote: > > Hi Anton, > > > > On Thu, 16 Oct 2008 21:12:53 +0400, Anton Vorontsov wrote: > > > If present the info->archdata is copied into the dev->archdata. > > > Some (OpenFirmware) platforms need it. > > > > > > Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com> > > > --- > > So who's pushing this one ? Jean ? As I wrote before, I'm happy to take this patch if you want me to, but I also have no objection if the author of this patchset wants to push it himself together with the rest of the set. Just let me know what you expect from me (preferably in the next few hours, as I will be sending my second and last round of i2c patches for kernel 2.6.28 to Linus today.)
On Wed, 2008-10-22 at 08:50 +0200, Jean Delvare wrote: > On Wed, 22 Oct 2008 11:27:34 +1100, Benjamin Herrenschmidt wrote: > > On Fri, 2008-10-17 at 11:21 +0200, Jean Delvare wrote: > > > Hi Anton, > > > > > > On Thu, 16 Oct 2008 21:12:53 +0400, Anton Vorontsov wrote: > > > > If present the info->archdata is copied into the dev->archdata. > > > > Some (OpenFirmware) platforms need it. > > > > > > > > Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com> > > > > --- > > > > So who's pushing this one ? Jean ? > > As I wrote before, I'm happy to take this patch if you want me to, but > I also have no objection if the author of this patchset wants to push > it himself together with the rest of the set. Just let me know what you > expect from me (preferably in the next few hours, as I will be sending > my second and last round of i2c patches for kernel 2.6.28 to Linus > today.) Anton, I've done my last batch for 2.6.28 before -rc1 so it might be better if it goes through Jean no ? Cheers, Ben.
On Wed, Oct 22, 2008 at 06:37:55PM +1100, Benjamin Herrenschmidt wrote: > On Wed, 2008-10-22 at 08:50 +0200, Jean Delvare wrote: > > On Wed, 22 Oct 2008 11:27:34 +1100, Benjamin Herrenschmidt wrote: > > > On Fri, 2008-10-17 at 11:21 +0200, Jean Delvare wrote: > > > > Hi Anton, > > > > > > > > On Thu, 16 Oct 2008 21:12:53 +0400, Anton Vorontsov wrote: > > > > > If present the info->archdata is copied into the dev->archdata. > > > > > Some (OpenFirmware) platforms need it. > > > > > > > > > > Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com> > > > > > --- > > > > > > So who's pushing this one ? Jean ? > > > > As I wrote before, I'm happy to take this patch if you want me to, but > > I also have no objection if the author of this patchset wants to push > > it himself together with the rest of the set. Just let me know what you > > expect from me (preferably in the next few hours, as I will be sending > > my second and last round of i2c patches for kernel 2.6.28 to Linus > > today.) > > Anton, I've done my last batch for 2.6.28 before -rc1 so it might be > better if it goes through Jean no ? Sorry, I guess the "few hours" limit was done exactly when I was sleeping. ;-) Anyway, I'm fine either way. Don't see any problem if it goes i2c or powerpc tree. Thanks,
On Wed, 22 Oct 2008 14:08:13 +0400, Anton Vorontsov wrote: > On Wed, Oct 22, 2008 at 06:37:55PM +1100, Benjamin Herrenschmidt wrote: > > On Wed, 2008-10-22 at 08:50 +0200, Jean Delvare wrote: > > > On Wed, 22 Oct 2008 11:27:34 +1100, Benjamin Herrenschmidt wrote: > > > > On Fri, 2008-10-17 at 11:21 +0200, Jean Delvare wrote: > > > > > Hi Anton, > > > > > > > > > > On Thu, 16 Oct 2008 21:12:53 +0400, Anton Vorontsov wrote: > > > > > > If present the info->archdata is copied into the dev->archdata. > > > > > > Some (OpenFirmware) platforms need it. > > > > > > > > > > > > Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com> > > > > > > --- > > > > > > > > So who's pushing this one ? Jean ? > > > > > > As I wrote before, I'm happy to take this patch if you want me to, but > > > I also have no objection if the author of this patchset wants to push > > > it himself together with the rest of the set. Just let me know what you > > > expect from me (preferably in the next few hours, as I will be sending > > > my second and last round of i2c patches for kernel 2.6.28 to Linus > > > today.) > > > > Anton, I've done my last batch for 2.6.28 before -rc1 so it might be > > better if it goes through Jean no ? > > Sorry, I guess the "few hours" limit was done exactly when I was > sleeping. ;-) Anyway, I'm fine either way. Don't see any problem > if it goes i2c or powerpc tree. OK, then I'm taking the patch in my tree and it will go to Linus later today.
On Wed, Oct 22, 2008 at 01:07:48PM +0200, Jean Delvare wrote: > On Wed, 22 Oct 2008 14:08:13 +0400, Anton Vorontsov wrote: > > On Wed, Oct 22, 2008 at 06:37:55PM +1100, Benjamin Herrenschmidt wrote: > > > On Wed, 2008-10-22 at 08:50 +0200, Jean Delvare wrote: > > > > On Wed, 22 Oct 2008 11:27:34 +1100, Benjamin Herrenschmidt wrote: > > > > > On Fri, 2008-10-17 at 11:21 +0200, Jean Delvare wrote: > > > > > > Hi Anton, > > > > > > > > > > > > On Thu, 16 Oct 2008 21:12:53 +0400, Anton Vorontsov wrote: > > > > > > > If present the info->archdata is copied into the dev->archdata. > > > > > > > Some (OpenFirmware) platforms need it. > > > > > > > > > > > > > > Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com> > > > > > > > --- > > > > > > > > > > So who's pushing this one ? Jean ? > > > > > > > > As I wrote before, I'm happy to take this patch if you want me to, but > > > > I also have no objection if the author of this patchset wants to push > > > > it himself together with the rest of the set. Just let me know what you > > > > expect from me (preferably in the next few hours, as I will be sending > > > > my second and last round of i2c patches for kernel 2.6.28 to Linus > > > > today.) > > > > > > Anton, I've done my last batch for 2.6.28 before -rc1 so it might be > > > better if it goes through Jean no ? > > > > Sorry, I guess the "few hours" limit was done exactly when I was > > sleeping. ;-) Anyway, I'm fine either way. Don't see any problem > > if it goes i2c or powerpc tree. > > OK, then I'm taking the patch in my tree and it will go to Linus later > today. Much appreciated, thanks.
diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c index 42e852d..5a485c2 100644 --- a/drivers/i2c/i2c-core.c +++ b/drivers/i2c/i2c-core.c @@ -266,6 +266,9 @@ i2c_new_device(struct i2c_adapter *adap, struct i2c_board_info const *info) client->dev.platform_data = info->platform_data; + if (info->archdata) + client->dev.archdata = *info->archdata; + client->flags = info->flags; client->addr = info->addr; client->irq = info->irq; diff --git a/include/linux/i2c.h b/include/linux/i2c.h index 0611512..3e358d3 100644 --- a/include/linux/i2c.h +++ b/include/linux/i2c.h @@ -240,6 +240,7 @@ static inline void i2c_set_clientdata (struct i2c_client *dev, void *data) * @flags: to initialize i2c_client.flags * @addr: stored in i2c_client.addr * @platform_data: stored in i2c_client.dev.platform_data + * @archdata: copied into i2c_client.dev.archdata * @irq: stored in i2c_client.irq * * I2C doesn't actually support hardware probing, although controllers and @@ -259,6 +260,7 @@ struct i2c_board_info { unsigned short flags; unsigned short addr; void *platform_data; + struct dev_archdata *archdata; int irq; };
If present the info->archdata is copied into the dev->archdata. Some (OpenFirmware) platforms need it. Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com> --- drivers/i2c/i2c-core.c | 3 +++ include/linux/i2c.h | 2 ++ 2 files changed, 5 insertions(+), 0 deletions(-)