On Wed, Jul 17, 2013 at 03:49:24PM -0700, Andy Lutomirski wrote: > On Wed, Jul 17, 2013 at 3:19 PM, Guenter Roeck <linux@xxxxxxxxxxxx> wrote: > > On Wed, Jul 17, 2013 at 01:53:07PM -0700, Andy Lutomirski wrote: > >> Signed-off-by: Andy Lutomirski <luto@xxxxxxxxxxxxxx> > >> --- > > > > Why don't you just use the existing jc42 driver ? > > Failure to notice it. *sigh*. I assumed that any driver for this > device would have "tsod" or "TSOD" somewhere greppable. I'll at least > send a patch to improve the description. > > That being said, the jc42 driver *writes*. The iMC controller > (AFAICS) uses the data from the TSOD for things like dynamically > adjusting the DRAM refresh interval and IMO the kernel has no business > at all writing to any registers on the TSOD on this platform. (FWIW, > I tried fiddling with the critical threshold and my box didn't blow up > or report thermal trip events. But still, this scares me a bit.) > I don't see a problem with that. If the i2c controller supports writes, one can perform the write from user space with i2cset, so what is the problem ? Besides, jc42 compliant temperature sensors expect to be written to in order to set the temperature limits. If you don't like that, you don't have to do it. > If I modify the i2c_imc driver to add .class = I2C_CLASS_SPD and get > rid of the TSOD probing and load the jc42 module, then it can't > actually find my TSODs (because the i2c core isn't capable of probing > on the iMC bus). If I manually add the device, it works. > You mean it doesn't support the SMBus quick command ? Did you set I2C_CLASS_HWMON as well, or just I2C_CLASS_SPD ? If you have the ability to use i2c_board_info, the failure to auto-detect the devices should not really matter. In that case you can just use one of the other methods to instantiate the i2c devices. > But I'd like to keep the enumeration code I wrote around, since the > main point of this exercise is to enumerate things that aren't > actually hwmon devices. But I don't want random sensors scripts > reprogramming the hardware. Is the some magic I can work (or should > add) with i2c_board_info to force the jc42 driver into a read-only > mode? > Again, that is user space code doing the write, which you can control without reverting to a "private" driver. That the driver permits the write doesn't mean you have to use it. The only write it always performs is to enable the sensor if it is disabled .. which presumably should be ok even for you, since otherwise loading the driver doesn't make any sense in the first place. Thanks, Guenter -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
