Re: [PATCH] Support for DS75 thermal sensor

2008-07-17 Thread Wolfgang Grandegger
Grant Likely wrote: On Thu, Jul 17, 2008 at 09:31:24AM +0200, Wolfgang Grandegger wrote: Jean Delvare wrote: On Wed, 16 Jul 2008 10:18:26 -0400, Jon Smirl wrote: I've found this thread now. Why can't we totally remove probing from i2c-mpc? These are embedded systems, not open boxes like a PC.

Re: [PATCH] Support for DS75 thermal sensor

2008-07-17 Thread Grant Likely
On Thu, Jul 17, 2008 at 09:31:24AM +0200, Wolfgang Grandegger wrote: > Jean Delvare wrote: >> On Wed, 16 Jul 2008 10:18:26 -0400, Jon Smirl wrote: >>> I've found this thread now. Why can't we totally remove probing from >>> i2c-mpc? These are embedded systems, not open boxes like a PC. If a >>> i2c

Re: [PATCH] Support for DS75 thermal sensor

2008-07-17 Thread Wolfgang Grandegger
Jean Delvare wrote: On Wed, 16 Jul 2008 10:18:26 -0400, Jon Smirl wrote: On 7/16/08, Grant Likely <[EMAIL PROTECTED]> wrote: On Wed, Jul 16, 2008 at 12:10:59PM +0200, Jean Delvare wrote: > On Wed, 16 Jul 2008 11:50:15 +0200, Wolfgang Grandegger wrote: > > Jean Delvare wrote: > > Yep, as pr

Re: [PATCH] Support for DS75 thermal sensor

2008-07-16 Thread Jean Delvare
On Wed, 16 Jul 2008 10:18:26 -0400, Jon Smirl wrote: > On 7/16/08, Grant Likely <[EMAIL PROTECTED]> wrote: > > On Wed, Jul 16, 2008 at 12:10:59PM +0200, Jean Delvare wrote: > > > On Wed, 16 Jul 2008 11:50:15 +0200, Wolfgang Grandegger wrote: > > > > Jean Delvare wrote: > > > > > > > > > > Yep, a

Re: [PATCH] Support for DS75 thermal sensor

2008-07-16 Thread Jon Smirl
On 7/16/08, Grant Likely <[EMAIL PROTECTED]> wrote: > On Wed, Jul 16, 2008 at 12:10:59PM +0200, Jean Delvare wrote: > > On Wed, 16 Jul 2008 11:50:15 +0200, Wolfgang Grandegger wrote: > > > Jean Delvare wrote: > > > > > > > Yep, as probing might not be acceptable in some cases, I makes sense to >

Re: [PATCH] Support for DS75 thermal sensor

2008-07-16 Thread Grant Likely
On Wed, Jul 16, 2008 at 12:10:59PM +0200, Jean Delvare wrote: > On Wed, 16 Jul 2008 11:50:15 +0200, Wolfgang Grandegger wrote: > > Jean Delvare wrote: > > > > Yep, as probing might not be acceptable in some cases, I makes sense to > > add a property to suppress probing: > > It'd rather make no-p

Re: [PATCH] Support for DS75 thermal sensor

2008-07-16 Thread Wolfgang Grandegger
Jean Delvare wrote: On Wed, 16 Jul 2008 11:50:15 +0200, Wolfgang Grandegger wrote: Jean Delvare wrote: The problem is that at this point in time, only a couple hwmon drivers have been converted to new-style i2c. So, dropping the I2C_CLASS_HWMON would break most systems. I have a set of patches

Re: [PATCH] Support for DS75 thermal sensor

2008-07-16 Thread Jean Delvare
On Wed, 16 Jul 2008 11:50:15 +0200, Wolfgang Grandegger wrote: > Jean Delvare wrote: > > The problem is that at this point in time, only a couple hwmon drivers > > have been converted to new-style i2c. So, dropping the I2C_CLASS_HWMON > > would break most systems. > > > > I have a set of patches c

Re: [PATCH] Support for DS75 thermal sensor

2008-07-16 Thread Wolfgang Grandegger
Hi Jean; Jean Delvare wrote: Hi Wolfgang, On Wed, 16 Jul 2008 11:12:50 +0200, Wolfgang Grandegger wrote: Jean Delvare wrote: For hwmon chips, probing only occurs if the i2c adapter accepts to be probed, by setting the I2C_CLASS_HWMON flag in i2c_adapter.class. If you do not want a given i2c b

Re: [PATCH] Support for DS75 thermal sensor

2008-07-16 Thread Jean Delvare
Hi Wolfgang, On Wed, 16 Jul 2008 11:12:50 +0200, Wolfgang Grandegger wrote: > Jean Delvare wrote: > > For hwmon chips, probing only occurs if the i2c adapter accepts to be > > probed, by setting the I2C_CLASS_HWMON flag in i2c_adapter.class. If > > you do not want a given i2c bus to be probed, the

Re: [lm-sensors] [PATCH] Support for DS75 thermal sensor

2008-07-16 Thread Wolfgang Grandegger
Hi Jean, Jean Delvare wrote: Hi Wolfgang, On Fri, 11 Jul 2008 14:40:58 +0200, Wolfgang Grandegger wrote: Jean Delvare wrote: There's no "detection" involved for the new-style i2c devices. Presumably you are still using the old binding model, not taking benefit of the patch at all. Use i2c_reg