You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/03/28 10:07:52 UTC

[GitHub] [incubator-nuttx] GUIDINGLI commented on pull request #5849: pm: remove unnecessary depends on OSINIT_OS_READY

GUIDINGLI commented on pull request #5849:
URL: https://github.com/apache/incubator-nuttx/pull/5849#issuecomment-1080455573


   @pkarashchenko 
   
   The pm_lock just a global value:
   #define pm_lock() nxsem_wait(&g_pmglobals.regsem);
   
   I hasn't meet error when do pm_register, but the code can be more easier, so comes this patch.
   And I guess someone who add the the judgement with OSINIT_OSREADY, maybe caused by:
   
   722   /* The OS is fully initialized and we are beginning multi-tasking */
   723 
   724   g_nx_initstate = OSINIT_OSREADY;
   
   But, I don't think we need this judgement, so remove it.
   


-- 
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-unsubscribe@nuttx.apache.org

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