[U-Boot] [PATCH v3 1/2] cmd_sf: let "sf update" erase last sector as a whole

Jagan Teki jagannadh.teki at gmail.com
Tue Aug 6 21:02:22 CEST 2013


On 07-08-2013 00:27, Jagan Teki wrote:
> On 04-07-2013 00:03, Gerlando Falauto wrote:
>> make "sf update" work with unaligned `len' parameter, by deleting the
>> whole last sector before writing, so to allow for:
>>
>>   sf update ${load_addr_r} 0 ${filesize}
>>
>> Signed-off-by: Gerlando Falauto <gerlando.falauto at keymile.com>
>> Cc: Valentin Longchamp <valentin.longchamp at keymile.com>
>> Cc: Holger Brunck <holger.brunck at keymile.com>
>> Acked-by: Simon Glass <sjg at chromium.org>
>> ---
>>   common/cmd_sf.c | 3 ++-
>>   1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/common/cmd_sf.c b/common/cmd_sf.c
>> index 19b0dc9..ab35a94 100644
>> --- a/common/cmd_sf.c
>> +++ b/common/cmd_sf.c
>> @@ -160,7 +160,8 @@ static const char *spi_flash_update_block(struct
>> spi_flash *flash, u32 offset,
>>           *skipped += len;
>>           return NULL;
>>       }
>> -    if (spi_flash_erase(flash, offset, len))
>> +    /* Erase the entire sector */
>> +    if (spi_flash_erase(flash, offset, flash->sector_size))
>>           return "erase";
>>       if (spi_flash_write(flash, offset, len, buf))
>>           return "write";
>>
> Reviewed-by: Jagannadha Sutradharudu Teki <jagannadh.teki at gmail.com>
>
> --
> Thanks,
> Jagan.

Applied to u-boot-spi/master

--
Thanks,
Jagan.


More information about the U-Boot mailing list