Skip to content

rockchip: add rtl8152 and dm9601 usb ethernet#9638

Open
netrunnereve wants to merge 1 commit intoarmbian:mainfrom
netrunnereve:main
Open

rockchip: add rtl8152 and dm9601 usb ethernet#9638
netrunnereve wants to merge 1 commit intoarmbian:mainfrom
netrunnereve:main

Conversation

@netrunnereve
Copy link
Copy Markdown

@netrunnereve netrunnereve commented Apr 7, 2026

Description

As discussed in #9630 this enables support for the dm9601 ethernet adapter on the Luckfox Lyra. I also had a rtl8152 adapter lying around and got that one working as well on my board.

My guess is that probably more adapters can be enabled here, but obviously I have no way to test that.

How Has This Been Tested?

Both usb ethernet adapters are working properly with network access on my Luckfox Lyra using the Lyra Plus board setup.

My compile command: ./compile.sh BOARD=luckfox-lyra-plus BRANCH=vendor PREFER_DOCKER=yes BUILD_DESKTOP=no BUILD_MINIMAL=yes RELEASE=trixie KERNEL_GIT=shallow EXTRAWIFI=no KERNEL_CONFIGURE=no KERNEL_BTF=yes

Checklist:

  • 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

  • New Features
    • Added support for USB-to-Ethernet adapters to enhance connectivity options.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 4d2878a3-aff3-409a-9882-26d4a5c1e67d

📥 Commits

Reviewing files that changed from the base of the PR and between 28862c7 and 757661b.

📒 Files selected for processing (1)
  • config/kernel/linux-rockchip-vendor.config

📝 Walkthrough

Walkthrough

The pull request enables specific USB-to-Ethernet network drivers (RTL8152, USBNET, DM9601) in the Rockchip vendor kernel configuration, replacing a previous blanket disable of USB network drivers while explicitly keeping other USB network driver options disabled.

Changes

Cohort / File(s) Summary
Kernel Configuration
config/kernel/linux-rockchip-vendor.config
Enabled three USB network drivers as modules (RTL8152, USBNET, DM9601) and explicitly disabled seven other USB network driver options. Replaced blanket USB network driver disable with selective driver configuration.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A kernel so fine, with drivers anew,
USB Ethernet ports now shine through,
RTL, USBNET, DM9601 sing,
Rockchip connectivity takes wing! 🌐

🚥 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 pull request title directly matches the main change: enabling USB Ethernet drivers (rtl8152 and dm9601) in the Rockchip kernel configuration.
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

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 size/small PR with less then 50 lines Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... 05 Milestone: Second quarter release labels Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

05 Milestone: Second quarter release Hardware Hardware related like kernel, U-Boot, ... Needs review Seeking for review size/small PR with less then 50 lines

Development

Successfully merging this pull request may close these issues.

1 participant