toolchain icon indicating copy to clipboard operation
toolchain copied to clipboard

Experimental support for CH340, FT232RL & CP2101 devices

Open tr1p1ea opened this issue 2 years ago • 10 comments

Experimental support for CH340, FT232RL & CP2102 devices.

Tested with ESP8266 D1 mini clone, ESP-WROOM-32 board and FTDI232 boards.

Currently baud rates are table driven so baud rates are limited to: 9600, 19200, 38400, 57600 & 115200 but can be easily expanded.

tr1p1ea avatar Nov 19 '23 09:11 tr1p1ea

Looks good to me!

mateoconlechuga avatar Nov 29 '23 14:11 mateoconlechuga

Does the C header need updating?

mateoconlechuga avatar Nov 29 '23 14:11 mateoconlechuga

I fixed an issue with CH340 clones and I'm adding CH341 today.

I'll also update the header file.

tr1p1ea avatar Nov 30 '23 00:11 tr1p1ea

Is it good to merge?

mateoconlechuga avatar Dec 04 '23 19:12 mateoconlechuga

rebased and launched CI just to check

adriweb avatar Dec 04 '23 19:12 adriweb

@tr1p1ea what is the status of this PR? Also @commandblockguy, which struct fields can we deprecate to make everything reserved?

mateoconlechuga avatar Jan 04 '24 12:01 mateoconlechuga

We can probably deprecate everything except for the USB device handle.

commandblockguy avatar Jan 04 '24 17:01 commandblockguy

Great :)

mateoconlechuga avatar Jan 04 '24 18:01 mateoconlechuga

After testing I've come to the conclusion that there is some further work to do to improve compatibility with some devices.

Should I recall this PR until that's complete?

Also did you want the struct optimisation to be part of this too?

tr1p1ea avatar Jan 05 '24 22:01 tr1p1ea

@tr1p1ea yes to the struct optimization and any fun new status updates for us :)

It's fine to just keep it in this PR

mateoconlechuga avatar Jan 18 '24 03:01 mateoconlechuga