[RESEND v4 04/14] mach-sc5xx: Switch from tftp to wget

[Simon Glass]

Hi Philip,

On Mon, 20 Apr 2026 at 21:33, Philip Molloy <philip at philipmolloy.com> wrote:
>
> Hi Simon,
>
> Thanks so much for this and the other reviews on the series. We really appreciate it!

You're welcome!

>
> On Monday, April 20th, 2026 at 4:51 AM, Simon Glass <sjg at chromium.org> wrote:
> > Hi Caleb,
> >
> > On 2026-04-16T15:19:35, Caleb Ethridge <jcethrid at gmail.com> wrote:
> > > mach-sc5xx: Switch from tftp to wget
> > >
> > > Switch the nfsboot and other relevant commands to use
> > > wget instead of tftp. This also includes the addition of
> > > the httpdstp variable for selecting the wget port.
> > >
> > > Signed-off-by: Caleb Ethridge <caleb.ethridge at analog.com>
> > >
> > > include/env/adi/adi_boot.env | 11 +++++------
> > >  1 file changed, 5 insertions(+), 6 deletions(-)
> >
> > > diff --git a/include/env/adi/adi_boot.env b/include/env/adi/adi_boot.env
> > > @@ -15,6 +15,7 @@
> > > +httpdstp=8000
> >
> > The default HTTP port is 80. The commit message does not explain why
> > 8000 is chosen. Please can you add a comment explaining the rationale,
> > or use port 80 if there is no specific need for a non-standard port?
>
> Explaining it in the commit message sounds good. It allows us to run an unprivileged web server.[1] 8000 is the default for `python3 -m http.server`.

Sounds good.

>
> [1]: https://analogdevicesinc.github.io/documentation/products/adsp/setup.html#boot-linux

Regards,
Simon