fr3ts0n / AndrOBD

Android OBD diagnostics with any ELM327 adapter
http://fr3ts0n.github.io/AndrOBD/
GNU General Public License v3.0
1.45k stars 310 forks source link

KW1281 support in sources #232

Open mvglasow opened 1 year ago

mvglasow commented 1 year ago

According to the FAQ, non-OBD protocols are not supported for valid reasons.

However, there is this file in the source code: https://github.com/fr3ts0n/AndrOBD/blob/master/library/src/main/java/com/fr3ts0n/ecu/prot/vag/Kw1281Prot.java

This looks like an implementation of the Volkswagen Group’s KW1281 protocol, although after a quick glimpse it does not appear to be used anywhere in the app.

How mature is this code? Does it work with an ELM327 adapter (if not, what type of adapter was it written for)?

(Not a feature request, but if this code works, I would like to build on it for my own project.)

fr3ts0n commented 1 year ago

Yes, this file is a implementation of VAG KW1281. It is not part of the app, but is contained in the generic java library There is a JRE GUI application in the repo (com/fr3ts0n/ecu/gui/application/VagTestFrame.java) which actually uses it. It can be used with a USB KKL adapter. I used it to diagnose my old Skoda Fabia.

mvglasow commented 1 year ago

Such a gem hidden deep down in the code :-) There didn’t seem to be a build recipe, so I pieced together a POM, based on https://github.com/fr3ts0n/AndrOBD/blob/master/library/build.gradle. A few fixes were needed:

This has eliminated all of the build errors.

mvglasow commented 1 year ago

...however, I am getting an UnsatisfiedLinkError because SerialExt.java tries to load a native library named SerialExt. Is that part of your code? Is it anywhere to be found in this repository?

fr3ts0n commented 1 year ago

Oh yes, there was a lib to it to allow communication on odd baud rates like 10400 ... I'll have a look to find it again ...