Skip to content

rockchip: Add CSC board Luckfox Lyra Zero W#8764

Merged
igorpecovnik merged 1 commit intoarmbian:mainfrom
vidplace7:luckfox-lyra
Oct 16, 2025
Merged

rockchip: Add CSC board Luckfox Lyra Zero W#8764
igorpecovnik merged 1 commit intoarmbian:mainfrom
vidplace7:luckfox-lyra

Conversation

@vidplace7
Copy link
Member

@vidplace7 vidplace7 commented Oct 13, 2025

Description

Add new RK3506 CSC board "Luckfox Lyra Zero W".
Depends on:

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

  • ./compile.sh BOARD=luckfox-lyra-zero-w BRANCH=vendor KERNEL_CONFIGURE=no DEB_COMPRESS=xz KERNEL_BTF=yes INSTALL_ARMBIAN_FIRMWARE=yes BUILD_DESKTOP=no BUILD_MINIMAL=yes RELEASE=noble
  • Luckfox Lyra Zero W boots, shell works over UART0
  • AIC8800DC WiFi is not working currently. Assistance would be greatly appreciated 🙏
  • WiFi working after adding PACKAGE_LIST_BOARD="rfkill bluetooth bluez bluez-tools" (bluetooth power-enable)

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
  • 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 11 Milestone: Fourth quarter release size/small PR with less then 50 lines labels Oct 13, 2025
@github-actions
Copy link
Contributor

Hey @vidplace7! 👋

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! 🚀

@github-actions github-actions bot added Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... labels Oct 13, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 13, 2025

Walkthrough

Adds a new board configuration file config/boards/luckfox-lyra-zero-w.csc for the Luckfox Lyra Zero W (Rockchip RK3506B). Defines board metadata (BOARD_NAME, BOARDFAMILY, BOARD_MAINTAINER), boot settings (BOOTCONFIG, BOOT_FDT_FILE, BOOT_SCENARIO, BOOT_SOC=rk3506, DDR_BLOB=rk35/rk3506b_ddr_750MHz_v1.06.bin), KERNEL_TARGET, and IMAGE_PARTITION_TABLE. Enables the radxa-aic8800 extension with AIC8800_TYPE="usb". Sets PACKAGE_LIST_BOARD entries: rfkill, bluetooth, bluez, bluez-tools.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

BSP

Suggested reviewers

  • clee
  • prahal
  • rpardini
  • igorpecovnik
  • catalinii
  • SuperKali
  • Tonymac32
  • chainsx
  • pyavitz

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title succinctly and accurately describes the main change by indicating that a new Rockchip CSC board, Luckfox Lyra Zero W, is being added, making it clear and focused for readers.
Description Check ✅ Passed The pull request description directly addresses the addition of the RK3506-based board, its dependencies, test procedures, and validation results, making it clearly relevant and informative about the changes introduced.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

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 543f330 and cb2ad57.

📒 Files selected for processing (1)
  • config/boards/luckfox-lyra-zero-w.csc (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • config/boards/luckfox-lyra-zero-w.csc

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.

@vidplace7
Copy link
Member Author

WiFi working after adding PACKAGE_LIST_BOARD="rfkill bluetooth bluez bluez-tools" (bluetooth power-enable also affects wifi)
Un-Drafting / ready for review.

@vidplace7 vidplace7 marked this pull request as ready for review October 13, 2025 22:20
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

📜 Review details

Configuration used: CodeRabbit UI

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 39cc97e and 543f330.

📒 Files selected for processing (1)
  • config/boards/luckfox-lyra-zero-w.csc (1 hunks)
🔇 Additional comments (3)
config/boards/luckfox-lyra-zero-w.csc (3)

2-5: LGTM!

Board metadata is correctly defined with appropriate values for name, family, boot configuration, and maintainer.


14-14: LGTM!

The package list correctly includes rfkill, bluetooth, bluez, and bluez-tools, which you've confirmed resolves the WiFi functionality issue. The shared power control between Bluetooth and WiFi is properly addressed.


10-11: radxa-aic8800 extension supports USB configuration.

The extensions/radxa-aic8800.sh script includes a "usb" case with a valid aic8800-usb-dkms download URL and installation steps. No changes needed.

Copy link
Member

@EvilOlaf EvilOlaf left a comment

Choose a reason for hiding this comment

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

If this DDR_BLOB thingy mentioned by rabbit isn't an actual issue, lgtm

@github-actions github-actions bot added the Ready to merge Reviewed, tested and ready for merge label Oct 16, 2025
@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 Oct 16, 2025
@igorpecovnik igorpecovnik merged commit d57bce6 into armbian:main Oct 16, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

11 Milestone: Fourth quarter release Hardware Hardware related like kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines

Development

Successfully merging this pull request may close these issues.

3 participants