lupyuen commented on PR #15188:
URL: https://github.com/apache/nuttx/pull/15188#issuecomment-2547379497

   > I could generate it so that it stays in `tools/bcm2711` with the other 
boot files?
   
   @linguini1 Hmmm the defconfig check might still fail, if it detects new 
files in `tools/bcm2711`? We need to be careful when generating new files in 
POSTBUILD. [(Here's an example of a complex 
POSTBUILD)](https://github.com/apache/nuttx/blob/master/tools/imx9/Config.mk#L64-L100)
   
   Maybe: We could generate the files inside `/staging` or `/build`, since 
[.gitignore](https://github.com/apache/nuttx/blob/master/.gitignore) will skip 
them
   
   Sorry I can't think of any good solution right now, and I think it's really 
good to merge this PR as-is, right now, with minimal tweaks. Wonder if we could 
hardcode `tools/bcm2711/config.txt` for now, and uncomment manually if 
required? (Until we fix it next release)
   ```
   ## Uncomment this for Debug Boot
   ## uart_2ndstage=1
   ```


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