Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

USB Gecko: give up after 5s timeout #38

Merged
merged 1 commit into from
Nov 4, 2023

Conversation

webhdx
Copy link
Collaborator

@webhdx webhdx commented Nov 2, 2023

This PR fixes one of the most annoying things in iplboot. When you have USB Gecko connected it will wait indefinitely for PC to accept data. This was problematic is all you wanted is to boot Swiss from SD card and use USB Gecko with Swiss. Now imagine testing 20 USB Geckos having to deal with this behavior :P
This PR adds 5s window to send data to USB Gecko. I think this is enough time because most likely you already have usb-load waiting for connection when you start iplboot.

source/main.c Outdated Show resolved Hide resolved
source/main.c Outdated Show resolved Hide resolved
Copy link
Member

@9ary 9ary left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, please address the style issues and we're good to merge.

Also the commit message feels a bit weird, how about this: "USB Gecko: give up after 5s timeout"
I feel like that would be a little more descriptive.

@webhdx
Copy link
Collaborator Author

webhdx commented Nov 3, 2023

Oh yeah I definitely had a brain fart when I was writing that title lol Fixed in d8b043b

@webhdx webhdx changed the title USB Gecko: Wait 5s for PC to respond USB Gecko: give up after 5s timeout Nov 3, 2023
@9ary 9ary merged commit eb5f796 into redolution:master Nov 4, 2023
1 check passed
@webhdx webhdx deleted the usbgecko_timeout branch November 4, 2023 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants