From 8e94a4925f9db4690aa5770434d9f17b8d0e24e1 Mon Sep 17 00:00:00 2001
From: "Mark D. Studebaker" <mdsxyz123@yahoo.com>
Date: Mon, 19 Nov 2001 21:14:43 +0000
Subject: [PATCH] add i2c block read capability to i2cdump

git-svn-id: http://lm-sensors.org/svn/lm-sensors/trunk@1244 7894878c-1315-0410-8ee3-d5d059ff63e0
---
 CHANGES             |  1 +
 prog/dump/i2cdump.c | 19 ++++++++++++++++---
 2 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/CHANGES b/CHANGES
index 36540eaa..ced9dbd7 100644
--- a/CHANGES
+++ b/CHANGES
@@ -20,6 +20,7 @@ ask CVS about it:
 2.6.3 (2001????)
   Library: Add PCF8591 support
   Module pcf8591: new
+  Program i2cdump: Add i2c block read capability
   Program sensors: Add PCF8591 support
   Program sensors-detect: Add several Nvidia chips, add PCF8591
 
diff --git a/prog/dump/i2cdump.c b/prog/dump/i2cdump.c
index 838fce99..ead99018 100644
--- a/prog/dump/i2cdump.c
+++ b/prog/dump/i2cdump.c
@@ -26,6 +26,18 @@
 #include <fcntl.h>
 #include <linux/i2c-dev.h>
 
+/*
+   We don't use this #define but it was put into i2c.h at the same time as
+   i2c_smbus_read_i2c_block_data() was implemented (i2c 2.6.3),
+   so we use it as a version check.
+*/
+#if defined(I2C_FUNC_SMBUS_READ_I2C_BLOCK_2)
+#define USE_I2C_BLOCK 1
+#else
+#define USE_I2C_BLOCK 0
+#endif
+
+
 void help(void)
 {
   FILE *fptr;
@@ -256,13 +268,14 @@ int main(int argc, char *argv[])
       if(size == I2C_SMBUS_BLOCK_DATA) {
         res = i2c_smbus_read_block_data(file,0,cblock);
       } else if(size == I2C_SMBUS_I2C_BLOCK_DATA) {
-/*
+#if USE_I2C_BLOCK
         res = i2c_smbus_read_i2c_block_data(file,0,cblock);
-*/
+#else
         fprintf(stderr, "Error: I2C block read unimplemented\n");
         exit(1);
+#endif
       }
-      if(res < 0) {
+      if(res <= 0) {
         fprintf(stderr, "Error: Block read failed, return code %d\n", res);
         exit(1);
       }