Re: [U-Boot] [PATCH v2 02/10] drivers: spi: consider command bytes when sending transfers

On Mon, May 22, 2017 at 11:51 PM, Álvaro Fernández Rojas noltari@gmail.com wrote:

Command bytes are part of the written bytes and they should be taken into account when sending a spi transfer.

Signed-off-by: Álvaro Fernández Rojas noltari@gmail.com

v2: Introduce changes requested by Simon Glass:

• Always include command bytes when determining max write size.

drivers/mtd/spi/spi_flash.c | 2 +- include/spi.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c index e44c10f..5b8cbc9 100644 --- a/drivers/mtd/spi/spi_flash.c +++ b/drivers/mtd/spi/spi_flash.c @@ -380,7 +380,7 @@ int spi_flash_cmd_write_ops(struct spi_flash *flash, u32 offset,

            if (spi->max_write_size)
                    chunk_len = min(chunk_len,

•                                   (size_t)spi->max_write_size);
  

•                                   spi->max_write_size - len);
  

This can be squashed with 01/10 with proper commit message?

thanks!