Changeset 5209

Timestamp:

04/24/08 14:43:10 (7 months ago)

Author:

khali

Message:

Move the confirmation to a separate function, as is done in i2cget
already.

Files:

• i2c-tools/trunk/CHANGES (modified) (1 diff)
• i2c-tools/trunk/tools/i2cset.c (modified) (2 diffs)

i2c-tools/trunk/CHANGES

@@ -14 +14 @@
-          Shorten the usage message
-          Restrict the chip address to 0x03-0x77
+          Split the code into several functions for clarity
-
-3.0.1 (2008-04-04)

i2c-tools/trunk/tools/i2cset.c

@@ -82 +82 @@
-
-        return 0;
+}
+
+static int confirm(const char *filename, int address, int size, int daddress,
+                   int value, int vmask, int pec)
+{
+        int dont = 0;
+
+        fprintf(stderr, "WARNING! This program can confuse your I2C "
+                "bus, cause data loss and worse!\n");
+
+        if (address >= 0x50 && address <= 0x57) {
+                fprintf(stderr, "DANGEROUS! Writing to a serial "
+                        "EEPROM on a memory DIMM\nmay render your "
+                        "memory USELESS and make your system "
+                        "UNBOOTABLE!\n");
+                dont++;
+        }
+
+        fprintf(stderr, "I will write to device file %s, chip address "
+                "0x%02x, data address\n0x%02x, data 0x%02x%s, mode "
+                "%s.\n", filename, address, daddress, value,
+                vmask ? " (masked)" : "",
+                size == I2C_SMBUS_BYTE_DATA ? "byte" : "word");
+        if (pec)
+                fprintf(stderr, "PEC checking enabled.\n");
+
+        fprintf(stderr, "Continue? [%s] ", dont ? "y/N" : "Y/n");
+        fflush(stderr);
+        if (!user_ack(!dont)) {
+                fprintf(stderr, "Aborting on user request.\n");
+                return 0;
+        }
+
+        return 1;
-}
-
@@ -171 +205 @@
-                exit(1);
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
-
-        if (vmask) {