Changeset 5393

Timestamp:

11/20/08 20:41:19 (4 years ago)

Author:

khali

Message:

Make reading back the written value optional.

Location:

i2c-tools/trunk

Files:

• CHANGES (modified) (1 diff)
• tools/i2cset.8 (modified) (2 diffs)
• tools/i2cset.c (modified) (3 diffs)

i2c-tools/trunk/CHANGES

@@ -24 +24 @@
           Better error message on missing adapter functionality
           Set the data value mask with -m
+          Make reading back the written value optional
   i2c-stub-from-dump: Add support for partial dumps
                       Report if only garbage is found in dump file

i2c-tools/trunk/tools/i2cset.8

@@ -8 +8 @@
 .RB [ -y ]
 .RB [ "-m mask" ]
+.RB [ -r ]
 .I i2cbus
 .I chip-address
@@ -47 +48 @@
 mode are symmetrical for the device you are accessing. This may or may not
 be the case, as neither I2C nor SMBus guarantees this.
+.TP
+.B -r
+Read back the value right after writing it, and compare the result with the
+value written. This used to be the default behavior. The same limitations
+apply as those of option \fB-m\fR.
 .PP
 There are three required options to i2cset. \fIi2cbus\fR indicates the number

i2c-tools/trunk/tools/i2cset.c

@@ -135 +135 @@
         int pec = 0;
         int flags = 0;
-        int force = 0, yes = 0, version = 0;
+        int force = 0, yes = 0, version = 0, readback = 0;
 
         /* handle (optional) flags first */
@@ -148 +148 @@
                         flags++;
                         break;
+                case 'r': readback = 1; break;
                 default:
                         fprintf(stderr, "Error: Unsupported option "
@@ -307 +308 @@
                         exit(1);
                 }
+        }
+
+        if (!readback) { /* We're done */
+                close(file);
+                exit(0);
         }