On Fri, Nov 05, 2021 at 18:23:45 +0900, Masami Hiramatsu wrote:
> Fix the number of erase blocks by rounding up the result.
> The erase blocks must include the last block covered by the
> length bytes.
> 
> Signed-off-by: Masami Hiramatsu <masami.hirama...@linaro.org>
> Reported-by: Kazuhiko Sakamoto <sakamoto.kazuh...@socionext.com>

Reviewed-by: Leif Lindholm <l...@nuviainc.com>

> ---
>  .../SynQuacerPlatformFlashAccessLib.c              |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git 
> a/Silicon/Socionext/SynQuacer/Library/SynQuacerPlatformFlashAccessLib/SynQuacerPlatformFlashAccessLib.c
>  
> b/Silicon/Socionext/SynQuacer/Library/SynQuacerPlatformFlashAccessLib/SynQuacerPlatformFlashAccessLib.c
> index bded74dc4f..ad4021cf59 100644
> --- 
> a/Silicon/Socionext/SynQuacer/Library/SynQuacerPlatformFlashAccessLib/SynQuacerPlatformFlashAccessLib.c
> +++ 
> b/Silicon/Socionext/SynQuacer/Library/SynQuacerPlatformFlashAccessLib/SynQuacerPlatformFlashAccessLib.c
> @@ -283,7 +283,7 @@ PerformFlashWriteWithProgress (
>    DEBUG ((DEBUG_INFO, "%a: erasing 0x%llx bytes at address %llx (LBA 
> 0x%lx)\n",
>      __FUNCTION__, Length, FlashAddress, Lba));
>  
> -  Status = Fvb->EraseBlocks (Fvb, Lba, Length / BlockSize,
> +  Status = Fvb->EraseBlocks (Fvb, Lba, (Length + BlockSize - 1) / BlockSize,
>                    EFI_LBA_LIST_TERMINATOR);
>    if (EFI_ERROR (Status)) {
>      DEBUG ((DEBUG_ERROR, "%a: Fvb->EraseBlocks () failed - %r\n",
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#84102): https://edk2.groups.io/g/devel/message/84102
Mute This Topic: https://groups.io/mt/86836380/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to