brambo123
brambo123
> There must be some sort of buffer, because writes don't delay for either ESP32 or ESP8266 when using the hardware uart. It does delay if I `flush` it. And...
> I tested it on both ESP8266 and ESP32 and had 0 delays on both. Are you sure it wasn't something else with your component? Please test with the modbus...
> > [W][component:214]: Component sdm_meter.sensor took a long time for an operation (0.15 s). > > Which version of ESPHome are you using? I would expect all times in msec...
I encountered the first problems on an ESP32-S3. Maybe it works a little differently there. I just don't know if I still have such a module to test, I'll have...
For information, the current fixed values for rx_timeout and rx_full_threshold: [Arduino rx_timeout: 2](https://github.com/espressif/arduino-esp32/blob/2.0.5/cores/esp32/HardwareSerial.cpp#L143) [Arduino rx_full_threshold: 112](https://github.com/espressif/arduino-esp32/blob/2.0.5/cores/esp32/HardwareSerial.h#L109) [ESP-IDF rx_timeout: 10](https://github.com/espressif/esp-idf/blob/v5.1.5/components/driver/uart/uart.c#L56) [ESP-IDF rx_full_threshold: 120](https://github.com/espressif/esp-idf/blob/v5.1.5/components/driver/uart/uart.c#L55)
> Just wondering; Are this changes you made also useful for the esp32s3 etc. with UART over USB? I haven't looked into this jet but in the near future i...
I came across a small thing. The get_rx_full_threshold() (and other functions) should of course also indicate a good value on other platforms. I will adjust it a bit. And of...
I was thinking about this and then I thought of something. Both rx_full_threshold and rx_timeout are purely for internal timing. Shouldn't we just give components the freedom to adjust (and...
> > So, as far as I can tell, the open points are: > > > > * default value for `rx_full_threshold` (I would propose 1 for baudrate > I...
> * naming for `rx_timeout` (maybe `rx_timeout_bytes`? Either way, a short description in the docs should clear up confusion that comes up) No one has a good alternative. Adding _bytes...