Skip to content

extensions: radxa-aic8800: allow 6.19; skip DKMS only on >= 6.20#9397

Open
HackingGate wants to merge 1 commit intoarmbian:mainfrom
HackingGate:allow-aic8800-6.19
Open

extensions: radxa-aic8800: allow 6.19; skip DKMS only on >= 6.20#9397
HackingGate wants to merge 1 commit intoarmbian:mainfrom
HackingGate:allow-aic8800-6.19

Conversation

@HackingGate
Copy link
Contributor

@HackingGate HackingGate commented Feb 13, 2026

How Has This Been Tested?

It builds on Linux 6.19.0-edge-rockchip64, and I have tested the functionality on my photonicat2 device.

Summary by CodeRabbit

  • Bug Fixes
    • Adjusted kernel version compatibility requirements to improve driver installation stability and reliability across supported platforms.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 13, 2026

📝 Walkthrough

Walkthrough

Updated the kernel version threshold in the AIC8800 DKMS package installation gate from 6.19 to 6.20. This adjusts the "kernel too new" check, reducing the range of kernel versions that trigger the skip condition for this specific driver installation.

Changes

Cohort / File(s) Summary
AIC8800 Kernel Version Gate
extensions/radxa-aic8800.sh
Updated conditional kernel version check from ge 6.19 to ge 6.20 in the post_install_kernel_debs__install_aic8800_dkms_package function, tightening the version compatibility threshold.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

Ready to merge

Suggested reviewers

  • igorpecovnik
  • krachlatte
  • prahal
  • pyavitz

Poem

🐰 A kernel check, so precise,
From 6.19 to 6.20 spice,
The threshold shifts, the bounds align,
AIC8800 now draws the line! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adjusting the kernel version gate from 6.19 to 6.20 in the radxa-aic8800 extension to allow 6.19 builds while skipping DKMS only on 6.20+.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

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

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 Framework Framework components 02 Milestone: First quarter release labels Feb 13, 2026
@github-actions github-actions bot added the Ready to merge Reviewed, tested and ready for merge label Feb 13, 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 13, 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 Framework Framework components 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.

2 participants