On Tue, Apr 15, 2025 at 1:25 PM 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? Hi, Good catch! Yes, please make a PR to fix that!! When it's ready, post about it in this thread so we can get it reviewed and merged quickly. Thanks, Nathan
