Skip to content

Add new bus bitrates: 666k, 83k#81

Open
esaulenka wants to merge 1 commit into
Nakakiyo092:masterfrom
esaulenka:bitrates_666_83
Open

Add new bus bitrates: 666k, 83k#81
esaulenka wants to merge 1 commit into
Nakakiyo092:masterfrom
esaulenka:bitrates_666_83

Conversation

@esaulenka

Copy link
Copy Markdown

Hello!

First of all, thanks fr supporting this project.
It looks like your branch is most 'alive'.

I'ld like to add a new bitrates that sometime may be found in the vehicle units: 83.333k (it can be found in Mercedes body units like radio and climatic) and 666.667k (it's a rare, it used in some very specific bootloaders).

I haven't checked the first one, but 666 works fine with the real hardware.

@Nakakiyo092

Nakakiyo092 commented Jan 10, 2026

Copy link
Copy Markdown
Owner

Hello,

It looks like your branch is most 'alive'.

Perhaps Elmue/CANable-2.5-firmware-Slcan-and-Candlelight is the most actively maintained slcan firmware for this hardware.
I’ve been focusing on another hardware platform recently, so this code hasn’t been updated for a while.

I'ld like to add a new bitrates that sometime may be found in the vehicle units: 83.333k (it can be found in Mercedes body units like radio and climatic) and 666.667k (it's a rare, it used in some very specific bootloaders).

I knew that some slcan projects support 83.333k, but I didn't know what it was for.
Thank you for sharing that information.

As for your pull request, I understand that it adds 83.333 k for nominal bit rate via S9[CR] and 666.667 k via SA[CR].
My firmware already supports flexible bit‑timing configuration through the s command,
and I prefer to use that mechanism instead.
For example, s780D0202 would configure the bit rate to 83.333k (see wiki for details).

In my opinion, using the flexible s command is preferable to pre-defining every possible bit rate,
especially those that are rarely used.

Regards.

@Nakakiyo092
Nakakiyo092 self-requested a review January 11, 2026 04:12
Comment thread inc/can.h
Comment thread inc/can.h
CAN_BITRATE_800K,
CAN_BITRATE_1000K,
CAN_BITRATE_83K,
CAN_BITRATE_666K,

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The SA option for 666 kbps is harder to accept.
I have never seen any slcan implementation that natively supports this bitrate.
In addition, there is a repository where SA is used for another purpose: "Set the nominal bit rate to 75k"
I’m not convinced it’s worth introducing, considering the confusion it will cause in the protocol.

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

Successfully merging this pull request may close these issues.

2 participants