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 > <nolt...@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 >>> <nolt...@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 <nolt...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot