Skip to content

doc: Update 'soc-cfg' usage on Nuclei qemu#61

Open
muxiff wants to merge 1 commit intomasterfrom
nuclei/lx-dev
Open

doc: Update 'soc-cfg' usage on Nuclei qemu#61
muxiff wants to merge 1 commit intomasterfrom
nuclei/lx-dev

Conversation

@muxiff
Copy link
Contributor

@muxiff muxiff commented Mar 3, 2026

Important

Update source/qemu/intro.rst to correct address allocation and enhance JSON configuration examples for Nuclei QEMU.

  • Documentation Updates:
    • Corrected address allocation table by moving IINFO to Peripherals (Offset) and MROM to Peripherals (Fixed) in source/qemu/intro.rst.
    • Added size attribute to uart0 and uart1 in JSON configuration example.
    • Added enable attribute to qspi2 and plic attribute to iregion in JSON configuration example.
    • Updated description of irqmax to specify total IRQs must be greater than 40.
    • Removed note about base attribute format requirements in JSON configuration.

This description was created by Ellipsis for 1b80648. You can customize this summary. It will automatically update as commits are pushed.

@muxiff muxiff requested a review from fanghuaqi March 3, 2026 07:18
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Looks good to me! 👍

Reviewed everything up to 1b80648 in 15 seconds. Click for details.
  • Reviewed 80 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

Workflow ID: wflow_0jxwbrlgZ0hB199w

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@fanghuaqi
Copy link
Member

需要更新一下changelog,暂定 2026.06 , 里面就写哪些特性是这个版本引入的

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants