tmedicci commented on PR #18373:
URL: https://github.com/apache/nuttx/pull/18373#issuecomment-3877861655

   > @tmedicci
   > 
   > Ran the requested ESP32 QEMU validation locally and confirmed boot to NSH.
   > ### QEMU command used
   > 
   > ```shell
   > ~/.local/espressif-qemu-xtensa/qemu/bin/qemu-system-xtensa \
   >   -nographic \
   >   -machine esp32 \
   >   -drive file=nuttx.merged.bin,if=mtd,format=raw \
   >   -nic user,model=open_eth
   > ```
   > 
   > ### Build/config check
   > 
   > ```shell
   > grep -n 
"CONFIG_ESP32_IGNORE_CHIP_REVISION_CHECK\|CONFIG_ARCH_CHIP_ESP32\|CONFIG_BOARD_ESP32_DEVKITC"
 .config
   > ```
   > 
   > Output:
   > 
   > ```
   > 137:CONFIG_ARCH_CHIP_ESP32=y
   > 192:CONFIG_ESP32_IGNORE_CHIP_REVISION_CHECK=y
   > ```
   > 
   > ### Boot logs (key excerpts)
   > 
   > ```
   > Adding SPI flash device
   > ets Jul 29 2019 12:21:46
   > ...
   > [esp32] [INF] *** Booting MCUboot build v2.2.0-151-g8a07053d ***
   > [esp32] [INF] [boot] chip revision: v0.0
   > ...
   > [esp32] [INF] Loading image 0 - slot 0 from flash, area id: 1
   > ...
   > WARNING: NuttX supports ESP32 chip revision >= v3.0 (chip is v0.0).
   > Ignoring this error and continuing because 
`ESP32_IGNORE_CHIP_REVISION_CHECK` is set...
   > THIS MAY NOT WORK! DON'T USE THIS CHIP IN PRODUCTION!
   > 
   > NuttShell (NSH) NuttX-12.12.0
   > nsh>
   > ```
   > 
   > ### Runtime sanity check in NSH
   > 
   > ```shell
   > uname -a
   > ```
   > 
   > Output:
   > 
   > ```
   > NuttX  12.12.0 637c3300a5 Feb 10 2026 19:09:42 xtensa esp32-devkitc
   > ```
   > 
   > ### Result
   > 
   >     * QEMU boot is successful and reaches NSH on 
`esp32-devkitc:qemu_openeth`.
   
   Thanks, @aviralgarg05. 
   
   It wasn't clear if you ran the tests you mentioned in the PR description 
using QEMU or not.
   
   I suggest you update the Testing section and include those tests (either 
with `mcuboot` or with `nxboot`, preferably both) and the commands you used to 
run the firmware and the test.
   
   You can use `esp32s3-devkit:nsh` as the base defconfig. The following 
commands append the necessary changes to build the firmware for QEMU:
   
   ```
   kconfig-tweak -e ESP32_QEMU_IMAGE && kconfig-tweak -e ESP32_MERGE_BIN
   ```
   
   I suggest you set other Kconfig options using `kconfig-tweak` as it is 
easier to replicate. Don't forget to run `make olddefconfig` before building 
the firmware.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to