Rewrite TwoWire with using HardwareI2C

- implement proposal espressif#8818 (comment)
  to bring the HARDWARE interface into compliance

Author: safocl

cores/esp32/HardwareI2C.h

@@ -0,0 +1,42 @@
+/*
+  Copyright (c) 2016 Arduino LLC.  All right reserved.
+
+  This library is free software; you can redistribute it and/or
+  modify it under the terms of the GNU Lesser General Public
+  License as published by the Free Software Foundation; either
+  version 2.1 of the License, or (at your option) any later version.
+
+  This library is distributed in the hope that it will be useful,
+  but WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
+  See the GNU Lesser General Public License for more details.
+
+  You should have received a copy of the GNU Lesser General Public
+  License along with this library; if not, write to the Free Software
+  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+*/
+
+#pragma once
+
+#include <inttypes.h>
+#include "Stream.h"
+
+class HardwareI2C : public Stream
+{
+  public:
+    virtual void begin() = 0;
+    virtual void begin(uint8_t address) = 0;
+    virtual void end() = 0;
+
+    virtual void setClock(uint32_t freq) = 0;
+  
+    virtual void beginTransmission(uint8_t address) = 0;
+    virtual uint8_t endTransmission(bool stopBit) = 0;
+    virtual uint8_t endTransmission(void) = 0;
+
+    virtual size_t requestFrom(uint8_t address, size_t len, bool stopBit) = 0;
+    virtual size_t requestFrom(uint8_t address, size_t len) = 0;
+
+    virtual void onReceive(void(*)(int)) = 0;
+    virtual void onRequest(void(*)(void)) = 0;
+};

libraries/Wire/src/Wire.cpp

@@ -145,7 +145,7 @@ bool TwoWire::setPins(int sdaPin, int sclPin)
     return !i2cIsInit(num);
 }
 
-bool TwoWire::allocateWireBuffer(void)
+bool TwoWire::allocateWireBuffer()
 {
     // or both buffer can be allocated or none will be
     if (rxBuffer == NULL) {
@@ -167,7 +167,7 @@ bool TwoWire::allocateWireBuffer(void)
     return true;
 }
 
-void TwoWire::freeWireBuffer(void)
+void TwoWire::freeWireBuffer()
 {
     if (rxBuffer != NULL) {
         free(rxBuffer);
@@ -409,7 +409,7 @@ uint16_t TwoWire::getTimeOut()
     return _timeOutMillis;
 }
 
-void TwoWire::beginTransmission(uint16_t address)
+void TwoWire::beginTransmission(uint8_t address)
 {
     if(is_slave){
         log_e("Bus is in Slave Mode");
@@ -475,7 +475,7 @@ uint8_t TwoWire::endTransmission(bool sendStop)
     return 4;
 }
 
-size_t TwoWire::requestFrom(uint16_t address, size_t size, bool sendStop)
+size_t TwoWire::requestFrom(uint8_t address, size_t size, bool sendStop)
 {
     if(is_slave){
         log_e("Bus is in Slave Mode");
@@ -524,6 +524,10 @@ size_t TwoWire::requestFrom(uint16_t address, size_t size, bool sendStop)
     return rxLength;
 }
 
+size_t TwoWire::requestFrom(uint8_t address, size_t size){
+	requestFrom(address, size, true);
+}
+
 size_t TwoWire::write(uint8_t data)
 {
     if (txBuffer == NULL){
@@ -548,13 +552,13 @@ size_t TwoWire::write(const uint8_t *data, size_t quantity)
 
 }
 
-int TwoWire::available(void)
+int TwoWire::available()
 {
     int result = rxLength - rxIndex;
     return result;
 }
 
-int TwoWire::read(void)
+int TwoWire::read()
 {
     int value = -1;
     if (rxBuffer == NULL){
@@ -567,7 +571,7 @@ int TwoWire::read(void)
     return value;
 }
 
-int TwoWire::peek(void)
+int TwoWire::peek()
 {
     int value = -1;
     if (rxBuffer == NULL){
@@ -580,70 +584,23 @@ int TwoWire::peek(void)
     return value;
 }
 
-void TwoWire::flush(void)
+void TwoWire::flush()
 {
     rxIndex = 0;
     rxLength = 0;
     txLength = 0;
     //i2cFlush(num); // cleanup
 }
 
-// size_t TwoWire::requestFrom(uint8_t address, size_t len, bool sendStop)
-// {
-//     return requestFrom(static_cast<uint16_t>(address), static_cast<size_t>(len), static_cast<bool>(sendStop));
-// }
-//   
-// uint8_t TwoWire::requestFrom(uint8_t address, uint8_t len, uint8_t sendStop)
-// {
-//     return requestFrom(static_cast<uint16_t>(address), static_cast<size_t>(len), static_cast<bool>(sendStop));
-// }
-//
-// uint8_t TwoWire::requestFrom(uint16_t address, uint8_t len, uint8_t sendStop)
-// {
-//     return requestFrom(address, static_cast<size_t>(len), static_cast<bool>(sendStop));
-// }
-//
-// /* Added to match the Arduino function definition: https://github.com/arduino/ArduinoCore-API/blob/173e8eadced2ad32eeb93bcbd5c49f8d6a055ea6/api/HardwareI2C.h#L39
-//  * See: https://github.com/arduino-libraries/ArduinoECCX08/issues/25
-// */
-// uint8_t TwoWire::requestFrom(uint16_t address, uint8_t len, bool stopBit)
-// {
-//     return requestFrom((uint16_t)address, (size_t)len, stopBit);
-// }
-//
-// uint8_t TwoWire::requestFrom(uint8_t address, uint8_t len)
-// {
-//     return requestFrom(static_cast<uint16_t>(address), static_cast<size_t>(len), true);
-// }
-//
-// uint8_t TwoWire::requestFrom(uint16_t address, uint8_t len)
-// {
-//     return requestFrom(address, static_cast<size_t>(len), true);
-// }
-//
-// uint8_t TwoWire::requestFrom(int address, int len)
-// {
-//     return requestFrom(static_cast<uint16_t>(address), static_cast<size_t>(len), true);
-// }
-//
-// uint8_t TwoWire::requestFrom(int address, int len, int sendStop)
-// {
-//     return static_cast<uint8_t>(requestFrom(static_cast<uint16_t>(address), static_cast<size_t>(len), static_cast<bool>(sendStop)));
-// }
-
-void TwoWire::beginTransmission(int address)
-{
-    beginTransmission(static_cast<uint16_t>(address));
-}
-
-void TwoWire::beginTransmission(uint8_t address)
+void TwoWire::onReceive( void (*function)(int) )
 {
-    beginTransmission(static_cast<uint16_t>(address));
+  user_onReceive = function;
 }
 
-uint8_t TwoWire::endTransmission(void)
+// sets function called on slave read
+void TwoWire::onRequest( void (*function)(void) )
 {
-    return endTransmission(true);
+  user_onRequest = function;
 }
 
 #if SOC_I2C_SUPPORT_SLAVE
@@ -688,17 +645,6 @@ void TwoWire::onRequestService(uint8_t num, void * arg)
     }
 }
 
-void TwoWire::onReceive( void (*function)(int) )
-{
-  user_onReceive = function;
-}
-
-// sets function called on slave read
-void TwoWire::onRequest( void (*function)(void) )
-{
-  user_onRequest = function;
-}
-
 #endif /* SOC_I2C_SUPPORT_SLAVE */
 
 TwoWire Wire = TwoWire(0);

libraries/Wire/src/Wire.h

@@ -35,6 +35,7 @@
 #include "freertos/task.h"
 #include "freertos/semphr.h"
 #endif
+#include "HardwareI2C.h"
 #include "Stream.h"
 
 // WIRE_HAS_BUFFER_SIZE means Wire has setBufferSize()
@@ -48,7 +49,7 @@
 typedef void(*user_onRequest)(void);
 typedef void(*user_onReceive)(uint8_t*, int);
 
-class TwoWire: public Stream
+class TwoWire: public HardwareI2C
 {
 protected:
     uint8_t num;
@@ -79,84 +80,61 @@ class TwoWire: public Stream
     static void onReceiveService(uint8_t, uint8_t*, size_t, bool, void *);
 #endif /* SOC_I2C_SUPPORT_SLAVE */
     bool initPins(int sdaPin, int sclPin);
-    bool allocateWireBuffer(void);
-    void freeWireBuffer(void);
+    bool allocateWireBuffer();
+    void freeWireBuffer();
 
 public:
     TwoWire(uint8_t bus_num);
     ~TwoWire();
+
+    void begin() override final{
+        begin(-1, -1);
+    }
+
+    void begin(uint8_t address) override final
+	{
+        begin(address, -1, -1, 0);
+    }
+
+
+    void end() override;
+
+    void setClock(uint32_t freq) override;
+  
+    void beginTransmission(uint8_t address) override;
+    uint8_t endTransmission(bool stopBit) override;
+    uint8_t endTransmission() override;
+
+    size_t requestFrom(uint8_t address, size_t len, bool stopBit) override;
+    size_t requestFrom(uint8_t address, size_t len) override;
+
+    void onReceive(void(*)(int)) override;
+    void onRequest(void(*)(void)) override;
+
 
     //call setPins() first, so that begin() can be called without arguments from libraries
     bool setPins(int sda, int scl);
 
     bool begin(int sda, int scl, uint32_t frequency=0); // returns true, if successful init of i2c bus
     bool begin(uint8_t slaveAddr, int sda, int scl, uint32_t frequency);
-    // Explicit Overload for Arduino MainStream API compatibility
-    inline bool begin()
-    {
-        return begin(-1, -1, static_cast<uint32_t>(0));
-    }
-    inline bool begin(uint8_t addr)
-    {
-        return begin(addr, -1, -1, 0);
-    }
-	bool end();
 
     size_t setBufferSize(size_t bSize);
 
     void setTimeOut(uint16_t timeOutMillis); // default timeout of i2c transactions is 50ms
     uint16_t getTimeOut();
 
-    bool setClock(uint32_t);
     uint32_t getClock();
 
-    void beginTransmission(uint16_t address);
-    void beginTransmission(uint8_t address);
-    void beginTransmission(int address);
-
-    uint8_t endTransmission(bool sendStop);
-    uint8_t endTransmission(void);
-
-    size_t requestFrom(uint16_t address, size_t size, bool sendStop = true);
-    // uint8_t requestFrom(uint16_t address, uint8_t size, bool sendStop);
-    // uint8_t requestFrom(uint16_t address, uint8_t size, uint8_t sendStop);
-    // size_t requestFrom(uint8_t address, size_t len, bool stopBit);
-    // uint8_t requestFrom(uint16_t address, uint8_t size);
-    // uint8_t requestFrom(uint8_t address, uint8_t size, uint8_t sendStop);
-    // uint8_t requestFrom(uint8_t address, uint8_t size);
-    // uint8_t requestFrom(int address, int size, int sendStop);
-    // uint8_t requestFrom(int address, int size);
-
-    size_t write(uint8_t);
-    size_t write(const uint8_t *, size_t);
-    int available(void);
-    int read(void);
-    int peek(void);
-    void flush(void);
-
-    inline size_t write(const char * s)
-    {
-        return write((uint8_t*) s, strlen(s));
-    }
-    inline size_t write(unsigned long n)
-    {
-        return write((uint8_t)n);
-    }
-    inline size_t write(long n)
-    {
-        return write((uint8_t)n);
-    }
-    inline size_t write(unsigned int n)
-    {
-        return write((uint8_t)n);
-    }
-    inline size_t write(int n)
-    {
-        return write((uint8_t)n);
-    }
+
+
+    size_t write(uint8_t) override;
+    size_t write(const uint8_t *, size_t) override;
+    int available() override;
+    int read() override;
+    int peek() override;
+    void flush() override;
+
 #if SOC_I2C_SUPPORT_SLAVE
-    void onReceive( void (*)(int) );
-    void onRequest( void (*)(void) );
     size_t slaveWrite(const uint8_t *, size_t);
 #endif /* SOC_I2C_SUPPORT_SLAVE */
 };