Staging: et131x: eeprom remove features
authorAlan Cox <alan@linux.intel.com>
Thu, 27 Aug 2009 10:01:04 +0000 (11:01 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 15 Sep 2009 19:02:28 +0000 (12:02 -0700)
We only read eeprom id 0, in byte mode - so the rest can go away

Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/et131x/et1310_eeprom.c
drivers/staging/et131x/et1310_eeprom.h
drivers/staging/et131x/et131x_initpci.c

index 94538624c103d586d38e8dab8c0ad677d3f8a441..7b2e4ea2ad09fc0b162fa56d39acb1d7843335da 100644 (file)
  * @etdev: pointer to our private adapter structure
  * @addr: the address to write
  * @data: the value to write
- * @eeprom_id: the ID of the EEPROM
- * @addrmode: how the EEPROM is to be accessed
  *
  * Returns SUCCESS or FAILURE
  */
-int EepromWriteByte(struct et131x_adapter *etdev, u32 addr,
-                       u8 data, u32 eeprom_id,
-                       u32 addrmode)
+int EepromWriteByte(struct et131x_adapter *etdev, u32 addr, u8 data)
 {
        struct pci_dev *pdev = etdev->pdev;
        int index;
@@ -238,9 +234,6 @@ int EepromWriteByte(struct et131x_adapter *etdev, u32 addr,
        control = 0;
        control |= LBCIF_CONTROL_LBCIF_ENABLE | LBCIF_CONTROL_I2C_WRITE;
 
-       if (addrmode == DUAL_BYTE)
-               control |= LBCIF_CONTROL_TWO_BYTE_ADDR;
-
        if (pci_write_config_byte(pdev, LBCIF_CONTROL_REGISTER_OFFSET,
                                  control)) {
                return FAILURE;
@@ -249,8 +242,6 @@ int EepromWriteByte(struct et131x_adapter *etdev, u32 addr,
        i2c_wack = 1;
 
        /* Prepare EEPROM address for Step 3 */
-       addr |= (addrmode == DUAL_BYTE) ?
-           (eeprom_id << 16) : (eeprom_id << 8);
 
        for (retries = 0; retries < MAX_NUM_WRITE_RETRIES; retries++) {
                /* Step 3:*/
@@ -357,9 +348,7 @@ int EepromWriteByte(struct et131x_adapter *etdev, u32 addr,
  *
  * Returns SUCCESS or FAILURE
  */
-int EepromReadByte(struct et131x_adapter *etdev, u32 addr,
-                      u8 *pdata, u32 eeprom_id,
-                      u32 addrmode)
+int EepromReadByte(struct et131x_adapter *etdev, u32 addr, u8 *pdata)
 {
        struct pci_dev *pdev = etdev->pdev;
        int index;
@@ -427,17 +416,12 @@ int EepromReadByte(struct et131x_adapter *etdev, u32 addr,
        control = 0;
        control |= LBCIF_CONTROL_LBCIF_ENABLE;
 
-       if (addrmode == DUAL_BYTE)
-               control |= LBCIF_CONTROL_TWO_BYTE_ADDR;
-
        if (pci_write_config_byte(pdev, LBCIF_CONTROL_REGISTER_OFFSET,
                                  control)) {
                return FAILURE;
        }
 
        /* Step 3: */
-       addr |= (addrmode == DUAL_BYTE) ?
-           (eeprom_id << 16) : (eeprom_id << 8);
 
        if (pci_write_config_dword(pdev, LBCIF_ADDRESS_REGISTER_OFFSET,
                                   addr)) {
index 8033743e888e0ce8504ab12c22b121dd6190d23c..d8ac9a0439e2ae7038d03e0cdd47ca395fba75e8 100644 (file)
 #define FAILURE                1
 #endif
 
-#ifndef READ
-#define READ           0
-#define WRITE          1
-#endif
-
-#ifndef SINGLE_BYTE
-#define SINGLE_BYTE    0
-#define DUAL_BYTE      1
-#endif
-
 /* Forward declaration of the private adapter structure */
 struct et131x_adapter;
 
 int32_t EepromWriteByte(struct et131x_adapter *adapter, u32 unAddress,
-                       u8 bData, u32 unEepromId,
-                       u32 unAddressingMode);
+                       u8 bData);
 int32_t EepromReadByte(struct et131x_adapter *adapter, u32 unAddress,
-                      u8 *pbData, u32 unEepromId,
-                      u32 unAddressingMode);
+                       u8 *pbData);
 
 #endif /* _ET1310_EEPROM_H_ */
index a719e903eed995d856526905f45f58c33517c1b2..bea9918d1686d03493e32f6ffdb0431899db72f6 100644 (file)
@@ -337,8 +337,7 @@ int et131x_find_adapter(struct et131x_adapter *adapter, struct pci_dev *pdev)
                         * corruption seen with 1310 B Silicon
                         */
                        for (nLoop = 0; nLoop < 3; nLoop++) {
-                               EepromWriteByte(adapter, nLoop, temp[nLoop],
-                                               0, SINGLE_BYTE);
+                               EepromWriteByte(adapter, nLoop, temp[nLoop]);
                        }
                }
 
@@ -364,8 +363,8 @@ int et131x_find_adapter(struct et131x_adapter *adapter, struct pci_dev *pdev)
        /* Read the EEPROM for information regarding LED behavior. Refer to
         * ET1310_phy.c, et131x_xcvr_init(), for its use.
         */
-       EepromReadByte(adapter, 0x70, &adapter->eepromData[0], 0, SINGLE_BYTE);
-       EepromReadByte(adapter, 0x71, &adapter->eepromData[1], 0, SINGLE_BYTE);
+       EepromReadByte(adapter, 0x70, &adapter->eepromData[0]);
+       EepromReadByte(adapter, 0x71, &adapter->eepromData[1]);
 
        if (adapter->eepromData[0] != 0xcd)
                /* Disable all optional features */