Bugfix: Change list of CAN baud rates to appropriate type

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

306 stars 91 forks source link

Bugfix: Change list of CAN baud rates to appropriate type #11

Closed ThorbjoernSchulz closed 3 years ago

ThorbjoernSchulz commented 3 years ago

The CAN baud rates are of type INT8U as specified by the respective library. Choosing INT32U leads to bugs apparently, when changing the baud rate with the S command.

latonita commented 3 years ago

@ThorbjoernSchulz thank you for this pull request. It makes proper sense. lw232CanSpeedSelection is int8