search

latonita / arduino-canbus-monitor

Can bus monitoring tool based on arduino and can bus shield. Implements CAN ASCII/SLCAN protocol compatible with Lawicel CAN232/CANUSB. Can be used for monitoring low speed CAN (interior can bus of most cars, less than 500kbps). For higher speeds serial port can become a bottleneck in case data density is high.
MIT License
308 stars 91 forks source link

Sending code seems to give several errors in data. Recieve working perfectly. #2

Closed pibrochman closed 8 years ago

pibrochman commented 8 years ago

Hi I built you code onto an arduino UNO, with CAN module with 16MHz clock. When I used the CANHACKER software to send messages they do not get sent correctly. I put a loop back onto another comm port and the comport serial messages seem to be correctly formatted as per the Lawicel spec. If a message is sent with DLC of 8, no data bytes are sent. If the DLC is less than 8, the bytes are sent but are wrong. If I send a can message with the following: ID006, DLC7, 12,34,56,78,50,40,30, what is recieved is ID006, DLC7, 22,44,66,88,00,00,00. Any ideas? Hardware setup: I checked this with a Peakcan USB, and IXXAT tincan, both showed same messages. By the way great project, and a real good way for people to get a low cost can analyser. Hope this info helps. Regards

latonita commented 8 years ago

Hi,

recently I found and logged quite similar error with 16mhz module. 8mhz was working fine. looking into this, but it might take time due to my availability

pibrochman commented 8 years ago

Hi Anton

I tried your recently updated code and its now working well.

I'm using a ch340 usb-ttl (£1)+ Aurduino pro mini (£1)+ Arduino MCP2515 CAN Bus Module TJA1050 Receiver SPI Module (£1.50). I had previously changed the crystal on the CAN module to 16MHz.

I have now tried the code using the original 8MHz crystal, having changed the code call to 8MHzand its all working well(I used an arduino nano instead £1.50)

So for £3.00-3.50 that makes a great little CAN analyser. What an excellent project, well done and thank you. The CANhacker software is also very good.

Kind Regards Matthew

On 04/05/2016 10:16, Anton Viktorov wrote:

Hi,

  • have you tried examples from MCP library to send data without my code?
  • which chips are on the board? mcp2515+mcp2551 or mcp2515+tjaXXXX ?

recently i logged quite similar error with 16mhz module. 8mhz was working fine. looking into this, but it might take time due to my availability

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/latonita/arduino-canbus-monitor/issues/2#issuecomment-216806963

latonita commented 8 years ago

Glad to hear everything is okay! :+1: Have a nice time using this tool :)