[GitHub] [incubator-nuttx-apps] Donny9 commented on pull request #1142: nsh/script: support rc.sysinit script

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

Donny9 commented on PR #1142:
URL: https://github.com/apache/incubator-nuttx-apps/pull/1142#issuecomment-1101464914

   > This PR changes the init order, so I support that couple of existing configurations have to be updated:
   > 
   > ```
   > nuttx$ find . -name rcS
   > ./boards/sim/sim/sim/src/etc/init.d/rcS
   > ./boards/risc-v/qemu-rv/rv-virt/src/etc/init.d/rcS
   > ./boards/risc-v/esp32c3/esp32c3-devkit/src/etc/init.d/rcS
   > ```
   > 
   > I mean that we need to rework configs that use rcS as behavior will change. For example description states that after this change `run rc.sysinit script(mount fs, run core service)` and `boards/risc-v/esp32c3/esp32c3-devkit/src/etc/init.d/rcS` contains:
   > 
   > ```
   > /* Create a RAMDISK and mount it at /tmp */
   > 
   > mkrd -m CONFIG_NSH_FATDEVNO -s CONFIG_NSH_FATSECTSIZE CONFIG_NSH_FATNSECTORS
   > mkfatfs CONCAT(/dev/ram, CONFIG_NSH_FATDEVNO)
   > mount -t vfat CONCAT(/dev/ram, CONFIG_NSH_FATDEVNO) CONFIG_NSH_FATMOUNTPT
   > ```
   > 
   > So according to new description the mount part of `boards/risc-v/esp32c3/esp32c3-devkit/src/etc/init.d/rcS` should be moved to `boards/risc-v/esp32c3/esp32c3-devkit/src/etc/init.d/rc.sysinit`. The same for other.
   
   Done. 


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