On Tue, 19 Sept 2023 at 21:42, Venkatesh Yadav Abbarapu venkatesh.abbarapu@amd.com wrote:
When flash erase is called with size parameter, code is checking if sectors are locked or not. But for checking, the whole device length minus offset is used instead of actual size which should be erased. That's why when only some sectors are locked it is not possible to erase unlocked sectors.
The length is calculated as "length = max_chipsize - offset", flash_is_unlocked() api is getting updated with length which is incorrect. Fix this flash_is_unlocked() api by passing the size parameter.
ZynqMP> sf erase 0 100000 len=0x8000000 which is flash size size=0x100000
We need to update the size in the flash_is_unlocked() api and not the length.
Signed-off-by: Venkatesh Yadav Abbarapu venkatesh.abbarapu@amd.com
Changes in v2:
- Updated the commit description.
cmd/sf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org