Gedare Bloom started a new discussion on cpukit/dhrl/dhrl.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1193#note_150604 > +rtems_status_code dhrl_init( > + const dhrl_config *config, > + uint32_t core_a, > + uint32_t core_b, > + rtems_task_priority priority > +) > +{ > + rtems_status_code sc; > + cpu_set_t cpuset; > + struct dhrl_control *ctx = &dhrl_instance; > + > + /* Validate essential pointers */ > + if ( > + config == NULL || config->signal_routine == NULL || > + config->work_routine == NULL > + ) { I'd break these and the later error checks into separate checks. -- View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1193#note_150604 You're receiving this email because of your account on gitlab.rtems.org.
_______________________________________________ bugs mailing list [email protected] http://lists.rtems.org/mailman/listinfo/bugs
