Add board: EASY EAI Nano RV1126#10154
Open
hqnicolas wants to merge 1 commit into
Open
Conversation
hqnicolas
requested review from
ColorfulRhino,
amazingfate,
paolosabatino and
vidplace7
as code owners
Contributor
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughAdds EASY-EAI Nano RV1126 board support across Armbian configuration, U-Boot, Linux device trees and defconfig, RV1126 family build logic, kernel patches, boot environment, and boot scripts. ChangesEASY-EAI Nano RV1126
Estimated code review effort: 4 (Complex) | ~60 minutes Suggested labels: Suggested reviewers: Sequence Diagram(s)sequenceDiagram
participant ArmbianBuild
participant UBoot
participant BootMedia
participant LinuxKernel
ArmbianBuild->>UBoot: build EASY-EAI Nano RV1126 artifacts
UBoot->>BootMedia: write idbloader.img and u-boot.itb
BootMedia->>UBoot: load boot.scr
UBoot->>BootMedia: load uInitrd, zImage, and DTB
UBoot->>LinuxKernel: bootz with bootargs and hardware DTB
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
github-actions
Bot
added
08
hqnicolas
requested review from
150balbes,
EvilOlaf,
HeyMeco,
andyshrk,
iav,
igorpecovnik and
rpardini
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@config/bootscripts/boot-rv1126.cmd`:
- Around line 6-9: Move the ramdisk_addr_r definition in the boot environment
settings above the kernel region, changing it from the low-memory 0x01000000
address to a safe address above kernel_addr_r, consistent with the other boot
scripts.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: d8774835-bbc4-48f8-8716-356ddcc9452f
📒 Files selected for processing (9)
config/boards/easy-eai-nano.cscconfig/bootenv/rv1126.txtconfig/bootscripts/boot-rv1126.cmdconfig/kernel/linux-rockchip-rv1126-vendor.configconfig/sources/families/rockchip-rv1126.confpatch/kernel/rv1126-vendor-6.1/0000.patching_config.yamlpatch/kernel/rv1126-vendor-6.1/dt/lmo-display.dtsipatch/kernel/rv1126-vendor-6.1/dt/rv1126-nano.dtspatch/u-boot/v2026.07/board_easy-eai-nano/add-board-easy-eai-nano-rv1126.patch
✅ Files skipped from review due to trivial changes (1)
- config/bootenv/rv1126.txt
🚧 Files skipped from review as they are similar to previous changes (7)
- patch/kernel/rv1126-vendor-6.1/0000.patching_config.yaml
- config/boards/easy-eai-nano.csc
- config/kernel/linux-rockchip-rv1126-vendor.config
- config/sources/families/rockchip-rv1126.conf
- patch/kernel/rv1126-vendor-6.1/dt/lmo-display.dtsi
- patch/u-boot/v2026.07/board_easy-eai-nano/add-board-easy-eai-nano-rv1126.patch
- patch/kernel/rv1126-vendor-6.1/dt/rv1126-nano.dts
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@patch/kernel/rv1126-vendor-6.1/0002-rknpu-fix-arm32-gem-build.patch`:
- Around line 39-46: Replace dmac_flush_range in the CONFIG_ARM branch of the
DMA_FROM_DEVICE cache-maintenance path with an ARM invalidate-only helper
equivalent to dcache_inval_poc, preserving the existing address range; do not
clean CPU cache lines before invalidation unless the buffer’s CPU-dirty state is
explicitly guaranteed impossible.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 395bc66e-98be-4ff4-92f0-6b3aa9d573d3
📒 Files selected for processing (11)
config/boards/easy-eai-nano.cscconfig/bootenv/rv1126.txtconfig/bootscripts/boot-rv1126.cmdconfig/kernel/linux-rockchip-rv1126-vendor.configconfig/sources/families/rockchip-rv1126.confpatch/kernel/rv1126-vendor-6.1/0000.patching_config.yamlpatch/kernel/rv1126-vendor-6.1/0001-rknpu-add-rv1126-compatible.patchpatch/kernel/rv1126-vendor-6.1/0002-rknpu-fix-arm32-gem-build.patchpatch/kernel/rv1126-vendor-6.1/dt/lmo-display.dtsipatch/kernel/rv1126-vendor-6.1/dt/rv1126-nano.dtspatch/u-boot/v2026.07/board_easy-eai-nano/add-board-easy-eai-nano-rv1126.patch
✅ Files skipped from review due to trivial changes (3)
- patch/kernel/rv1126-vendor-6.1/0000.patching_config.yaml
- config/bootenv/rv1126.txt
- config/kernel/linux-rockchip-rv1126-vendor.config
🚧 Files skipped from review as they are similar to previous changes (4)
- config/boards/easy-eai-nano.csc
- patch/kernel/rv1126-vendor-6.1/dt/lmo-display.dtsi
- config/sources/families/rockchip-rv1126.conf
- patch/u-boot/v2026.07/board_easy-eai-nano/add-board-easy-eai-nano-rv1126.patch
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@config/bootscripts/boot-rv1126.cmd`:
- Line 30: The logo=disabled branch in the boot script sets the logo environment
variable but never propagates it to the kernel command line. Update the final
bootargs construction to include ${logo}, or remove the branch if logo
suppression is not supported; use the existing logo and bootargs definitions to
make the change.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 0fb9b193-0ee1-433a-aea7-851a5f1bcf5b
📒 Files selected for processing (11)
config/boards/easy-eai-nano.cscconfig/bootenv/rv1126.txtconfig/bootscripts/boot-rv1126.cmdconfig/kernel/linux-rockchip-rv1126-vendor.configconfig/sources/families/rockchip-rv1126.confpatch/kernel/rv1126-vendor-6.1/0000.patching_config.yamlpatch/kernel/rv1126-vendor-6.1/0001-rknpu-add-rv1126-compatible.patchpatch/kernel/rv1126-vendor-6.1/0002-rknpu-fix-arm32-gem-build.patchpatch/kernel/rv1126-vendor-6.1/dt/lmo-display.dtsipatch/kernel/rv1126-vendor-6.1/dt/rv1126-nano.dtspatch/u-boot/v2026.07/board_easy-eai-nano/add-board-easy-eai-nano-rv1126.patch
✅ Files skipped from review due to trivial changes (3)
- patch/kernel/rv1126-vendor-6.1/0000.patching_config.yaml
- config/boards/easy-eai-nano.csc
- config/bootenv/rv1126.txt
🚧 Files skipped from review as they are similar to previous changes (7)
- patch/kernel/rv1126-vendor-6.1/0001-rknpu-add-rv1126-compatible.patch
- patch/kernel/rv1126-vendor-6.1/0002-rknpu-fix-arm32-gem-build.patch
- config/kernel/linux-rockchip-rv1126-vendor.config
- patch/kernel/rv1126-vendor-6.1/dt/lmo-display.dtsi
- config/sources/families/rockchip-rv1126.conf
- patch/u-boot/v2026.07/board_easy-eai-nano/add-board-easy-eai-nano-rv1126.patch
- patch/kernel/rv1126-vendor-6.1/dt/rv1126-nano.dts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add initial Armbian support for the EASY-EAI Nano RV1126 board.
Forum topic:
https://forum.armbian.com/topic/60848-board-bring-up-easy-eai-nano-rv1126/
Build Logs:
1: https://paste.armbian.eu/garowadifa
2: https://paste.armbian.eu/xepadufujo
3: https://paste.armbian.eu/ozadenipas
Latest: https://paste.armbian.eu/itarutexiy
Board Log:
1: https://paste.armbian.com/ijefusiqun
2: https://paste.armbian.com/zabeyaseno
3: https://paste.armbian.com/korafirizu
Latest: https://paste.armbian.com/nopudunije
Introducing a new CSC board target for Rockchip RV1126 using the vendor 6.1 Rockchip kernel branch and Mainline U-Boot 2026.07. The board is an RV1126-based module with 2 GB DDR3, onboard eMMC, SD-card boot support.
How Has This Been Tested?
BOARD=easy-eai-nano BRANCH=vendor.Checklist:
Summary by CodeRabbit
Summary