Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-15 Thread Ilias Apalodimas
[...] > > > > + > > > > > + return 0; > > > > > > > > Yep, that's identical to what I had above without the allocation, > > > > which indeed isn't needed. > > > > Both of the versions are correct and I ask internally mbedTLS devs if > > > > they have a preference. > > > > > > > > In any cas

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-15 Thread Simon Glass
Hi Ilias, On Wed, 13 Nov 2024 at 22:10, Ilias Apalodimas wrote: > > On Thu, 14 Nov 2024 at 05:53, Simon Glass wrote: > > > > Hi Ilias, > > > > On Wed, 13 Nov 2024 at 09:11, Ilias Apalodimas > > wrote: > > > > > > On Wed, 13 Nov 2024 at 18:04, Simon Glass wrote: > > > > > > > > Hi Ilias, > > >

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Ilias Apalodimas
On Thu, 14 Nov 2024 at 05:53, Simon Glass wrote: > > Hi Ilias, > > On Wed, 13 Nov 2024 at 09:11, Ilias Apalodimas > wrote: > > > > On Wed, 13 Nov 2024 at 18:04, Simon Glass wrote: > > > > > > Hi Ilias, > > > > > > On Wed, 13 Nov 2024 at 08:11, Ilias Apalodimas > > > wrote: > > > > > > > > On W

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Simon Glass
Hi Ilias, On Wed, 13 Nov 2024 at 09:11, Ilias Apalodimas wrote: > > On Wed, 13 Nov 2024 at 18:04, Simon Glass wrote: > > > > Hi Ilias, > > > > On Wed, 13 Nov 2024 at 08:11, Ilias Apalodimas > > wrote: > > > > > > On Wed, 13 Nov 2024 at 17:09, Simon Glass wrote: > > > > > > > > Hi Ilias, > > >

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Ilias Apalodimas
On Wed, 13 Nov 2024 at 18:04, Simon Glass wrote: > > Hi Ilias, > > On Wed, 13 Nov 2024 at 08:11, Ilias Apalodimas > wrote: > > > > On Wed, 13 Nov 2024 at 17:09, Simon Glass wrote: > > > > > > Hi Ilias, > > > > > > On Wed, 13 Nov 2024 at 07:46, Ilias Apalodimas > > > wrote: > > > > > > > > On W

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Simon Glass
Hi Ilias, On Wed, 13 Nov 2024 at 08:11, Ilias Apalodimas wrote: > > On Wed, 13 Nov 2024 at 17:09, Simon Glass wrote: > > > > Hi Ilias, > > > > On Wed, 13 Nov 2024 at 07:46, Ilias Apalodimas > > wrote: > > > > > > On Wed, 13 Nov 2024 at 15:39, Simon Glass wrote: > > > > > > > > HI Ilias, > > >

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Ilias Apalodimas
On Wed, 13 Nov 2024 at 17:09, Simon Glass wrote: > > Hi Ilias, > > On Wed, 13 Nov 2024 at 07:46, Ilias Apalodimas > wrote: > > > > On Wed, 13 Nov 2024 at 15:39, Simon Glass wrote: > > > > > > HI Ilias, > > > > > > On Mon, 11 Nov 2024 at 07:20, Ilias Apalodimas > > > wrote: > > > > > > > > Thank

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Simon Glass
Hi Ilias, On Wed, 13 Nov 2024 at 07:46, Ilias Apalodimas wrote: > > On Wed, 13 Nov 2024 at 15:39, Simon Glass wrote: > > > > HI Ilias, > > > > On Mon, 11 Nov 2024 at 07:20, Ilias Apalodimas > > wrote: > > > > > > Thanks Simon, > > > > > > On Mon, 11 Nov 2024 at 15:03, Simon Glass wrote: > > >

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Ilias Apalodimas
On Wed, 13 Nov 2024 at 15:39, Simon Glass wrote: > > HI Ilias, > > On Mon, 11 Nov 2024 at 07:20, Ilias Apalodimas > wrote: > > > > Thanks Simon, > > > > On Mon, 11 Nov 2024 at 15:03, Simon Glass wrote: > > > > > > On Sun, 10 Nov 2024 at 01:32, Ilias Apalodimas > > > wrote: > > > > > > > > With

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-13 Thread Simon Glass
HI Ilias, On Mon, 11 Nov 2024 at 07:20, Ilias Apalodimas wrote: > > Thanks Simon, > > On Mon, 11 Nov 2024 at 15:03, Simon Glass wrote: > > > > On Sun, 10 Nov 2024 at 01:32, Ilias Apalodimas > > wrote: > > > > > > With the recent changes of lwip & mbedTLS we can now download from > > > https://

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-12 Thread Jerome Forissier
On 11/10/24 08:28, Ilias Apalodimas wrote: > With the recent changes of lwip & mbedTLS we can now download from > https:// urls instead of just http://. > Adjust our wget lwip version parsing to support both URLs. > While at it adjust the default TCP window for QEMU since https seems to > requir

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-11 Thread Ilias Apalodimas
Thanks Simon, On Mon, 11 Nov 2024 at 15:03, Simon Glass wrote: > > On Sun, 10 Nov 2024 at 01:32, Ilias Apalodimas > wrote: > > > > With the recent changes of lwip & mbedTLS we can now download from > > https:// urls instead of just http://. > > Adjust our wget lwip version parsing to support bot

Re: [PATCH v3 4/6] net: lwip: Enable https:// support for wget

2024-11-11 Thread Simon Glass
On Sun, 10 Nov 2024 at 01:32, Ilias Apalodimas wrote: > > With the recent changes of lwip & mbedTLS we can now download from > https:// urls instead of just http://. > Adjust our wget lwip version parsing to support both URLs. > While at it adjust the default TCP window for QEMU since https seems