I2C: remove useless parameter in onReceiveService()

With introduction of 'I2C multi instances: Remove static qualifier on TwoWire attributes'
parameters 'inBytes' and 'numBytes' become useless as they are part of new obj structure parameter.

Author: ABOSTM

libraries/Wire/src/Wire.cpp

@@ -394,8 +394,11 @@ void TwoWire::flush(void)
 }
 
 // behind the scenes function that is called when data is received
-void TwoWire::onReceiveService(i2c_t *obj, uint8_t *inBytes, int numBytes)
+void TwoWire::onReceiveService(i2c_t *obj)
 {
+  uint8_t *inBytes = (uint8_t *) obj->i2cTxRxBuffer;
+  int numBytes = obj->slaveRxNbData;
+
   TwoWire *TW = (TwoWire *)(obj->__this);
 
   // don't bother if user hasn't registered a callback

libraries/Wire/src/Wire.h

@@ -56,7 +56,7 @@ class TwoWire : public Stream {
     void (*user_onRequest)(void);
     void (*user_onReceive)(int);
     static void onRequestService(i2c_t *);
-    static void onReceiveService(i2c_t *, uint8_t *, int);
+    static void onReceiveService(i2c_t *);
 
     void allocateRxBuffer(size_t length);
     void allocateTxBuffer(size_t length);

libraries/Wire/src/utility/twi.c

@@ -948,7 +948,7 @@ i2c_t *get_i2c_obj(I2C_HandleTypeDef *hi2c)
   * @param  function: callback function to use
   * @retval None
   */
-void i2c_attachSlaveRxEvent(i2c_t *obj, void (*function)(i2c_t *, uint8_t *, int))
+void i2c_attachSlaveRxEvent(i2c_t *obj, void (*function)(i2c_t *))
 {
   if ((obj != NULL) && (function != NULL)) {
     obj->i2c_onSlaveReceive = function;
@@ -1026,7 +1026,7 @@ void HAL_I2C_ListenCpltCallback(I2C_HandleTypeDef *hi2c)
   /*  Previous master transaction now ended, so inform upper layer if needed
    *  then prepare for listening to next request */
   if ((obj->slaveMode == SLAVE_MODE_RECEIVE) && (obj->slaveRxNbData != 0)) {
-    obj->i2c_onSlaveReceive(obj, (uint8_t *) obj->i2cTxRxBuffer, obj->slaveRxNbData);
+    obj->i2c_onSlaveReceive(obj);
   }
   obj->slaveMode = SLAVE_MODE_LISTEN;
   obj->slaveRxNbData = 0;

libraries/Wire/src/utility/twi.h

@@ -107,7 +107,7 @@ struct i2c_s {
   IRQn_Type irqER;
 #endif /* !STM32F0xx && !STM32G0xx && !STM32L0xx */
   volatile int slaveRxNbData; // Number of accumulated bytes received in Slave mode
-  void (*i2c_onSlaveReceive)(i2c_t *, uint8_t *, int);
+  void (*i2c_onSlaveReceive)(i2c_t *);
   void (*i2c_onSlaveTransmit)(i2c_t *);
   volatile uint8_t i2cTxRxBuffer[I2C_TXRX_BUFFER_SIZE];
   volatile uint8_t i2cTxRxBufferSize;
@@ -139,7 +139,7 @@ i2c_status_e i2c_master_read(i2c_t *obj, uint8_t dev_address, uint8_t *data, uin
 
 i2c_status_e i2c_IsDeviceReady(i2c_t *obj, uint8_t devAddr, uint32_t trials);
 
-void i2c_attachSlaveRxEvent(i2c_t *obj, void (*function)(i2c_t *, uint8_t *, int));
+void i2c_attachSlaveRxEvent(i2c_t *obj, void (*function)(i2c_t *));
 void i2c_attachSlaveTxEvent(i2c_t *obj, void (*function)(i2c_t *));
 
 #ifdef __cplusplus