joshbenner / esphome-daikin-s21

ESPHome Component for Daikin Mini-Splits using the S21 serial port.
GNU General Public License v3.0
64 stars 21 forks source link

Improve compatibility and silent mode #11

Open danijelt opened 1 year ago

danijelt commented 1 year ago

First commit improves compatibility (discussed in issue #7) with units that don't respond to RH, RI, Ra and RL queries. This should be a minimally intrusive way to do it, by moving them to a "failable" queries list. This way, we now always expect response from F9 (all of them should according to what I found online), and for units that don't return their external temperature there, we'll get it from Ra query anyway.

Second commit adds silent fan mode. Nothing special, just another value in enum. Tested and working.

dhbfischer commented 1 year ago

Tested and working on my FTXC-B. Didn't got the main branch working due to unknown queries: RH, RI, Ra, RL. F9 does work and gives me indeed an indoor and outdoor temperature.

I've tested also some "experimental" queries and "F2", "F3", "F4", "M" seem to work in addition to "F1", "F5", "Rd". I can possibly check in the future. Currently on my FTXC-B I don't have the coil temp and fan speed (rpm), possibly this isn't an option on mine either.

[14:02:28][D][daikin_s21:367]: ** UNKNOWN QUERIES ** [14:02:28][D][daikin_s21:294]: S21: G2 -> 4:\x80\x80 (4) [14:02:28][D][daikin_s21:341]: Unknown response G2 -> "4:\x80\x80" [14:02:28][D][daikin_s21:294]: S21: G3 -> 0000 (4) [14:02:28][D][daikin_s21:341]: Unknown response G3 -> "0000" [14:02:28][D][daikin_s21:294]: S21: G4 -> 0 \x800 (4) [14:02:28][D][daikin_s21:341]: Unknown response G4 -> "0 \x800" [14:02:28][D][daikin_s21:261]: NAK from S21 for RN query [14:02:28][D][daikin_s21:261]: NAK from S21 for RX query [14:02:28][D][daikin_s21:261]: NAK from S21 for RD query [14:02:28][D][daikin_s21:294]: S21: M -> 1344 (4) [14:02:28][D][daikin_s21:341]: Unknown response M -> "1344" [14:02:29][D][daikin_s21:294]: S21: G\x00\x00\x00 -> \x00\x00 (2) [14:02:29][D][daikin_s21:341]: Unknown response G\x00\x00\x00 -> "\x00\x00"

dhbfischer commented 1 year ago

@joshbenner what about this pull request? Can it be merged? The change request seems to be fixed.