ncmreynolds/ld2410

Adding support for ESP32 S3?

Humancell opened this issue · 3 comments

Humancell commented

Hello,

Thank you very much for this library! It is great and is working well after a little troubleshooting.

I'm using a SmartBee Bee S3 (https://www.tindie.com/products/smartbee/bee-s3-esp32-s3/) and found a few issues:

  1. Per some suggestions here, I had to add a 5 second delay in setup, right after the "MONITOR_SERIAL.begin(115200); //Feedback over Serial Monitor" or things didn't get output to the Serial Monitor. This appears to be S3 related. I found this suggestion here: https://flashgamer.com/blog/comments/solving-platformio-issues-with-the-adafruit-feather-s3
  MONITOR_SERIAL.begin(115200); //Feedback over Serial Monitor
  delay(5000);
  1. I added a new section to the pin configuration as shown below. These were the exposed UART pins on my Bee S3:
    #elif CONFIG_IDF_TARGET_ESP32S3
      #define MONITOR_SERIAL Serial
      #define RADAR_SERIAL Serial1
      #define RADAR_RX_PIN 44
      #define RADAR_TX_PIN 43

After these changes it worked great!

ncmreynolds commented

Thanks, I don't have any S3 boards so haven't tried it yet. Serial port numbering on these ESP32s with multiple hardware Serial and also the USB Serial can get a bit wordy to cover all the cases.

Humancell commented

Hello ... yes, I didn't expect that you had total coverage. It is a great library, and easy to get things working. I wasn't super familiar with the S3 either, but it only took a little bit to get it working.

BTW, I created a little more complete sensor solution using the library, adding MQTT publishing and OTA. https://github.com/Wovyn/esp32-ld2410-radar-sensor-mqtt-ota

Non-optimized code at this point, but your library helped this become a 3-4 hour project instead of more. :-)