Remove extra begin() to avoid compiler confusion

Fix #34

Author: nseidle

src/SparkFun_External_EEPROM.cpp

@@ -26,8 +26,14 @@
 #include "Arduino.h"
 #include "Wire.h"
 
-bool ExternalEEPROM::begin(uint8_t deviceAddress, TwoWire &wirePort)
+bool ExternalEEPROM::begin(uint8_t deviceAddress, TwoWire &wirePort, uint8_t WP)
 {
+    if(WP != 255)
+    {
+        pinMode(WP, OUTPUT);
+        digitalWrite(WP, HIGH);
+        settings.wpPin = WP;
+    }
     settings.i2cPort = &wirePort; // Grab which port the user wants us to use
     settings.deviceAddress = deviceAddress;
 
@@ -66,24 +72,6 @@ bool ExternalEEPROM::begin(uint8_t deviceAddress, TwoWire &wirePort)
 
     return true;
 }
-bool ExternalEEPROM::begin(uint8_t deviceAddress, TwoWire &wirePort, uint8_t WP)
-{
-    if(WP != 255)
-    {
-        pinMode(WP, OUTPUT);
-        digitalWrite(WP, HIGH);
-        settings.wpPin = WP;
-    }
-    settings.i2cPort = &wirePort; // Grab which port the user wants us to use
-    settings.deviceAddress = deviceAddress;
-
-    if (isConnected() == false)
-    {
-        return false;
-    }
-
-    return true;
-}
 
 // Erase entire EEPROM
 void ExternalEEPROM::erase(uint8_t toWrite)

src/SparkFun_External_EEPROM.h

@@ -122,7 +122,6 @@ class ExternalEEPROM
     int write(uint32_t eepromLocation, uint8_t dataToWrite);
     int write(uint32_t eepromLocation, const uint8_t *dataToWrite, uint16_t blockSize);
 
-    bool begin(uint8_t deviceAddress = 0b1010000, TwoWire &wirePort = Wire); // By default use the Wire port
     bool begin(uint8_t deviceAddress = 0b1010000, TwoWire &wirePort = Wire, uint8_t WP = 255); // By default use the Wire port
 
     bool isConnected(uint8_t i2cAddress = 255);