libretiny icon indicating copy to clipboard operation
libretiny copied to clipboard

OTA implementation & fixes for RTL8720CF

Open prokoma opened this issue 1 year ago • 8 comments

I've cleaned the history and removed the code to pull ltchiptool from git. You need to make a release of ltchiptool first before merging this PR and also update the minimum required version in builder/main.py.

prokoma avatar Dec 03 '24 21:12 prokoma

Hi @fcol95, OTA for rtl8720cf is not implemented yet in the official release. My solution works, but the code is not ideal and there are some open questions.

I've described steps how to install esphome with working OTA in https://github.com/libretiny-eu/libretiny/issues/44#issuecomment-2514974466 . You can skip step 0, because ltchiptool >= 4.11.5 already contains the mentioned fix.

prokoma avatar Mar 29 '25 20:03 prokoma

@prokoma thanks for your quick reply, just had deleted my comment when stumbling on your open PR in ltchiptool repo and understanding a bit better the issue! thanks for the link to your custom solution! I'll give it a go once I finish working on the component driver I need to do for the chip that interfaces with this rtl8720c in my device!

fcol95 avatar Mar 29 '25 21:03 fcol95

Just tested the OTA through esphome and your "source: https://github.com/prokoma/libretiny#55aacc8" framework, works quite well, thanks for the effort, would be glad to see that merged in the production release soon!

fcol95 avatar Mar 31 '25 02:03 fcol95

Hi @prokoma,

Thanks for a great contribution! I wonder if there are any plans to merge this PR soon (given that libretiny-eu/libretiny#307 merged already)?

MottiG avatar Aug 10 '25 23:08 MottiG

Just want to confirm this works perfectly for me in esphome once the latest libretiny commits are pulled in. Would love to see this in master!

DominicChm avatar Aug 13 '25 06:08 DominicChm

I have used this with a tuya smart weather station that uses WBR3 module and can confirm it works well. Thanks!!

vitoralb avatar Sep 09 '25 20:09 vitoralb

Hi @prokoma , could you please update your fork to include this commit from the original repo? image

It cannot be compiled with the newest esphome without this change.

szupi-ipuzs avatar Nov 30 '25 10:11 szupi-ipuzs

Rebased on latest LibreTiny dev version.

kuba2k2 avatar Dec 08 '25 18:12 kuba2k2