Gedare Bloom started a new discussion on cpukit/dhrl/dhrl.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1193#note_150853

 > +  best_bit = 6;
 > +  lowest_latency = UINT64_MAX;
 > +  max_safe_bit = 0;
 > +  safe_size = ctx->active_config.memory_region_size / 2;
 > +
 > +  if ( safe_size > 1 ) {
 > +    max_safe_bit = (uint8_t) ( ( sizeof( unsigned long long ) * 8 - 1 ) -
 > +                               __builtin_clzll(
 > +                                 (unsigned long long) safe_size
 > +                               ) );
 > +  }
 > +
 > +  rtems_dhrl_debug_printk(
 > +    "\nDetermining DDR Interleave Bit (Testing bits 6 to %u)...\n",
 > +    max_safe_bit
 > +  );

This function is the only place you use these debug print statements. Unless 
you think this information is going to be critically useful to application 
developers, you might consider removing these print functions to reduce 
complexity. Just a thought.

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