On 07/25/2019 02:19 PM, tudor.amba...@microchip.com wrote: > static int spi_nor_spimem_xfer_reg(struct spi_nor *nor, struct spi_mem_op *op) > { > if (!op || (op->data.nbytes && !nor->cmd_buf))
!nor->cmd_buf can't be NULL, we can get rid of this check too, and use spi_mem_exec_op() directly when interacting with registers. > return -EINVAL; > > return spi_mem_exec_op(nor->spimem, op); > }