Enable BCACHE on Raspberry Pi build#9550
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdded the kernel configuration symbol CONFIG_BCACHE set to module ("m") in two BCM2711 kernel config files, enabling BCACHE as a loadable kernel module in those defconfigs. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 Tip You can customize the high-level summary generated by CodeRabbit.Configure the |
Hey @aulanov! 👋Thanks for submitting your first pull request to the Armbian project — we're excited to have you contributing! 🧡 If you'd like to stay informed about project updates or collaborate more closely with the team, Also, don’t forget to ⭐ star the repo if you haven’t already — and welcome aboard! 🚀 |
|
Can you check against edge and legacy config too? |
Legacy already has this enabled. I've now updated edge config as well. |
igorpecovnik
left a comment
There was a problem hiding this comment.
We can enable that, no problem.
Documentation summary for feature / change
This PR enables CONFIG_BCACHE on bcm2711 builds.
CONFIG_BCACHE is enabled on most other boards. There are no reasons it should be enabled on other boards but not Raspberries.
How Has This Been Tested?
I've tested this by compiling the kernel and making sure bcache works on Raspberry Pi 5.
Checklist:
Please delete options that are not relevant.
Summary by CodeRabbit