radxa-rock-4d: Enable Wi-Fi & Bluetooth (same logic as rock-2a Commit 501f5a7 @CodeChenL)

[letwir]

Description

install scripts has aic8800 drivers, but missing.
so, I apply same vendor config. Wi-Fi and Bluetooth Enabled !

by porting the configuration from the rock-2a config

GitHub issue reference:
Jira reference number AR-2791

Documentation summary for feature / change

radxa-rock-4d: Enable Wi-Fi & Bluetooth

• short description (copy / paste of PR title)
• summary (description relevant for end users)
• example of usage (how to see this in function)

How Has This Been Tested?

build normal kernel (this option add).

• On board (radxa-rock-4d,Armbian v26.2.0-trunk.12). Verified that both Wi-Fi and Bluetooth are now operational.

Checklist:

Please delete options that are not relevant.

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• My changes generate no new warnings

Summary by CodeRabbit

• Chores
  • Added support for AIC8800 module on Radxa Rock 4D board configuration via USB connectivity.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

This PR adds WiFi and Bluetooth support to the Radxa ROCK 4D board by enabling the AIC8800 wireless extension and configuring it for USB connection in the board configuration file.

Changes

Cohort / File(s)	Summary
Radxa ROCK 4D Wireless Configuration config/boards/radxa-rock-4d.conf	Added enable_extension for radxa-aic8800 and set AIC8800_TYPE to usb to enable WiFi/Bluetooth support

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

• Single file modification with two straightforward configuration additions
• No logic changes, control flow modifications, or complex interactions

Possibly related issues

• [Feature Request]: Enable WiFi/BT for Radxa ROCK 4D on Armbian 26.02.0 #9024 — Original feature request that directly proposes these exact configuration changes for Radxa ROCK 4D WiFi/BT enablement

Possibly related PRs

• Radxa Zero 3: fix bluetooth, for edge & current, switch to u-boot master #8511 — Adds similar AIC8800 configuration flags to other Radxa boards
• radxa-zero3: enable wifi extension by default #8049 — Adds enable_extension "radxa-aic8800" configuration to board setup files
• rockchip: Add CSC board Luckfox Lyra Zero W #8764 — Applies identical AIC8800_TYPE="usb" configuration pattern to Radxa boards

Suggested reviewers

• igorpecovnik
• RadxaYuntian
• chainsx
• pyavitz
• HeyMeco

Poem

🐰 A wireless wish comes into sight,
Two lines of config set it right—
AIC8800 takes its place,
WiFi blooms across the space! ✨

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: enabling Wi-Fi and Bluetooth support for the Radxa ROCK 4D board by applying the same configuration logic from the ROCK 2A board.
Linked Issues check	✅ Passed	The PR successfully implements the coding requirements from AR-2791: adds enable_extension and AIC8800_TYPE configuration to the board file, matching the proposed change in the feature request.
Out of Scope Changes check	✅ Passed	All changes are scoped to the stated objective: only the radxa-rock-4d.conf board configuration file is modified to enable Wi-Fi and Bluetooth support, with no unrelated changes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between d878139 and 5badc05.

📒 Files selected for processing (1)

• config/boards/radxa-rock-4d.conf (1 hunks)

🧰 Additional context used

🧠 Learnings (7)

📓 Common learnings

Learnt from: igorpecovnik
Repo: armbian/build PR: 8789
File: config/kernel/linux-sunxi64-edge.config:839-839
Timestamp: 2025-10-22T07:51:53.015Z
Learning: In Armbian's wireless driver configuration, CONFIG_RK_WIFI_DEVICE_* options are Rockchip-specific device registration settings, while CONFIG_WLAN_UWE* are the actual cross-platform driver modules for UWE5622 wireless chips. The UWE5622 chip (manufactured by Unisoc/Spreadtrum) is used on multiple Allwinner boards including Orange Pi Zero 2/2W/3 and Orange Pi 3 LTS, so CONFIG_WLAN_UWE5621/5622 along with Unisoc infrastructure options (CONFIG_SPARD_WLAN_SUPPORT, CONFIG_SC23XX, CONFIG_UNISOC_WIFI_PS, CONFIG_WCN_BSP_DRIVER_BUILDIN) are correct in sunxi64 kernel configs.

📚 Learning: 2025-09-18T03:36:17.862Z

Learnt from: EvilOlaf
Repo: armbian/build PR: 8537
File: config/boards/qcom-robotics-rb5.conf:1-1
Timestamp: 2025-09-18T03:36:17.862Z
Learning: In Armbian board configuration files (regardless of file extension), the standard pattern is to have only one line as a comment describing the board hardware specifications. This single-line description typically includes the SoC model, core count, RAM options, and key features like connectivity options, storage interfaces, and special features. The file extensions indicate board support status: .conf (standard support), .csc (community supported), .eos (end of support), .tvb (tvbox), .wip (work in progress).

Applied to files:

• config/boards/radxa-rock-4d.conf

📚 Learning: 2025-09-18T03:36:17.862Z

Learnt from: EvilOlaf
Repo: armbian/build PR: 8537
File: config/boards/qcom-robotics-rb5.conf:1-1
Timestamp: 2025-09-18T03:36:17.862Z
Learning: In Armbian board configuration files, .wip and .conf extensions require BOARD_MAINTAINER to be present and contain at least one maintainer. If no maintainer is present, the board support rules state it does not qualify for standard support and must be moved to community support (.csc extension). This is documented at https://docs.armbian.com/User-Guide_Board-Support-Rules/

Applied to files:

• config/boards/radxa-rock-4d.conf

📚 Learning: 2025-11-02T20:49:56.719Z

Learnt from: igorpecovnik
Repo: armbian/build PR: 8849
File: config/boards/radxa-e54c.csc:14-28
Timestamp: 2025-11-02T20:49:56.719Z
Learning: In Armbian board configuration files (config/boards/*.conf, *.csc, etc.), do not use kernel_config_set, kernel_config_set_m, kernel_config_set_y, or custom_kernel_config__* functions to modify kernel configuration. Kernel configuration is associated with LINUXFAMILY/BOARDFAMILY, not individual BOARD. Board-specific kernel modifications cause inconsistency in kernel packages published to the apt repository because boards within a family share the same kernel packages. Kernel configuration changes must be made in the appropriate kernel config file (e.g., config/kernel/linux-*-*.config) or in family configuration files (config/sources/families/*.conf, *.inc) instead.

Applied to files:

• config/boards/radxa-rock-4d.conf

📚 Learning: 2025-10-26T10:41:35.118Z

Learnt from: HackingGate
Repo: armbian/build PR: 8665
File: config/boards/photonicat2.csc:4-4
Timestamp: 2025-10-26T10:41:35.118Z
Learning: In the Armbian build system, rk3576 boards consistently use BOARDFAMILY="rk35xx" for both vendor and edge kernel targets. The rk35xx family configuration sources rockchip64_common.inc, which provides edge and current kernel branch definitions, making these branches available even though they're not defined directly in rk35xx.conf.

Applied to files:

• config/boards/radxa-rock-4d.conf

📚 Learning: 2025-09-12T21:44:09.061Z

Learnt from: Grippy98
Repo: armbian/build PR: 8624
File: config/boards/sk-am62p.conf:8-8
Timestamp: 2025-09-12T21:44:09.061Z
Learning: For TI K3 family boards in Armbian, BOOT_FDT_FILE uses .dts extension (not .dtb) as the standard convention. The build system handles this correctly by automatically compiling .dts to .dtb during kernel build and using the BOOT_FDT_FILE value directly in bootloader configurations.

Applied to files:

• config/boards/radxa-rock-4d.conf

📚 Learning: 2025-09-14T06:32:29.806Z

Learnt from: amazingfate
Repo: armbian/build PR: 8619
File: config/sources/families/rockchip.conf:222-230
Timestamp: 2025-09-14T06:32:29.806Z
Learning: In the Armbian build system, the write_uboot_platform() function implementations follow different patterns across Rockchip family files. The newer standard (used in rockchip64_common.inc and rk3506) includes 'status=none' parameter in dd commands, while older implementations (rk3288, rk322x) use an older pattern without this parameter. The rk3506 implementation correctly follows the current Rockchip family standard.

Applied to files:

• config/boards/radxa-rock-4d.conf

🔇 Additional comments (1)

config/boards/radxa-rock-4d.conf (1)

15-16: Extension configuration is correct

The enable_extension "radxa-aic8800" with AIC8800_TYPE="usb" follows the standard Armbian pattern and matches the working rock-2a.conf configuration. The variable name aligns with what the radxa-aic8800 extension script expects.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Hey @letwir! 👋

Thanks for submitting your first pull request to the Armbian project — we're excited to have you contributing! 🧡
Your effort doesn’t just improve Armbian — it benefits the entire community of users and developers.

If you'd like to stay informed about project updates or collaborate more closely with the team,
you can optionally share some personal contact preferences at armbian.com/update-data.
This helps us keep in touch without relying solely on GitHub notifications.

Also, don’t forget to ⭐ star the repo if you haven’t already — and welcome aboard! 🚀

[HeyMeco]

@letwir thanks for the PR merging now!