dentra / esphome-ewh

Control Electrolux Water Heater via uart
MIT License
25 stars 8 forks source link

Wrong UART config for Lilygo T-Dongle S3 #10

Closed 7leman7 closed 3 months ago

7leman7 commented 3 months ago

First of all - thanks for the hard work on the component!

Since Lilygo T-Dongle S3 is the recommended stick it would probably make sense to mention directly in the README that it uses GPIO19 and GPIO20 pins as USB_D+ and USB_D-, so UART confirguartion should be adjusted for it to work:

_uart: tx_pin: 19 rx_pin: 20 baudrate: 9600

My working yaml config for Lilygo T-Dongle S3 (no display) is as follows:

substitutions:
  name: "water-heater"
  # main prefix for all entities
  friendly_name: Water Heater
  # name of your node
  node_name: "water-heater"
  # use "esp12e" for iot-uni-dongle, "esp8285" for coolrf-heatstick, or your own if you know it
  board: "esp32-s3-devkitc-1"
  # time platform: "sntp" or "homeassistant"
  time_platform: "sntp"
  # SSID of your wifi
  wifi_ssid: !secret wifi_ssid
  # password of your wifi
  wifi_password: !secret wifi_password
  # password for fallback wifi hotspot
  wifi_ap_password: !secret wifi_fallback_password
  # UART configuration.
  tx_pin: TX
  rx_pin: RX

# please do not change packages order it is very important, just comment/uncomment
packages:
  remote:
    url: https://github.com/dentra/esphome-ewh
    ref: master # cant use substitutions here
    files:
      ## Enable additional WiFi signal sensor. Comment next line to disable.
      - packages/wifi_signal.yaml
      ## Enable additional Uptime sensor. Comment next line to disable.
      - packages/uptime.yaml
      ## Uncomment next line to enable additional restart button.
      - packages/restart.yaml
      ## Uncomment next line to enable additional diagnostic clock sensor
      # - packages/clock.yaml
      ## Uncomment next line to enable additional diagnostic timer sensor
      # - packages/timer.yaml
      ## Uncomment next line to enable standalone web ui
      - packages/ewh_web.yaml
      ## optional package, uncomment next line to enable experimental cloud support
      # - packages/cloud.yaml
      # Required package, do not comment
      - packages/base.yaml
      # Required package, replace with esp32.yaml if you use ESP32
      - packages/esp32_s3.yaml

esphome:
  name: ${name}
  name_add_mac_suffix: false
  friendly_name: ${friendly_name}

time:
  platform: sntp

# Enable Home Assistant API connection.
api:

# Uncomment to enable MQTT connecion.
# mqtt:
#   broker: !secret mqtt_borker
#   username: !secret mqtt_username
#   password: !secret mqtt_password

# WiFi hotspot configuration.
# Comment next 3 lines to enable manual WiFi conviguration via access point.
wifi:
  ssid: $wifi_ssid
  password: $wifi_password

logger:
  # Make sure logging is not using the serial port
  baud_rate: 0

uart:
  tx_pin: 19
  rx_pin: 20
  baud_rate: 9600

vport:
  - platform: ewh

climate:
  - platform: ewh
    id: whc
    name: None # use friendly_name
    # Optional, Bacteria Stop Technology switch
    bst:
      name: "Bacteria Stop Technology"
## Uncomment to enable and override cloud support
# rka_cloud:
#   id: cloud
#   mac: !secret cloud_mac
#   uid: !secret cloud_uid
7leman7 commented 3 months ago

Also, in the current version of the readme yaml configuration shown in the text seems outdated, an the correct one is only available via "download" link...

dentra commented 3 months ago

thanks

dentra commented 3 months ago

update in master