Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@EvilOlaf
Copy link
Member

@EvilOlaf EvilOlaf commented Dec 17, 2025

Description

  • Abandon 6.6.y as legacy
  • Move current 6.12.y to legacy
  • Move edge 6.18.y to current
  • bump edge to 6.19-rc1
  • drop orphaned patch folders

How Has This Been Tested?

Checklist:

  • 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

Summary by CodeRabbit

  • Updates
    • Updated kernel versions for BCM2711 hardware across multiple branches.
    • Modified USB mass storage device identification strings for USB gadget functionality.
    • Improved CPU info display consistency.
    • Adjusted USB remote wakeup capabilities across kernel versions.

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 17, 2025

Walkthrough

Kernel version updates for bcm2711 across legacy (6.6→6.12), current (6.12→6.18), and edge (6.18→6.19) branches with corresponding patch files for each kernel version, including CPU info consistency, platform selection configuration, USB gadget wakeup functionality, and mass storage product naming customization.

Changes

Cohort / File(s) Change Summary
Kernel Version Configuration
config/sources/families/bcm2711.conf
Updates KERNEL_MAJOR_MINOR and KERNELBRANCH across legacy, current, and edge branches; legacy: 6.6→6.12, current: 6.12→6.18, edge: 6.18→6.19
bcm2711-6.1 Patches
patch/kernel/archive/bcm2711-6.1/0001-Make-proc-cpuinfo-consistent-on-arm64-and-arm.patch, patch/kernel/archive/bcm2711-6.1/001-force-platform-selection-brcm-soc-support.patch.disabled, patch/kernel/archive/bcm2711-6.1/1003-remote-wakeup.patch, patch/kernel/archive/bcm2711-6.1/2001-rename-msd.patch
Removes conditional model name printing in cpuinfo; removes BCM platform selection from ARM/ARM64 Kconfig; removes USB gadget wakeup functionality (DWC2 wakeup, HID wakeup_on_write); changes mass storage vendor/product strings from Linux/File-based to PiKVM/Drive-based
bcm2711-6.6 Patches
patch/kernel/archive/bcm2711-6.6/0001-Make-proc-cpuinfo-consistent-on-arm64-and-arm.patch, patch/kernel/archive/bcm2711-6.6/001-force-platform-selection-brcm-soc-support.patch.disabled, patch/kernel/archive/bcm2711-6.6/1003-remote-wakeup.patch, patch/kernel/archive/bcm2711-6.6/2001-rename-msd.patch
Same pattern as bcm2711-6.1: removes conditional cpuinfo printing, removes BCM Kconfig selects, removes USB gadget wakeup (DWC2 and HID implementations), changes mass storage product identification strings
bcm2711-6.10 Patches
patch/kernel/archive/bcm2711-6.10/0001-Make-proc-cpuinfo-consistent-on-arm64-and-arm.patch, patch/kernel/archive/bcm2711-6.10/001-force-platform-selection-brcm-soc-support.disabled, patch/kernel/archive/bcm2711-6.10/1003-remote-wakeup.patch
Removes cpuinfo conditional printing and BCM Kconfig platform selection; removes USB gadget wakeup features (includes DWC2 wakeup hook, HID wakeup_on_write option, and related configuration attributes)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20–25 minutes

  • Repetitive pattern recognition: Same patch modifications applied across three kernel versions (6.1, 6.6, 6.10), which reduces unique reasoning required per file
  • Kernel version alignment verification: Ensure version bumps in bcm2711.conf (6.6→6.12→6.18→6.19) correctly correspond to updated patch directories
  • USB gadget wakeup implementation: Review consistency of wakeup feature removal across DWC2 driver and HID gadget function (complex logic with spinlocks, state management, and multiple code paths in 6.1/6.6/6.10)
  • Potential cross-version inconsistencies: Verify that 6.10 patch file does not include 2001-rename-msd.patch (appears to be missing/removed vs 6.1 and 6.6)

Possibly related PRs

Suggested reviewers

  • PanderMusubi
  • igorpecovnik
  • teknoid

Poem

🐰 Versions hop through branches neat,
Six to twelve, the kernels meet,
Patches dance from old to new,
USB wakes where gadgets flew!
PiKVM's drive takes center stage,
A rabbit's delight in this kernel age!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: bumping kernel versions (legacy, current, edge) to new major versions for rpi4b, which aligns with the primary objective and changes in config/sources/families/bcm2711.conf.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ 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/large PR with 250 lines or more 02 Milestone: First quarter release Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... labels Dec 17, 2025
Copy link
Member

@igorpecovnik igorpecovnik left a comment

Choose a reason for hiding this comment

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

I think here we can go straight to 6.18 right away.

@github-actions github-actions bot added the Ready to merge Reviewed, tested and ready for merge label Dec 17, 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 Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

02 Milestone: First quarter release Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/large PR with 250 lines or more

Development

Successfully merging this pull request may close these issues.

2 participants