[PATCH v2 1/1] dfu: remove UPDATE_TFTP

[Lukasz Majewski]

Hi Marek,

> On 8/28/20 4:32 AM, Heinrich Schuchardt wrote:
> > On 7/21/20 8:02 PM, Heinrich Schuchardt wrote:  
> >> 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. It is not used by any board.
> >>
> >> Remove TFTP update via CONFIG_UPDATE_TFTP.
> >>
> >> Adjust the documentation.
> >>
> >> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> >> ---
> >> v2:
> >> 	rework the documentation  
> > 
> > On 8/28/20 12:17 AM, Marek Vasut wrote (in reply to a pull
> > request):  
> >> Also note that the UPDATE_TFTP is being actively used, why is it
> >> removed here and this late in rc3 ? I think these patches should
> >> really be postponed until after the release.  
> > 
> > Hello Marek,  
> 
> Hi,
> 
> > do you see a problem in principal with the removal of UPDATE_TFTP
> > which is redundant to what you can do with DFU or is it only the
> > timing issue?  
> 
> I don't see how it is redundant. The usecase I see is a fitImage which
> contains the update fragments is applied with a single command this
> way. I don't see a suitable replacement.

It is good that it now is clear that there is a use case for it - so
more discussion is needed.


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200828/f3728749/attachment.sig>