rpi4b: fix build and boot issues#9267
Conversation
📝 WalkthroughWalkthroughUpdates to BCM2711 kernel configuration files adding many ARM64 erratum mitigations, expanded VirtIO/SCSI/VIRTIO_NET support, broader DRM/display and I2C/SPI/peripheral drivers, network/NF_TABLES changes, audio/thermal/RTC additions, and various power/regulator and crypto option expansions across current and legacy configs. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@config/kernel/linux-bcm2711-legacy.config`:
- Around line 469-471: Add the missing CONFIG_VIRTIO_VSOCKETS option to the
legacy bcm2711 kernel config so vsockets have the virtio transport;
specifically, insert CONFIG_VIRTIO_VSOCKETS=m alongside the existing
CONFIG_VSOCKETS=m / CONFIG_VMWARE_VMCI_VSOCKETS=m / CONFIG_NETLINK_DIAG=m block
(and apply the same insertion at the other occurrence later in the file) so both
vsocket sections include the virtio transport option.
🧹 Nitpick comments (2)
config/kernel/linux-bcm2711-current.config (1)
553-553: RP1/BCM2712 options are included from Raspberry Pi's upstream kernel—verify whether to keep them aligned with upstream or trim for Pi 4 only.The kernel config includes RP1/BCM2712-specific options (
CONFIG_MISC_RP1=y,CONFIG_PINCTRL_BCM2712=y,CONFIG_SENSORS_RP1_ADC=m,CONFIG_MFD_RP1=y,CONFIG_VIDEO_RP1_CFE=m,CONFIG_DRM_RP1_DSI=m,CONFIG_DRM_RP1_DPI=m,CONFIG_DRM_RP1_VEC=m,CONFIG_COMMON_CLK_RP1_SDIO=m,CONFIG_BCM2712_IOMMU=y,CONFIG_PWM_RP1=m,CONFIG_BCM2712_MIP=y). These are sourced from Raspberry Pi's official kernel branches (rpi-6.18.y for current), which support multiple board versions in a single kernel. Since Armbian currently supports only Pi 4, these options compile unused code. However, keeping them aligns with Raspberry Pi's upstream approach. Consider whether the maintenance benefit of staying synchronized with upstream outweighs the code bloat.config/kernel/linux-bcm2711-legacy.config (1)
2419-2438: Confirm debug-heavy options are intended for release configs.BOOT_PRINTK_DELAY, KGDB/KDB, LATENCYTOP, DETECT_HUNG_TASK, and relaxed STRICT_DEVMEM can impact boot time and security posture. If these were only for diagnosing boot issues, consider moving them to a debug flavor.
igorpecovnik
left a comment
There was a problem hiding this comment.
perhaps also fix that Rabbit's suggestion for legacy?
|
✅ This PR has been reviewed and approved — all set for merge! |
* rpi4b: rename kernel configs * rewrite kernel configs
Description
Follow-up of #9097
It was reported that legacy doesn't build and current doesn't boot. This should fix both.
kernel config rewrite pending to get back in sync
How Has This Been Tested?
Checklist:
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.