fix the bug when those are some data in wire

[watershade]

1. when I use rp2040 to request data from MCP3425, I find it can not read anything and I debug the code and find because of those code:

wire->requestFrom(address, len);
  if (wire->available() != len)
    return errorReadFailed;

Because those many be some data inside it before read them out. (The data if generated when I try to test if there are any data inside of it). I can clear data out of just flush the old data before read new data. So I fix it.

2. there are those functions have been changed from static funtion to normal one:

  uint8_t generalCallReset(void);
  uint8_t generalCallLatch(void);
  uint8_t generalCallConversion(void);

But those examples to use them haven't been changed, I try to fix them.

3. the version you expect is "1.0.5", but it is "1.0.4" now. So I modify it.

[vzahradnik]

Why do you change this call from MCP342x::generalCallReset(). Is it necessary to fix your bug?

[watershade]

Yes, it is necessary. It cannot be compiled (I used platformio not arduino ide) because it is not an static member function anymore.
In the original code, "Wire" is an static member, so you can define them to static member functions. But when you use an changeable "wire"(It is the functions we need), it become an normal functions. I think it is the reason why you remove "static" in 647b2ce. So I chose to change MCP342x to adc. Thanks.

[vzahradnik]

I don't like the approach to fix a bug with two solutions which are affected by a build flag. Do you really need both? If not, I'd suggest to just flush the buffer.

[watershade]

Yes, best to flush it. But in Arduino_mbed, flush haven't been implemented.:

void arduino::MbedI2C::flush() {
}

It is the reason why I give another solution. The 2nd solution have a bug unintentionally，you can fix if to while like this:

  // Solution2: Just to read out every useless data before get useful one
  while(wire->available() > 0 ){
    wire->read();
  }