ESP32RET icon indicating copy to clipboard operation
ESP32RET copied to clipboard

missing the DLC byte for the ATH1 case.

Open kc427hk opened this issue 3 years ago • 2 comments

In file ELM327_Emulator.cpp, function "processCANReply()", when the H1 is used, ie, bHeader is true, it seems that DLC byte is not set into the output buffer.

kc427hk avatar Apr 30 '22 07:04 kc427hk

I believe the default behavior is still to have no DLC value unless you issue an ATD1 command. But, the firmware as it sits on github doesn't respond to ATD1 so it would never show the DLC. I've added support to my local copy. I should be uploading a new version in a couple of days.

collin80 avatar May 02 '22 16:05 collin80

Just curious, is it ATD1 or ATH1? I tried using Car Scanner, and and the firmware won't respond correctly because Car Scanner app issues ATH1. However, it works find on Torque because it uses ATH0. I have tired to added the DLC if ATH1 locally, and finally, Car Scanner app works fine.

Also, in your another repository file, https://github.com/collin80/GEVCU/blob/master/ELM327Processor.cpp, line 86, you also added DLC if ATH1 case, ie bHeader is true.

kc427hk avatar May 03 '22 02:05 kc427hk