Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KW1281 support in sources #232

Open
mvglasow opened this issue Mar 14, 2023 · 4 comments
Open

KW1281 support in sources #232

mvglasow opened this issue Mar 14, 2023 · 4 comments

Comments

@mvglasow
Copy link

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
Copy link
Owner

fr3ts0n commented Mar 15, 2023

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
Copy link
Author

mvglasow commented Mar 16, 2023

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:

  • Add JFreeChart as a dependency
  • Install RXTX, add it as a dependency and re-add missing imports in SerialConfigPanel (which got removed in 59795d6).
  • Also in SerialConfigPanel, import java.util.logging.Logger, fix the calls to getLogger (the JDK version takes a class name as a string, but not a class) and change warn to warning. (This code was written for Log4J, 59795d6 removed the import but left the method calls in place).
  • @SuppressWarnings("unchecked") in PvTreeNode#findChild(Object) no longer works as expected, so I had to change a few PvTreeNode declarations to TreeNode and add typecasts where PvTreeNode is needed.

This has eliminated all of the build errors.

@mvglasow
Copy link
Author

...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
Copy link
Owner

fr3ts0n commented Mar 17, 2023

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 ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants