[GitHub] [incubator-nuttx] patacongo edited a comment on pull request #4233: board/ctrl: Add BOARDIOC_LOAD_APPIMAGE for loading an application image

[GitBox <gi...@apache.org>]

patacongo edited a comment on pull request #4233:
URL: https://github.com/apache/incubator-nuttx/pull/4233#issuecomment-887482585


   > How do you suggest that an application performs the loading and execution of the next application image?
   
   I would suggest using BOARDIOC_IOCTL.  That allows board-specific boarctl() commands.  It works exactly like the other BOARDIOC commands but does not clutter the system with garbage BOARDIOC commmands.  Imagine where that will go in the long run i we allow everyone to do that?
   
   The only real difference is that the board-specific IOCTL commands would be defined in a different header file in the board/include or in the arch/include directory.
   
   boardctl() is a nasty kludge to bypass the standard POSIX interface.  It really needs to be avoid as much as possible if you want to claim to be a portable, POSIX system.  But, unfortunately, there are things that need to be done that are not covered by any POSIX interfaces.  So we created this backdoor.  But we can only allow the backdoor to be open a crack or we will be swamped with board-/applicatoin-specific IOCTLs which would be a very bad consequence.
   
   Creating another BOARDIOC command does make it it has general usability but should never be done for a one-off solution.
   
   


-- 
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