On 17/05/21 02:14PM, Zhengxun Li wrote: > Driver patch for octal DTR mode support. > > Owing to the spi_mem_default_supports_op() is not support dtr > operation. Based on Pratyush patch "spi: spi-mem: add spi_mem_dtr I don't like the "Based on Pratyush patch". Please mention the commit hash so it is much easier to find later. For example, saying based on commit 539cf68cd51b (spi: spi-mem: add spi_mem_dtr_supports_op(), 2021-02-04) is much better. But while we are on this topic, I don't think it is needed all that much. People use pre-existing APIs all the time. I don't see any need to specifically point it out. So I think saying "Call spi_mem_dtr_supports_op() to ..." is better. > _supports_op()" add spi_mem_dtr_supports_op() to support dtr and > keep checking the buswidth and command bytes. > > Signed-off-by: Zhengxun Li <zhengxunli@xxxxxxxxxxx> > --- -- Regards, Pratyush Yadav Texas Instruments Inc.