danak6jq / ESP32-WSPR

Complete stand-alone WSPR2 transmitter using ESP32 + Si5351 modules. Uses SNTP for transmit time-sync
GNU General Public License v3.0
26 stars 10 forks source link

i2c clock allocator flag #2

Closed richardhenryhopper closed 1 year ago

richardhenryhopper commented 2 years ago

Error building in ESP-IDF 4.4, as the i2c clock allocator flag needs to be set in i2c_master_init(). conf.clk_flags = 0;

danak6jq commented 1 year ago

Is there a PR for this? Would love to review and merge

richardhenryhopper commented 1 year ago

I don't think so, Richard

On Mon, 17 Jul 2023, 17:39 Dana Myers, @.***> wrote:

Is there a PR for this? Would love to review and merge

— Reply to this email directly, view it on GitHub https://github.com/danak6jq/ESP32-WSPR/issues/2#issuecomment-1638494950, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMBFQWEN6M3PK3C6XLA357LXQVTD7ANCNFSM6AAAAAAQP6HHEQ . You are receiving this because you authored the thread.Message ID: @.***>

danak6jq commented 1 year ago

I apologize for the year-long delay - just realized you had a PR! Merged. Thank you!

richardhenryhopper commented 1 year ago

Welcome Dana :)

I just tested it with my ESP32 and spotted this missing flag issue. I am now working on some more transmitter hardware but intend to use the firmware in the future.

Best wishes, Richard Hopper Cambridge, UK

On Wed, Sep 27, 2023 at 8:59 PM Dana Myers @.***> wrote:

I apologize for the year-long delay - just realized you had a PR! Merged. Thank you!

— Reply to this email directly, view it on GitHub https://github.com/danak6jq/ESP32-WSPR/issues/2#issuecomment-1737996508, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMBFQWEZZNLZOUKUYY3XPCLX4SASDANCNFSM6AAAAAAQP6HHEQ . You are receiving this because you authored the thread.Message ID: @.***>