[PATCH] dfu: dfu_mtd: remove the mtd_block_op error when mtd_lock is not supported

Patrick DELAUNAY patrick.delaunay at foss.st.com
Fri May 28 11:32:54 CEST 2021


Hi,

On 3/10/21 12:15 PM, Sughosh Ganu wrote:
>
>
> On Wed, 10 Mar 2021 at 14:57, Patrick Delaunay 
> <patrick.delaunay at foss.st.com <mailto:patrick.delaunay at foss.st.com>> 
> wrote:
>
>     Fix the result of DFU_OP_WRITE operation in mtd_block_op function
>     when mtd_lock is not supported (-EOPNOTSUPP) to avoid DFU stack
>     error on the DFU manifestation of the MTD device, when
>     dfu_flush_medium_mtd is called.
>
>     Without this patch, dfu-util failed on dfuERROR state at the end
>     of the write operation on the alternate even if MTD write
>     opeartion is correctly performed.
>
>     $> dfu-util -a 3 -D test.bin
>     ....
>     DFU mode device DFU version 0110
>     Device returned transfer size 4096
>     Copying data from PC to DFU device
>     ....
>     Download        [=========================] 100%  225469 bytes
>     Download done.
>     state(10) = dfuERROR, status(14) = Something went wrong,
>       but the device does not know what it was Done!
>
>     Fixes: 65f3fc18fc1e ("dfu_mtd: Add provision to unlock mtd device")
>     Signed-off-by: Patrick Delaunay <patrick.delaunay at foss.st.com
>     <mailto:patrick.delaunay at foss.st.com>>
>     ---
>
>
> Acked-by: Sughosh Ganu <sughosh.ganu at linaro.org 
> <mailto:sughosh.ganu at linaro.org>>
>
>
>      drivers/dfu/dfu_mtd.c | 4 +++-
>      1 file changed, 3 insertions(+), 1 deletion(-)
>
>     diff --git a/drivers/dfu/dfu_mtd.c b/drivers/dfu/dfu_mtd.c
>     index ca67585a7e..ec40b8f6bb 100644
>     --- a/drivers/dfu/dfu_mtd.c
>     +++ b/drivers/dfu/dfu_mtd.c
>     @@ -150,7 +150,9 @@ static int mtd_block_op(enum dfu_op op, struct
>     dfu_entity *dfu,
>                     /* Write done, lock again */
>                     debug("Locking the mtd device\n");
>                     ret = mtd_lock(mtd, lock_ofs, lock_len);
>     -               if (ret && ret != -EOPNOTSUPP)
>     +               if (ret == -EOPNOTSUPP)
>     +                       ret = 0;
>     +               else if (ret)
>                             printf("MTD device lock failed\n");
>             }
>             return ret;
>     -- 
>     2.17.1
>

Applied to u-boot-stm/master, thanks!

See request in

Re: [PATCH v4 12/14] dfu_mtd: Ignore non-implemented lock device failure

http://patchwork.ozlabs.org/project/uboot/patch/162140319389.47256.629728945246178860.stgit@localhost/#2685391

http://u-boot.10912.n7.nabble.com/PATCH-v4-00-14-arm64-synquacer-Add-SynQuacer-DeveloperBox-support-tp450231p450243.html


Patrick



More information about the U-Boot mailing list