jks-prv
commented
3 years ago Hi Kari. Thanks for doing the merge. The glitch with my last commit, where I confused the s7 and s4a drivers, has a story behind it. Earlier this week a guy complained that we had an s7 but it would only work with the s4a driver (s4a being ripperfox's new, untested (by me) driver). He made his Kiwi and s4a web interface available to me. After a lot of confusion I was convinced the switch was indeed speaking the new, simpler s4a protocol. Unfortunately there was nothing in the protocol or the web interface that indicated which type of switch it was. So I only had his word that it was an s7.
The only thing I could think of is that at some point the s7 firmware was changed to speak the simpler s4a protocol. But that seems unlikely to me as it only seems to allow "up/down" selection with no multiple, simultaneous antenna selection etc. I thought that was the differentiation between the s7 and s4a. But at least the exercise meant I was able to test ripperfox's code and verify that it works.
Cheers, John
Hi Kari. I think these changes are pretty stable now. So maybe consider adding them to your repo after your review and approval. Cheers, John