Document in_80chars() and remove reference_image_header[]

[libam7xxx.git] / src / am7xxx.c

diff --git a/src/am7xxx.c b/src/am7xxx.c
index 182ea27..3aedd7b 100644 (file)
--- a/src/am7xxx.c
+++ b/src/am7xxx.c
@@ -26,20 +26,6 @@
 #define AM7XXX_VENDOR_ID  0x1de1
 #define AM7XXX_PRODUCT_ID 0xc101
 
-#if 1
-static uint8_t reference_image_header[] = {
-       0x02, 0x00, 0x00, 0x00,
-       0x00,
-       0x10,
-       0x3e,
-       0x10,
-       0x01, 0x00, 0x00, 0x00,
-       0x20, 0x03, 0x00, 0x00,
-       0xe0, 0x01, 0x00, 0x00,
-       0x53, 0xE8, 0x00, 0x00
-};
-#endif
-
 static void dump_image_header(struct am7xxx_image_header *i)
 {
        if (i == NULL)
@@ -78,6 +64,8 @@ static void dump_header(struct am7xxx_header *h)
 
 static inline unsigned int in_80chars(unsigned int i)
 {
+       /* The 3 below is the length of "xx " where xx is the hex string
+        * representation of a byte */
        return ((i+1) % (80/3));
 }
 
@@ -99,7 +87,10 @@ static int send_data(am7xxx_device dev, uint8_t *buffer, unsigned int len)
        int ret;
        int transferred;
 
+#if DEBUG
        dump_buffer(buffer, len);
+       printf("\n");
+#endif
 
        ret = libusb_bulk_transfer(dev, 1, buffer, len, &transferred, 0);
        if (ret != 0 || (unsigned int)transferred != len) {
@@ -131,6 +122,11 @@ static int send_header(am7xxx_device dev, struct am7xxx_header *h)
        uint8_t *buffer;
        int ret;
 
+#if DEBUG
+       dump_header(h);
+       printf("\n");
+#endif
+
        buffer = calloc(AM7XXX_HEADER_WIRE_SIZE, 1);
        if (buffer == NULL) {
                perror("calloc buffer");
@@ -204,12 +200,6 @@ int am7xxx_send_image(am7xxx_device dev,
                },
        };
 
-       dump_header(&h);
-       printf("\n");
-
-       printf("Dump Buffers\n");
-       dump_buffer(reference_image_header, sizeof(struct am7xxx_header));
-
        ret = send_header(dev, &h);
        if (ret < 0)
                return ret;