simplicity_sdk: Patch to handle callbacks in IRQ

Updated the IRQ handler in sl_i2c.c to manage
callbacks within the SiSDK layer. In Zephyr,
the driver utilizes dispatch functions, and
callbacks should not be invoked directly within
these functions, as Zephyr handles callback
execution in its own IRQ handler.
The callback logic has therefore been moved to the
SiSDK-specific handler to align with Zephyr's
interrupt and callback management practices.

Upstream-Status: Pending
Signed-off-by: S Mohamed Fiaz <[email protected]>

Author: fimohame

simplicity_sdk/platform/driver/i2c/src/sl_i2c.c

@@ -1597,9 +1597,6 @@ void sli_i2c_leader_dispatch_interrupt(sli_i2c_instance_t *sl_i2c_instance)
     if (sl_i2c_instance->transfer_event == SL_I2C_EVENT_IN_PROGRESS) {
       sl_i2c_instance->transfer_event = SL_I2C_EVENT_COMPLETED;
     }
-    if (sl_i2c_instance->callback) {
-      sl_i2c_instance->callback(sl_i2c_instance->transfer_event, sl_i2c_instance->context);
-    }
   } else if (pending_irq & I2C_IF_NACK) {
     sl_hal_i2c_clear_interrupts(i2c_base_addr, I2C_IF_NACK);
     switch (sl_i2c_instance->state) {
@@ -1688,9 +1685,6 @@ void sli_i2c_follower_dispatch_interrupt(sli_i2c_instance_t *sl_i2c_instance)
     if (sl_i2c_instance->transfer_event == SL_I2C_EVENT_IN_PROGRESS) {
       sl_i2c_instance->transfer_event = SL_I2C_EVENT_COMPLETED;
     }
-    if (sl_i2c_instance->callback) {
-      sl_i2c_instance->callback(sl_i2c_instance->transfer_event, sl_i2c_instance->context);
-    }
   } else if (pending_irq & I2C_IF_ADDR) {
     (i2c_base_addr)->CTRL_SET = I2C_CTRL_AUTOACK;
     sl_hal_i2c_clear_interrupts(i2c_base_addr, _I2C_IF_MASK);
@@ -1763,6 +1757,11 @@ static void i2c_common_irq_handler(sli_i2c_instance_t *sl_i2c_instance)
     } else {  // Follower mode
       sli_i2c_follower_dispatch_interrupt(sl_i2c_instance);
     }
+    if (sl_i2c_instance->transfer_event != SL_I2C_EVENT_IN_PROGRESS) {
+      if (sl_i2c_instance->callback) {
+        sl_i2c_instance->callback(sl_i2c_instance->transfer_event, sl_i2c_instance->context);
+      }
+    }
   }
 }