I've already applied this, but could you follow up with a patch to remove all the debug print statements that we've added while working on this? The UV5R driver is really chatty and I think we can clean up most of that stuff now that we know it's working. Just if/when you have time, I'd appreciate it.
Dang it. I meant to do that. Sorry. It shall be done.
I'm also a little concerned at the size of the get_settings() function now that you've gone and added every dang feature the radio supports :)
I think we should make a point of going back and splitting these out into separate functions, just for cleanliness. It won't really affect how it works, just how it looks to the developer. I think maybe one helper function per page would be fine.
Are you interested in helping with this? I don't want to derail your progress towards getting anything else closed out, but definitely don't want it to get away from us :)
Sure. But I'm not sure exactly what you are looking for. But I'm pretty sure that once you get me started, I can help with this.
Thanks!
You're welcome
Jim