On Tue, Apr 20, 2021 at 02:29:39PM +0800, Zhengxun Li wrote: > - return spi_mem_default_supports_op(mem, op); > + all_false = !op->cmd.dtr && !op->addr.dtr && !op->dummy.dtr && > + !op->data.dtr; > + > + if (all_false) This feels like we might want a spi_mem_op_is_dtr() helper? I can see other controllers wanting a similar check.