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.) (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? 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 (#44038): https://edk2.groups.io/g/devel/message/44038 Mute This Topic: https://groups.io/mt/32502331/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
