Bluetooth ELM327 Support

openDsh / dash

Join us on Slack! https://join.slack.com/t/opendsh/shared_invite/zt-la398uly-a6eMH5ttEQhbtE6asVKx4Q

GNU General Public License v3.0

238 stars 69 forks source link

Closed icecube45 closed 2 years ago

icecube45 commented 2 years ago

Configured through vehicle page settings - can select any paired BT device.

Adapter must be connected to the pi for this to work - no autoconnect functionality rn.

r1de commented 2 years ago

Confirmed working with a OBDLink MX+ Bluetooth ODB2 dongle. Nice work!! +100

rsjudka commented 2 years ago

i know its stilll WIP but just some comments so far with what i see

might be cleaner to have vehicle_can_bus Config as an enum (which i think QConfig would just cast to/from an int)
use a QMap for the device name/address instead of maintaining 2 separate lists
your g37 stuff was included here
some small changes that'll be easier for me to point out in the code lol

overall looks good!

icecube45 commented 2 years ago

@rsjudka I'm away from my test setup for a few days, but if the changes I just pushed compile / don't segfault then I think it's good to go