-
Notifications
You must be signed in to change notification settings - Fork 149
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
Ensure part is up for some 100 ms before resetting it #1939
Conversation
Yes, I would like to help.
Please let me know what you would like me to test.
Peter
|
@PeterHouseJr We typically want independent confimration that this specific pull request compiles and works as expected. The easiest way to do this is to head over to https://github.com/stefanrueger/avrdude/tree/linuxspi-start then click on the green code button to download a zip of that directory. Then best to build the specific avrdude according to your OS as in the wiki https://github.com/avrdudes/avrdude/wiki Then run the compiled new version and test with your setting. Thanks! |
Yes I will try out some simple tests soon. I believe this PR should be fine. |
No issues on my Raspberry Pi 400.
|
@stefanrueger - I have been out of the office for a couple of weeks and will be back next week. I plan to work my 328PB project. Would you still like me to test the PR? |
@PeterHouseJr We have had independent tests from @mcuee (thanks!) - so will merge soon. |
Fixes #1929
Would need some regression testing and confirmation it really helps ... @PeterHouseJr?