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 2021/07/27 13:54:52 UTC

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

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


   > A few days ago, Xiomi added they one application-specific, boot-related BOARDIOC, PR #4191
   > 
   > This is really getting out of hand quickly. We need to do something to avoid the clutter. Perhaps there should be a single BOARDIOC_BOOTCMD that has sub-commands (like BOARDIOC_IOCTL) that at least encapsulates bootloader/boot commands and minimizes the uncontrolled, unmanaged, useless (to most people) clutter.
   
   Ok, you want to design the board ioctl to the different layer. It's fine to encapsulate all boot related fucntionality in to sub IOCTL. My point is that IOCTL need standardize, but either top IOCTL or sub IOCTL is fine for me.


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