[RFC] dfu: remove UPDATE_TFTP

[Michal Simek]

On 15. 07. 20 15:23, Lukasz Majewski wrote:
> Hi Heinrich,
>
> On 15.07.20 13:24, Lukasz Majewski wrote:
>>>> Hi Heinrich,
>>>>  
>>>>> Using UPDATE_TFTP the firmware can be updated from tFTP by
>>>>> writing to NOR flash. The same is possible by defining a dfu
>>>>> command in CONFIG_PREBOOT.
>>>>>
>>>>> The dfu command cannot only write to NOR but also to other
>>>>> devices. In README.dfutfp UPDATE_TFTP has been marked as
>>>>> deprecated.  
>>>>
>>>> Could you also write in the proper README the steps necessary to
>>>> have the same functionality as with UPDATE_TFTP with dfu and
>>>> preboot? I think that it would be good to have it written down in
>>>> ./doc.
>>>>  
> 
> doc/README.dfutftp already describes the necessary steps as:
> 
> * enable CONFIG_DFU_TFTP
> * enable CONFIG_PREBOOT
> * put a string like "dfu tftp 0 mmc 0" into the "preboot" env variable
> 
> Do you want me to explicitly write that "dfu tftp 0 mtd 0" replaces
> the former auto-update functionality?

I am curious if that command is really dfu tftp 0 mtd 0.

I have just tested it on zynqmp with copying images to ram and command
is just dfu tftp ram 0.
It means I think it would be worth to list options which are really used
by different back ends because it is not fully clear from help how to
exactly define it.

dfu tftp [<interface> <dev>] [<addr>]
  - device firmware upgrade via TFTP
    on device <dev>, attached to interface
    <interface>
    [<addr>] - address where FIT image has been stored

Thanks,
Michal

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: OpenPGP digital signature
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200715/ca1d8383/attachment.sig>