Kinsey Moore commented on a discussion on 
bsps/shared/dev/flash/flash_sim_flashdev.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/863#note_136674

 > +
 > +static void complete_operation_delay(
 > +  uint64_t start_time,
 > +  uint64_t target_delay
 > +)
 > +{
 > +  if ( target_delay == 0 ) {
 > +    return;
 > +  }
 > +  while ( rtems_clock_get_uptime_nanoseconds() - start_time < target_delay 
 > ) {
 > +    ;
 > +  }
 > +}
 > +
 > +static int flash_sim_get_flash_type(
 > +  rtems_flashdev *flash,

As far as I'm aware, all the other flashdev backends trust the passed 
parameters. I suppose debug assertions wouldn't hurt, but those might be better 
placed in the flashdev framework itself to avoid duplication.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/863#note_136674
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

Reply via email to