Re: [PATCH 2/7] i2c: add info-archdata field

2008-10-22 Thread Jean Delvare
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
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH 2/7] i2c: add info-archdata field

2008-10-22 Thread Benjamin Herrenschmidt
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.


___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH 2/7] i2c: add info-archdata field

2008-10-22 Thread Anton Vorontsov
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
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH 2/7] i2c: add info-archdata field

2008-10-22 Thread Jean Delvare
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
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH 2/7] i2c: add info-archdata field

2008-10-21 Thread Benjamin Herrenschmidt
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.

___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH 2/7] i2c: add info-archdata field

2008-10-17 Thread Jean Delvare
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
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev