Fix sata2 target-supply for Orange Pi 3B and Station M2#10137
Conversation
The sata2 node for rk356x has changed in kernel 7.1 (see kernel commit 5918bf2). Update patches and migrate target-supply based on change.
📝 WalkthroughWalkthroughUpdates duplicated Rockchip device-tree patches across rockchip64-7.1 and rockchip64-7.2 kernel archives: an Orangepi 3B patch wires SATA2's target-supply via a new sata2_port0 node, and a Station M2 (rk3566-roc-pc) patch adds LED/headset nodes, GPIO/pinctrl reassignments, thermal trip, storage/SATA/UART/USB changes, and removes i2c1/i2c2/i2c3 blocks. ChangesRockchip64 kernel patch updates (7.1 and 7.2)
Estimated code review effort: 3 (Moderate) | ~25 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 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 |
|
Is this relevant for |
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/archive/rockchip64-7.1/board-station-m2.patch`:
- Around line 212-214: The added SATA supply reference uses a non-existent node
label, so update the change to point at the existing SATA node used in this
tree. Adjust the DTS fragment around the sata2 target to reference the real
RK356x SATA label or port subnode instead of sata2_port0, keeping the
target-supply assignment to vcc3v3_pcie unchanged.
🪄 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: 7cb8dbea-3bc3-42ff-897d-dfea9fd59ad3
📒 Files selected for processing (4)
patch/kernel/archive/rockchip64-7.1/board-orangepi3b-0002-specify-target-supply-for-sata2.patchpatch/kernel/archive/rockchip64-7.1/board-station-m2.patchpatch/kernel/archive/rockchip64-7.2/board-orangepi3b-0002-specify-target-supply-for-sata2.patchpatch/kernel/archive/rockchip64-7.2/board-station-m2.patch
|
@chainsx I took the liberty to update the Station M2 patch, as the changes are similar. FYI. |
Nope, since the change is introduced in kernel 7.1. |
could have been backported 🤷 |
|
Change is not backported to linux-6.18.y: |
Description
The sata2 node for rk356x has changed in kernel 7.1 (see kernel commit 5918bf2). Update patches and migrate target-supply based on change.
How Has This Been Tested?
Tested on Orange Pi 3B:
rockchip-rk3566-sata2.dtbo. After restart,lsblkdoes not detect sata SSD../compile.sh kernel-dtb BOARD=orangepi3b BRANCH=edge.lsblknow detects sata SSD.Not tested on Station M2.
Checklist:
Summary by CodeRabbit
New Features
Bug Fixes