On Wed, Apr 16, 2025 at 1:25 AM Carlos Sanchez
<carlossanc...@geotab.com.invalid> wrote:

> Related PR on the apps side, to make slcan work after the "bitrate no
> longer brings interface up" change:
> https://github.com/apache/nuttx-apps/pull/3059
>
> While testing this, I think I have discovered a small mistake on my
> previous, Nuttx-side PR, which slipped by me and by revision: "ret"
> might be used uninitialized now because I removed the assignment on
> stm32_fdcan_sock:1976
>
>
> https://github.com/apache/nuttx/pull/16199/files#diff-8efec4e19b7401adcc3961c078b98ab945ebcada8635105fb591d16cbd6e5210L1976
>
> Should I make a mini PR for this?
>
> I was going to propose another global CAN change, removing the 1000
> factor from bitrate thus making bitrate calls use units in Hz instead
> of kHz, so we can discuss that and (if agreed) I make the change on
> the same PR to avoid creating so a tiny one.
>

What's the unit used on Linux? we need to align the interface with Linux.

>
> Thoughts?
>
> Carlos
>

Reply via email to