[PATCH v4 12/14] dfu_mtd: Ignore non-implemented lock device failure
Masami Hiramatsu
masami.hiramatsu at linaro.org
Wed May 19 08:34:55 CEST 2021
Hi Sughosh,
2021年5月19日(水) 15:07 Sughosh Ganu <sughosh.ganu at linaro.org>:
>
>
>
> On Wed, 19 May 2021 at 11:16, Masami Hiramatsu <masami.hiramatsu at linaro.org> wrote:
>>
>> Ignore the non-implemented lock device failure on writing mtd
>> via DFU. Without this fix, DFU write shows an error on such device
>> even if it succeeded, because dfu->write_medium returns -EOPNOTSUPP.
>>
>> Signed-off-by: Masami Hiramatsu <masami.hiramatsu at linaro.org>
>> Cc: Lukasz Majewski <lukma at denx.de>
>> ---
>> drivers/dfu/dfu_mtd.c | 2 ++
>> 1 file changed, 2 insertions(+)
>
>
> A patch has already been submitted for this[1].
Ah, thanks for pointing! If that is merged, I can drop this patch.
Thank you,
>
> Patrick, can you please apply this in your tree and send it as part of the next PR. Thanks.
>
> -sughosh
>
> [1] - https://lists.denx.de/pipermail/u-boot/2021-March/443896.html
>
>
>>
>> diff --git a/drivers/dfu/dfu_mtd.c b/drivers/dfu/dfu_mtd.c
>> index ca67585a7e..e58302c32d 100644
>> --- a/drivers/dfu/dfu_mtd.c
>> +++ b/drivers/dfu/dfu_mtd.c
>> @@ -152,6 +152,8 @@ static int mtd_block_op(enum dfu_op op, struct dfu_entity *dfu,
>> ret = mtd_lock(mtd, lock_ofs, lock_len);
>> if (ret && ret != -EOPNOTSUPP)
>> printf("MTD device lock failed\n");
>> + if (ret == -EOPNOTSUPP)
>> + ret = 0;
>> }
>> return ret;
>> }
>>
--
Masami Hiramatsu
More information about the U-Boot
mailing list