raiden00pl commented on PR #15073:
URL: https://github.com/apache/nuttx/pull/15073#issuecomment-2522742374

   > PRIMASK is a bad design in armv6-m, that's why arm introduce BASEPRI from 
armv7-m. It's wrong to provide this option for armv7-m/armv8-m arch.
   
   I think the timeline is different. armv7-m was introduced before armv6-m. 
The second one is a cut version of the first one and if ARM decided to get rid 
of BASEPRI they must have had a reason for this (but of course they could have 
made a design mistake here)
   
   > I don't see any bad side effect.
   
   I don't see any flaws in this solution either and it can simplify a lot of 
things. But people use NuttX in different ways. A change like this should be 
more widely communicated in the community, maybe someone sees drawbacks.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to