Skip to content

rockchip: Add CSC board ForLinx OK3506-S12 (RK3506J)#9394

Merged
igorpecovnik merged 1 commit intoarmbian:mainfrom
vidplace7:forlinx-ok3506-s12
Feb 12, 2026
Merged

rockchip: Add CSC board ForLinx OK3506-S12 (RK3506J)#9394
igorpecovnik merged 1 commit intoarmbian:mainfrom
vidplace7:forlinx-ok3506-s12

Conversation

@vidplace7
Copy link
Member

@vidplace7 vidplace7 commented Feb 12, 2026

Description

Adds new RK3506J CSC board ForLinx OK3506-S12.
This PR also includes uboot patches (dts/defconfig) to enable this board.

This board uses 0.45W at idle (0.36W with Ethernet disabled). 🌞

Depends on:

How Has This Been Tested?

  • ./compile.sh BOARD=forlinx-ok3506-s12 BRANCH=vendor KERNEL_CONFIGURE=no DEB_COMPRESS=xz KERNEL_BTF=yes INSTALL_ARMBIAN_FIRMWARE=no BUILD_DESKTOP=no BUILD_MINIMAL=yes RELEASE=trixie
  • ./compile.sh BOARD=forlinx-ok3506-s12 BRANCH=vendor KERNEL_CONFIGURE=no DEB_COMPRESS=xz KERNEL_BTF=yes INSTALL_ARMBIAN_FIRMWARE=no BUILD_DESKTOP=no BUILD_MINIMAL=yes RELEASE=noble
  • Shell works over builtin USB debug interface (CH340)
  • Ethernet works

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

@github-actions github-actions bot added 02 Milestone: First quarter release size/large PR with 250 lines or more labels Feb 12, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 12, 2026

📝 Walkthrough

Walkthrough

Adds comprehensive support for the Forlinx OK3506-S12 board (RK3506-based SoC) by introducing board configuration, U-Boot build settings, patching directives, and device tree definitions across five new files totaling 319 lines.

Changes

Cohort / File(s) Summary
Board Configuration
config/boards/forlinx-ok3506-s12.csc
Defines board identity, vendor, family, boot configuration, maintainer, kernel target, DTB file, partition table, serial console settings, SoC type, and DDR blob for the RK3506-based platform.
U-Boot Build System
patch/u-boot/u-boot-rk3506/0000.patching_config.yaml
Establishes overlay mappings for patching: defconfig→configs and dt→arch/arm/dts directories.
U-Boot Configuration
patch/u-boot/u-boot-rk3506/defconfig/forlinx-ok3506-s12-rk3506j_defconfig
Provides comprehensive U-Boot defconfig enabling RK3506 architecture, device tree support, boot/serial parameters, storage (MMC/MTD/USB), networking (Ethernet/MDIO), USB gadget, GPIO, LED, SFC, and rockchip-specific drivers with UART baud rate set to 1500000.
Device Trees
patch/u-boot/u-boot-rk3506/dt/rk3506j-forlinx-ok3506-s12-u-boot.dtsi, patch/u-boot/u-boot-rk3506/dt/rk3506j-forlinx-ok3506-s12.dts
U-Boot device tree fragment defining boot-phase designations for SPI flash and SD/MMC; main DTS file defining board metadata, power regulators, Ethernet (gmac0/gmac1), pinctrl mappings, UART, USB OTG, SD/MMC, ADC keys, and voltage rails.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A new board hops into view,
RK3506 dreams come true,
Config files all neat and clean,
The finest board we've ever seen!
Device trees bloom like spring grass,
Forlinx folks will have a blast! 🌱

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding support for a specific board (ForLinx OK3506-S12) to the Rockchip platform.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
patch/u-boot/u-boot-rk3506/dt/rk3506j-forlinx-ok3506-s12.dts (1)

13-18: Inconsistent alias value syntax: phandle vs. path reference.

Lines 14–15 use <&label> (phandle encoding) while lines 16–17 use &label (path reference). In the aliases node, values should be path references (strings). While modern dtc tolerates both, the canonical form for aliases is without angle brackets.

♻️ Suggested fix for consistency
 	aliases {
-		ethernet0 = <&gmac0>;
-		ethernet1 = <&gmac1>;
+		ethernet0 = &gmac0;
+		ethernet1 = &gmac1;
 		mmc0 = &sdmmc;
 		serial0 = &uart0;
 	};

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot added Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... labels Feb 12, 2026
@github-actions
Copy link
Contributor

🚫 Missing required board assets

This PR adds new board configuration(s). Required assets must already exist in github/armbian/armbian.github.io.
They are required by Armbian Imager to ensure all boards are displayed with proper images.

  • Board images: board-images/<board>.png (1920x1080 px transparent)
  • Vendor logos: board-vendor-logos/<vendor>-logo.png (512x512 px transparent)

Missing items

  • Board image missing for forlinx-ok3506-s12

    • Expected: board-images/forlinx-ok3506-s12.png
    • Fix: add the file to armbian/armbian.github.io (folder board-images/)
  • Vendor logo missing for vendor forlinx (used by board forlinx-ok3506-s12)

    • Expected: board-vendor-logos/forlinx-logo.png
    • Fix: add the file to armbian/armbian.github.io (folder board-vendor-logos/)
    • Naming rules: lowercase, dashes (e.g. kobol-logo.png, not Kobol_logo.png)

Once the missing files are added (or a PR is opened in armbian/armbian.github.io), re-run this check.

@igorpecovnik igorpecovnik merged commit 396ccb4 into armbian:main Feb 12, 2026
12 checks passed
@github-actions github-actions bot added the Ready to merge Reviewed, tested and ready for merge label Feb 12, 2026
@github-actions
Copy link
Contributor

✅ This PR has been reviewed and approved — all set for merge!

@github-actions github-actions bot removed the Needs review Seeking for review label Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

02 Milestone: First quarter release Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/large PR with 250 lines or more

Development

Successfully merging this pull request may close these issues.

2 participants