A set bit means load D2XX driver. Revers logic and names
[libftdi] / src / ftdi.c
index c3d5fd8..ac0a651 100644 (file)
@@ -2206,8 +2206,11 @@ void ftdi_eeprom_initdefaults(struct ftdi_context *ftdi)
     default:
         eeprom->release = 0;
     }
-    eeprom->usb_version = 0x0200;
-    eeprom->max_power = 100;
+    if (ftdi->type == TYPE_AM)
+        eeprom->usb_version = 0x0101;
+    else
+        eeprom->usb_version = 0x0200;
+    eeprom->max_power = 50;
 
     eeprom->manufacturer = NULL;
     eeprom->product = NULL;
@@ -2261,6 +2264,7 @@ void ftdi_eeprom_free(struct ftdi_context *ftdi)
     \retval -3: Invalid cbus function setting
     \retval -4: Chip doesn't support invert
     \retval -5: Chip doesn't support high current drive
+    \retval -6: No connected EEPROM or EEPROM Type unknown
 */
 int ftdi_eeprom_build(struct ftdi_context *ftdi, unsigned char *output)
 {
@@ -2268,7 +2272,6 @@ int ftdi_eeprom_build(struct ftdi_context *ftdi, unsigned char *output)
     unsigned short checksum, value;
     unsigned char manufacturer_size = 0, product_size = 0, serial_size = 0;
     int size_check;
-    const int cbus_max[5] = {13, 13, 13, 13, 9};
     struct ftdi_eeprom *eeprom;
 
     if (ftdi == NULL)
@@ -2278,6 +2281,9 @@ int ftdi_eeprom_build(struct ftdi_context *ftdi, unsigned char *output)
 
     eeprom= ftdi->eeprom;
 
+    if(eeprom->chip == -1)
+        ftdi_error_return(-5,"No connected EEPROM or EEPROM Type unknown");
+
     if (eeprom->manufacturer != NULL)
         manufacturer_size = strlen(eeprom->manufacturer);
     if (eeprom->product != NULL)
@@ -2285,18 +2291,6 @@ int ftdi_eeprom_build(struct ftdi_context *ftdi, unsigned char *output)
     if (eeprom->serial != NULL)
         serial_size = strlen(eeprom->serial);
 
-    // highest allowed cbus value
-    for (i = 0; i < 5; i++)
-    {
-        if ((eeprom->cbus_function[i] > cbus_max[i]) ||
-            (eeprom->cbus_function[i] && ftdi->type != TYPE_R)) return -3;
-    }
-    if (ftdi->type != TYPE_R)
-    {
-        if (eeprom->invert) return -4;
-        if (eeprom->high_current_a) return -5;
-    }
-
     size_check = 0x80;
     switch(ftdi->type)
     {
@@ -2378,7 +2372,8 @@ int ftdi_eeprom_build(struct ftdi_context *ftdi, unsigned char *output)
     // Bit 7: 0 - reserved
     // Bit 6: 0 - reserved
     // Bit 5: 0 - reserved
-    // Bit 4: 1 - Change USB version
+    // Bit 4: 1 - Change USB version 
+    //            not seen on FT2232C)
     // Bit 3: 1 - Use the serial number string
     // Bit 2: 1 - Enable suspend pull downs for lower power
     // Bit 1: 1 - Out EndPoint is Isochronous
@@ -2393,20 +2388,15 @@ int ftdi_eeprom_build(struct ftdi_context *ftdi, unsigned char *output)
         j = j | 4;
     if (eeprom->use_serial == 1)
         j = j | 8;
-    if (eeprom->change_usb_version == 1)
-        j = j | 16;
     output[0x0A] = j;
 
     // Addr 0B: Invert data lines
     output[0x0B] = eeprom->invert & 0xff;
 
-    // Addr 0C: USB version low byte when 0x0A bit 4 is set
-    // Addr 0D: USB version high byte when 0x0A bit 4 is set
-    if (eeprom->change_usb_version == 1)
-    {
-        output[0x0C] = eeprom->usb_version;
-        output[0x0D] = eeprom->usb_version >> 8;
-    }
+    // Addr 0C: USB version low byte
+    // Addr 0D: USB version high byte
+    output[0x0C] = eeprom->usb_version;
+    output[0x0D] = eeprom->usb_version >> 8;
 
 
     // Addr 0E: Offset of the manufacturer string + 0x80, calculated later
@@ -2522,17 +2512,23 @@ int ftdi_eeprom_decode(struct ftdi_context *ftdi, unsigned char *buf, int size,
     // Addr 00: Channel A setting
 
     eeprom->channel_a_type   = buf[0x00] & 0x7;
-    eeprom->channel_a_driver = buf[0x00] & DRIVER_VCP;
+    eeprom->high_current     = buf[0x00] & HIGH_CURRENT_DRIVE_R;
+    eeprom->channel_a_driver = buf[0x00] & DRIVER_D2XX;
     eeprom->high_current_a   = buf[0x00] & HIGH_CURRENT_DRIVE;
 
     // Addr 01: Channel B setting
 
     eeprom->channel_b_type   = buf[0x01] & 0x7;
-    eeprom->channel_b_driver = buf[0x01] & DRIVER_VCP;
+    eeprom->channel_b_driver = buf[0x01] & DRIVER_D2XX;
     eeprom->high_current_b   = buf[0x01] & HIGH_CURRENT_DRIVE;
 
     eeprom->suspend_dbus7    = buf[0x01] & SUSPEND_DBUS7;
 
+    if((ftdi->type == TYPE_R) && ((buf[0x01]&0x40) != 0x40))
+        fprintf(stderr,
+                "TYPE_R EEPROM byte[0x01] Bit 6 unexpected Endpoint size. If this happened with the\n"
+                " EEPROM programmed by FTDI tools, please report to libftdi@developer.intra2net.com\n");
+
     // Addr 02: Vendor ID
     eeprom->vendor_id = buf[0x02] + (buf[0x03] << 8);
 
@@ -2567,15 +2563,15 @@ int ftdi_eeprom_decode(struct ftdi_context *ftdi, unsigned char *buf, int size,
     eeprom->out_is_isochronous = buf[0x0A]&0x02;
     eeprom->suspend_pull_downs = buf[0x0A]&0x04;
     eeprom->use_serial         = buf[0x0A] & USE_SERIAL_NUM;
-    eeprom->change_usb_version = buf[0x0A]&0x10;
+    if(buf[0x0A]&0x10)
+        fprintf(stderr,
+                "EEPROM byte[0x0a] Bit 4 unexpected set. If this happened with the EEPROM\n"
+                "programmed by FTDI tools, please report to libftdi@developer.intra2net.com\n");
 
 
-    // Addr 0C: USB version low byte when 0x0A bit 4 is set
-    // Addr 0D: USB version high byte when 0x0A bit 4 is set
-    if ((eeprom->change_usb_version == 1) || ftdi->type == TYPE_2232C)
-    {
-        eeprom->usb_version = buf[0x0C] + (buf[0x0D] << 8);
-    }
+    // Addr 0C: USB version low byte when 0x0A
+    // Addr 0D: USB version high byte when 0x0A 
+    eeprom->usb_version = buf[0x0C] + (buf[0x0D] << 8);
 
     // Addr 0E: Offset of the manufacturer string + 0x80, calculated later
     // Addr 0F: Length of manufacturer string
@@ -2726,12 +2722,12 @@ int ftdi_eeprom_decode(struct ftdi_context *ftdi, unsigned char *buf, int size,
         if (ftdi->type >= TYPE_2232C)
             fprintf(stdout,"Channel A has Mode %s%s%s\n", 
                     channel_mode[eeprom->channel_a_type],
-                    (eeprom->channel_a_driver)?" VCP":"",
+                    (eeprom->channel_a_driver)?" D2XX":"",
                     (eeprom->high_current_a)?" High Currenr IO":"");
         if (ftdi->type >= TYPE_2232C)
             fprintf(stdout,"Channel B has Mode %s%s%s\n", 
                     channel_mode[eeprom->channel_b_type],
-                    (eeprom->channel_b_driver)?" VCP":"",
+                    (eeprom->channel_b_driver)?" D2XX":"",
                     (eeprom->high_current_b)?" High Currenr IO":"");
         if ((ftdi->type == TYPE_2232H) || (ftdi->type == TYPE_4232H)) 
         {