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