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/12/12 03:21:57 UTC

[GitHub] [incubator-nuttx] nandojve removed a comment on pull request #4939: boards/arm/samv7: Introduce common folder

nandojve removed a comment on pull request #4939:
URL: https://github.com/apache/incubator-nuttx/pull/4939#issuecomment-986139617


   Hi @pkarashchenko ,
   
   Thank you for your comments.
   
   I was thinking more about how integrade bootloaders into NuttX. Besides this PR helps to unify samv7 it still lacks for all NuttX platforms. If makes sense, I would like split this PR and keep only first 2 commits. The next step will be create a RFC to define all those variables that are required to allow an agnostic bootloader. I mean, there is no sense an App outside NuttX define things that are HW dependent. I think it should be the contrary. This way it will be possible that any external app pick configs from NuttX and configure itself which allows an agnostic bootloader.
   
   If above makes sense I prefer split PR.
   
   What do you think?


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