Error in Waveshare 7.5v2 after 2023.9.0 upgrade

primeroz commented 11 months ago

The problem

after upgrading to 2023.9.0 , so i could get https://github.com/esphome/esphome/pull/5283 , i am seeing some errors in the logs of my display

[09:01:00][D][main:102]: Sensor data updated and activity in home detected: Refreshing display...
[09:01:00][I][waveshare_epaper:1628]: Power on the display and hat
[09:01:10][E][waveshare_epaper:1573]: Timeout while displaying image!
[09:01:21][E][waveshare_epaper:1573]: Timeout while displaying image!

The display referesh works so i just thought i would bring this up to see if is an issue . pinging @phoenixswiss as the auehtor of that PR

Which version of ESPHome has the issue?

2023.9.0

What type of installation are you using?

Home Assistant Add-on

Which version of Home Assistant has the issue?

2023.9.2

What platform are you using?

ESP32

Board

No response

Component causing the issue

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

phoenixswiss commented 11 months ago

Hi @primeroz

You need to invert your busy pin to get the correct busy status

https://github.com/esphome/esphome/pull/5283#:~:text=Fixed%20the%20wait_until_idle_()%20function%20to%20work%20properly%20with%20Waveshare%207.5v2%20screens%20(Warning%3A%20You%20also%20need%20to%20invert%20the%20busy%20pin%20in%20the%20yaml%20config!)

display:
  - platform: waveshare_epaper
    id: epaper_display
    reset_duration: 2ms
    setup_priority: 1000
    cs_pin: 5
    dc_pin: 0
    busy_pin: 
      number: 15
      inverted: True
    reset_pin: 2
    model: 7.50inv2
    update_interval: never

Unfortunately, the documentation update that mentions this hasn't been approved yet. If you don't invert the busy pin, the fix for https://github.com/esphome/esphome/pull/5283 won't work!

primeroz commented 11 months ago

Did it and working fine :pray: thanks for your work that will make my brand new panel last for ages ! :D

phoenixswiss commented 11 months ago

Great to hear. By the way, the documentation change was approved just a minute ago:

https://esphome.io/components/display/waveshare_epaper.html?highlight=waveshare#:~:text=Warning,in%20the%20config.

esphome / issues