search

markqvist / rnodeconfigutil

MIT License
25 stars 11 forks source link

Sleep a little longer when probing #7

Closed snh closed 2 years ago

snh commented 2 years ago

While setting up RNode on some Arduino Mega 2560s today, I was only managing about a 1 out of 10 success rate with having rnodeconf detect them. Increasing the sleep here resolved the issue.

snh commented 2 years ago

Ran in to this again, but this time my fix didn't help. Closing this PR, as I'm not convinced it is actually the solution now.

markqvist commented 2 years ago

Could there be something else sending serial output that messes up the detection process? Did you add any debug output to the firmware you are using?

markqvist commented 2 years ago

Did you ever figure this out? I am going to release support for 2560-based homebrew devices in the official firmware and rnodeconf, and I'd love to have this ironed out. I currently don't have 2560-based hardware here to test on myself (on its way, but it is taking its time due to shipping issues), so I'm trying to prepare as much as I can without the actual hardware ;)

snh commented 2 years ago

Hey @markqvist, apologies for the silence, I haven't had a chance to get back to this yet but will keep you updated when I do.

markqvist commented 2 years ago

Thanks, no worries. Let me know if you figure something out!

markqvist commented 2 years ago

Hi @snh I finally got my hands on some 2560-based boards, and I'm pretty sure I found the cause of your issue. I'll have an "official" 2560-compatible firmware available shortly :)

markqvist commented 2 years ago

I added official support for homebrew RNodes based on ATmega2560 to both the RNode Firmware and the rnodconf utility. You can now autoinstall and provision homebrew RNodes based on 2560-boards with:

rnodeconf --autoinstall

I have not tested it heavily myself, since my access to 2560-based boards is limited at the moment, but I would very much appreciate any feedback you have on this if you decide to try it out.

snh commented 2 years ago

@markqvist Thanks ❤️ I'll give it a try!

markqvist commented 2 years ago

Cool, let me know if there is any issues!

markqvist commented 2 years ago

OH, I totally forgot to mention this, I might have changed some pin assignments from how they were in your original PR. I actually can't remember now, but just wanted to give you a heads up on it. If your devices don't match my assignments, you will have to change them in the Config.h file again ;)