Re: [i2c] [PATCH 2/7] i2c: add info->archdata field
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 <[EMAIL PROTECTED]>
> ---
> 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 <[EMAIL PROTECTED]>
Let me know if I should push this patch upstream myself.
--
Jean Delvare
___
i2c mailing list
i2c@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/i2c
Re: [i2c] [PATCH 2/7] i2c: add info->archdata field
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 <[EMAIL PROTECTED]> > > --- So who's pushing this one ? Jean ? Cheers, Ben. ___ i2c mailing list i2c@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/i2c
Re: [i2c] [PATCH 2/7] i2c: add info->archdata field
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 <[EMAIL PROTECTED]> > > > --- > > 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.) -- Jean Delvare ___ i2c mailing list i2c@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/i2c
Re: [i2c] [PATCH 2/7] i2c: add info->archdata field
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 <[EMAIL PROTECTED]> > > > > --- > > > > 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. ___ i2c mailing list i2c@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/i2c
Re: [i2c] [PATCH 2/7] i2c: add info->archdata field
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 <[EMAIL PROTECTED]> > > > > > --- > > > > > > 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, -- Anton Vorontsov email: [EMAIL PROTECTED] irc://irc.freenode.net/bd2 ___ i2c mailing list i2c@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/i2c
Re: [i2c] [PATCH 2/7] i2c: add info->archdata field
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 <[EMAIL PROTECTED]> > > > > > > --- > > > > > > > > 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. -- Jean Delvare ___ i2c mailing list i2c@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/i2c
Re: [i2c] [PATCH 2/7] i2c: add info->archdata field
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 <[EMAIL PROTECTED]> > > > > > > > --- > > > > > > > > > > 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. -- Anton Vorontsov email: [EMAIL PROTECTED] irc://irc.freenode.net/bd2 ___ i2c mailing list i2c@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/i2c
