[edk2-devel] [edk2-platforms: PATCH 0/3] Platform/RPi3: Improve timeout handling in DwUsbHostDxe
Pete Batard
pete at akeo.ie
Fri Jul 19 15:40:48 UTC 2019
Hi Leif,
On 2019.07.19 12:53, Leif Lindholm wrote:
> Hi Pete,
>
> On Wed, Jul 17, 2019 at 12:46:42PM +0100, Pete Batard wrote:
>> Networking applications (e.g. iPXE) might experience failures when submitting
>> a bulk IN for the NIC's RX endpoint, because the bulk IN (correctly) times out
>> when no received packet is waiting, but DwUsbHostDxe.c treats this as a fatal
>> error.
>>
>> With these patches, iPXE is able to successfully download a 128MB test file
>> via HTTP.
>
> The patches look good, and I don't mind you upstreaming Michael's
> code, *but* I don't want patches submitted with anyone other than the
> contributor's Signed-off-by:. (It's the equivalent of saying "Yeah,
> Michael says he's OK with https://developercertificate.org/, and of no
> actual use.)
I see. It does make sense, now that you explain it.
> (If patches are modified after contribution, but before being pushed,
> then then additional contributions can be reflected with additional
> Signed-off-bys. Make sense?)
>
> The From: tag ensures he still retains authorship.
> Are you OK with me dropping Michael's Signed-off-by before pushing?
That's fine with me. I reviewed these patches and can therefore vouch
for them.
Regards,
/Pete
PS: Thanks for pushing the previous series.
>
> Best Regards,
>
> Leif
>
>> Michael Brown (3):
>> Platform/RPi3: Use Wait4Bit return value consistently
>> Platform/RPi3: Gracefully disable USB channel after a timeout
>> Platform/RPi3: Reduce debug noise when using a USB network device
>>
>> Platform/RaspberryPi/RPi3/Drivers/DwUsbHostDxe/DwUsbHostDxe.c | 44 +++++++++++---------
>> 1 file changed, 25 insertions(+), 19 deletions(-)
>>
>> --
>> 2.21.0.windows.1
>>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#44049): https://edk2.groups.io/g/devel/message/44049
Mute This Topic: https://groups.io/mt/32502331/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list