On Sun, Jan 21, 2024 at 11:13:36AM +1300, Daniel Beer wrote: > On Sat, Jan 20, 2024 at 08:41:28PM +0200, Michael Zaidman wrote: > > Hi Daniel and Christina, > > > > My concern with the implemented workaround is that sending a dummy report > > every 4.8 seconds eliminates the chip's power save mode functionality. > > > > I did more testing and figured out that the baud rates 4800 and below work > > fine with power saving mode and do not require this workaround. > > > > So, I modified the code, making it dependent on the baud rate in this commit: > > https://github.com/MichaelZaidman/hid-ft260/commit/b5a2ad68c7cebbaaba0aa1675ae376f2895e19aa > > > > > > Another improvement is not to activate the wakeup workaround if power > > saving mode is not enabled in EEPROM. > > https://github.com/MichaelZaidman/hid-ft260/commit/0a41f3f3a4c664edc3bb90718807f2e62fe6d375 > > > > For UART testing, I sent files via picocom opened on ttyUSB0 (ch340 USB > > dongle device) connected to ft260 UART, receiving and displaying the > > trafic in screen manager utility. > > > > --Michael > > Hi Michael, > > That sounds like a good idea to me. Are you planning to submit your > improved version from here? > > I don't have any FT260 hardware with me any more, so there's > unfortunately not much I can do now to help, aside from answering > questions. > > Cheers, > Daniel > Yeah, I mean to do more testing, and then I will submit a new patch set based on your work. --Michael
