Small fix in the PCF8591 driver

Aurelien Jarno aurelien at aurel32.net
Sun Mar 14 20:42:15 CET 2004


Hello,

Please find attached a patch to fix a small bug in pcf8591.c: the
function i2c_func_smbus_write_byte_data() was used without checking for
it before.

Cheers,
Aurelien

-- 
  .''`.  Aurelien Jarno	              GPG: 1024D/F1BCDB73
 : :' :  Debian GNU/Linux developer | Electrical Engineering Student 
 `. `'   aurel32 at debian.org         | aurelien at aurel32.net
   `-    people.debian.org/~aurel32 | www.aurel32.net
-------------- next part --------------
Index: kernel/chips/pcf8591.c
===================================================================
RCS file: /home/cvs/lm_sensors2/kernel/chips/pcf8591.c,v
retrieving revision 1.6
diff -u -1 -b -p -r1.6 pcf8591.c
--- kernel/chips/pcf8591.c	21 Jan 2003 20:01:26 -0000	1.6
+++ kernel/chips/pcf8591.c	14 Mar 2004 19:45:51 -0000
@@ -181,3 +181,4 @@ int pcf8591_detect(struct i2c_adapter *a
 
-        if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE))
+        if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE) 
+	                             | I2C_FUNC_SMBUS_BYTE_DATA)
                 goto ERROR0;


More information about the lm-sensors mailing list