Extending Daikin support for wired remote: steer requested

[caliston]

I've done some reverse engineering of the wire protocol for the Daikin FWT fan coil and its Merca / Netware3 hardwired wall controller. The protocol looks very similar to the other pulse distance modulated IR protocols like Daikin64. I'd guess they use similar code in the microcontroller to drive both IR and wired controls - the main difference being it's 5V TTL on the wire and not modulated. However it's not the same - timing is slightly different, the meaning of bits in the 64 bit packet are different, some functions operate differently (eg the clock is local to the controller and never gets sent to the fan coil). I tried talking to it with the existing Daikin64 code and I was completely ignored. Also, unlike the IR protocols, there's a repeated broadcast message from the fan coil (which mostly just sends the room temperature I think - can possibly just be ignored)

If I want to add support for this, I was wondering if there was any steer on what I should implement:

• Is this, as a wired protocol, out of scope for IRremoteESP8266? Even though 95% of the code will be shared with the IR implementation?
• If I were to add it, is it OK to add another protocol (eg 'DaikinWired64') to ir_Daikin.cpp? That file is getting quite large with duplicated-but-not-quite code
• Is there a better way to clean up that file? For example breaking out each DaikinXXX into a separate file might be a simple thing to do. Or something else to avoid duplication maybe?
• Any other suggestions for what (not) to do?