Looks good to me. Only one comment:
> + /* RX_LOW_PRI */
> + if (int_src & WCN36XX_DXE_INT_CH1_MASK) {
> + wcn36xx_dxe_read_register(wcn, WCN36XX_DXE_0_CH1_STATUS,
> + &int_reason);
is int_reason checked anywher?
--
Best regards,
Eugene