Gedare Bloom started a new discussion on bsps/shared/grlib/can/grcanfd.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/390#note_119131

 >                              return GRCAN_RET_INVARG;
 >                      break;
 >              }
 > -            dest->head[1] = (dlc << 28) |
 > +            dest->head[1] = ((dlc & 0xf) << 28) |

I would suggest keeping `uint8_t dlc` and use `if (dlc > 15)` instead of `if 
(dlc < 0)`.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/390#note_119131
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
bugs@rtems.org
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to