Staging: et131x: kill SUCCESS and FAILURE defines
authorAlan Cox <alan@linux.intel.com>
Tue, 6 Oct 2009 14:46:24 +0000 (15:46 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 11 Dec 2009 20:23:05 +0000 (12:23 -0800)
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/et1310_phy.h

index c853a2c243a87ea5b3723f73d3558fe633a0d702..520a7c938c395ed8b6a17fd25db8b9007c3f6e5a 100644 (file)
  * @addr: the address to write
  * @data: the value to write
  *
- * Returns SUCCESS or FAILURE
+ * Returns 1 for a successful write.
  */
 int EepromWriteByte(struct et131x_adapter *etdev, u32 addr, u8 data)
 {
@@ -227,7 +227,7 @@ int EepromWriteByte(struct et131x_adapter *etdev, u32 addr, u8 data)
        }
 
        if (err || (index >= MAX_NUM_REGISTER_POLLS))
-               return FAILURE;
+               return 0;
 
        /* Step 2: */
        control = 0;
@@ -235,7 +235,7 @@ int EepromWriteByte(struct et131x_adapter *etdev, u32 addr, u8 data)
 
        if (pci_write_config_byte(pdev, LBCIF_CONTROL_REGISTER_OFFSET,
                                  control)) {
-               return FAILURE;
+               return 0;
        }
 
        i2c_wack = 1;
@@ -334,7 +334,7 @@ int EepromWriteByte(struct et131x_adapter *etdev, u32 addr, u8 data)
                index++;
        }
 
-       return writeok ? SUCCESS : FAILURE;
+       return writeok;
 }
 
 /**
@@ -345,7 +345,7 @@ int EepromWriteByte(struct et131x_adapter *etdev, u32 addr, u8 data)
  * @eeprom_id: the ID of the EEPROM
  * @addrmode: how the EEPROM is to be accessed
  *
- * Returns SUCCESS or FAILURE
+ * Returns 1 for a successful read
  */
 int EepromReadByte(struct et131x_adapter *etdev, u32 addr, u8 *pdata)
 {
@@ -409,7 +409,7 @@ int EepromReadByte(struct et131x_adapter *etdev, u32 addr, u8 *pdata)
        }
 
        if (err || (index >= MAX_NUM_REGISTER_POLLS))
-               return FAILURE;
+               return 0;
 
        /* Step 2: */
        control = 0;
@@ -417,14 +417,14 @@ int EepromReadByte(struct et131x_adapter *etdev, u32 addr, u8 *pdata)
 
        if (pci_write_config_byte(pdev, LBCIF_CONTROL_REGISTER_OFFSET,
                                  control)) {
-               return FAILURE;
+               return 0;
        }
 
        /* Step 3: */
 
        if (pci_write_config_dword(pdev, LBCIF_ADDRESS_REGISTER_OFFSET,
                                   addr)) {
-               return FAILURE;
+               return 0;
        }
 
        /* Step 4: */
@@ -446,10 +446,10 @@ int EepromReadByte(struct et131x_adapter *etdev, u32 addr, u8 *pdata)
        }
 
        if (err || (index >= MAX_NUM_REGISTER_POLLS))
-               return FAILURE;
+               return 0;
 
        /* Step 6: */
        *pdata = EXTRACT_DATA_REGISTER(dword1);
 
-       return (status & LBCIF_STATUS_ACK_ERROR) ? FAILURE : SUCCESS;
+       return (status & LBCIF_STATUS_ACK_ERROR) ? 0 : 1;
 }
index d8ac9a0439e2ae7038d03e0cdd47ca395fba75e8..b329623ace9f8d4de450aaa4b8defbb94d6785a2 100644 (file)
 
 #include "et1310_address_map.h"
 
-#ifndef SUCCESS
-#define SUCCESS                0
-#define FAILURE                1
-#endif
-
 /* Forward declaration of the private adapter structure */
 struct et131x_adapter;
 
-int32_t EepromWriteByte(struct et131x_adapter *adapter, u32 unAddress,
+int EepromWriteByte(struct et131x_adapter *adapter, u32 unAddress,
                        u8 bData);
-int32_t EepromReadByte(struct et131x_adapter *adapter, u32 unAddress,
+int EepromReadByte(struct et131x_adapter *adapter, u32 unAddress,
                        u8 *pbData);
 
 #endif /* _ET1310_EEPROM_H_ */
index 080656c6142c18135f08788c19a2196360607b06..6978b25c7e13190555b3fc8e6fb4d5f867fb8978 100644 (file)
@@ -61,9 +61,6 @@
 
 #include "et1310_address_map.h"
 
-#define TRUEPHY_SUCCESS 0
-#define TRUEPHY_FAILURE 1
-
 /* MI Register Addresses */
 #define MI_CONTROL_REG                      0
 #define MI_STATUS_REG                       1