search

geomatsi / ftdi-embedded-hal-archive

Check https://github.com/ftdi-rs project for the up-to-date implementation of FTDI MPSSE embedded HAL.
MIT License
31 stars 6 forks source link

Update code, fix clippy, update dependencies #7

Closed barafael closed 3 years ago

barafael commented 3 years ago

Hi Geomatsi, This project is super cool. I updated the code to the best of my knowledge with help of the compiler and clippy. I also updated to GPIO v2 from embedded-hal. Posting the changes here for your consideration :)

Right now, I cannot run the tests because they require attaching an FTDI device and I think there might be something wrong with my configuration (FTDI cannot be detected).

Is the NRF24L01 also required for the tests?

geomatsi commented 3 years ago

Hello Rafael,

Thanks for your MR! Looks good, I will merge after CI completion.

Concerning your questions:

  1. No, nrf24 is not required at all. In fact, I plan to replace this example by something more useful, probably some SPI display.
  2. If you can not attach to you FTDI device, then probably ftdi_sio kernel module has been loaded to handle FTDI serial converter. Check using lsmod and then remove that module to enable direct access to FTDI device.

Regards, Sergey