Sebastian Huber started a new discussion on 
cpukit/score/cpu/arm/include/rtems/score/armv7-pmsa.h: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/287#note_114264

 >    return UINT32_MAX;
 >  }
 >  
 > +ARMV7_PMSA_TEXT_SECTION static inline bool _ARMV7_PMSA_Check_region_empty(
 > +    uint32_t index
 > +)
 > +{
 > +  uint32_t region_count = _ARMV7_PMSA_Get_max_regions();
 > +  uint32_t rsr;
 > +
 > +  if (index >= region_count) {

The internal functions should use a minimum of error checks to avoid potential 
dead code. I would make this an assert.

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