Hi Jagan,
El 07/06/2017 a las 19:28, Jagan Teki escribió:
On Wed, Jun 7, 2017 at 9:03 PM, Álvaro Fernández Rojas noltari@gmail.com wrote:
Hi Jagan,
El 7/6/17 a las 9:30, Jagan Teki escribió:
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?
I think it's better if we keep it as two patches, because they are different features...
Fine, but adding immediate change followed by previous change deletion doesn't look feature that exist.
Nope, there's no previous change deletion. First patch adds support for limited reads and second patch fixes limited writes...
thanks!
Thanks