drivers: wifi: Remove the RPU awake assumption in IRQ

It was assumed that RPU would stay awake while servicing an interrupt,
but in few scenarios we had seen RPU entering sleep with a pending
interrupt, so, independent of the state (IRQ or not), wakeup RPU if its
asleep.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>

drivers/wifi/nrf700x/osal/hw_if/hal/inc/hal_structs.h

@@ -233,7 +233,6 @@ struct nrf_wifi_hal_dev_ctx {
  void *rpu_ps_timer;
  void *rpu_ps_lock;
  bool dbg_enable;
- bool irq_ctx;
  bool rpu_fw_booted;
 #endif /* CONFIG_NRF_WIFI_LOW_POWER */
  char *event_data;

drivers/wifi/nrf700x/osal/hw_if/hal/src/hal_api.c

@@ -423,11 +423,9 @@ enum nrf_wifi_status hal_rpu_ps_wake(struct nrf_wifi_hal_dev_ctx *hal_dev_ctx)
 #endif /* CONFIG_NRF_WIFI_RPU_RECOVERY_PS_STATE_DEBUG */
 
 out:
- if (!hal_dev_ctx->irq_ctx) {
- nrf_wifi_osal_timer_schedule(hal_dev_ctx->hpriv->opriv,
- hal_dev_ctx->rpu_ps_timer,
- CONFIG_NRF700X_RPU_PS_IDLE_TIMEOUT_MS);
- }
+ nrf_wifi_osal_timer_schedule(hal_dev_ctx->hpriv->opriv,
+ hal_dev_ctx->rpu_ps_timer,
+ CONFIG_NRF700X_RPU_PS_IDLE_TIMEOUT_MS);
  return status;
 }
 
@@ -515,13 +513,6 @@ static void hal_rpu_ps_deinit(struct nrf_wifi_hal_dev_ctx *hal_dev_ctx)
  hal_dev_ctx->rpu_ps_lock);
 }
 
-
-static void hal_rpu_ps_set_state(struct nrf_wifi_hal_dev_ctx *hal_dev_ctx,
- enum RPU_PS_STATE ps_state)
-{
- hal_dev_ctx->rpu_ps_state = ps_state;
-}
-
 enum nrf_wifi_status nrf_wifi_hal_get_rpu_ps_state(
  struct nrf_wifi_hal_dev_ctx *hal_dev_ctx,
  int *rpu_ps_ctrl_state)
@@ -1556,9 +1547,6 @@ enum nrf_wifi_status nrf_wifi_hal_irq_handler(void *data)
  struct nrf_wifi_hal_dev_ctx *hal_dev_ctx = NULL;
  enum nrf_wifi_status status = NRF_WIFI_STATUS_FAIL;
  unsigned long flags = 0;
-#ifdef CONFIG_NRF_WIFI_LOW_POWER
- enum RPU_PS_STATE ps_state = RPU_PS_STATE_ASLEEP;
-#endif /* CONFIG_NRF_WIFI_LOW_POWER */
  bool do_rpu_recovery = false;
 
  hal_dev_ctx = (struct nrf_wifi_hal_dev_ctx *)data;
@@ -1578,25 +1566,8 @@ enum nrf_wifi_status nrf_wifi_hal_irq_handler(void *data)
  goto unlock;
  }
 
-#ifdef CONFIG_NRF_WIFI_LOW_POWER
- ps_state = hal_dev_ctx->rpu_ps_state;
- hal_rpu_ps_set_state(hal_dev_ctx,
- RPU_PS_STATE_AWAKE);
-#ifdef CONFIG_NRF_WIFI_LOW_POWER_DBG
- hal_dev_ctx->irq_ctx = true;
-#endif /* CONFIG_NRF_WIFI_LOW_POWER_DBG */
-#endif /* CONFIG_NRF_WIFI_LOW_POWER */
-
  status = hal_rpu_irq_process(hal_dev_ctx, &do_rpu_recovery);
 
-#ifdef CONFIG_NRF_WIFI_LOW_POWER
- hal_rpu_ps_set_state(hal_dev_ctx,
- ps_state);
-#ifdef CONFIG_NRF_WIFI_LOW_POWER_DBG
- hal_dev_ctx->irq_ctx = false;
-#endif /* CONFIG_NRF_WIFI_LOW_POWER_DBG */
-#endif /* CONFIG_NRF_WIFI_LOW_POWER */
-
  if (status != NRF_WIFI_STATUS_SUCCESS) {
  goto unlock;
  }