| Summary: | Missing overlays directory in VFAT image for raspberry pi 3 | ||
|---|---|---|---|
| Product: | buildroot | Reporter: | diez.roggisch |
| Component: | Other | Assignee: | unassigned |
| Status: | CLOSED INVALID | ||
| Severity: | normal | CC: | buildroot, diez.roggisch, yann.morin.1998 |
| Priority: | P5 | ||
| Version: | 2016.05 | ||
| Target Milestone: | --- | ||
| Hardware: | Other | ||
| OS: | Linux | ||
| Host: | Target: | ||
| Build: | |||
I found that BR2_PACKAGE_RPI_FIRMWARE_INSTALL_DTB_OVERLAYS actually does the job - but only after a clean rebuild. I will close. Diez,
If you found that your bug was in fact not resolved and still valid, you
should have reopenned it rather than sending an email to an individual.
So, you said:
> BR2_PACKAGE_RPI_FIRMWARE_INSTALL_DTB_OVERLAYS=y does copy the overlays
> into output/images/rpi-firmware/overlays which is all fine and dandy -
> but does *not* include them then in the actual FAT boot image, which is
> controlled by: buildroot/board/raspberrypi/genimage-raspberrypi3.cfg
Indeed, that is not the case.
The defconfig files we provide in Buildroot are "minimalist" configs,
aimed at building the strict minimum to get a board to actually boot up
to a shell prompt, with wired networkingv (no wireless). No more.
This is because there are so many possibilities to use a board that we
will definitely never be able to cover all use cases. At least, a minimal
defconfig gets the board booting and ready to use in its basic setup. It
is a known-working starting point for further customisation by the user.
In your casse, edit (or copy then edit) the genimage-raspberrypi3.cfg to
suit your needs.
I'll keep this report closed, however, because I believe the behaviour
you reported is what is intended.
Feel free to comment, though. ;-)
Regards,
Yann E. MORIN.
First, my apologies for violating the unknown protocol. Won't happen again. I checked, and matter of factly the patch would break buildroot if BR2_PACKAGE_RPI_FIRMWARE_INSTALL_DTB_OVERLAYS was not selected. So it's moot anyway. I still would like a solution that takes care of this semi-automatically, as it's simply surprising for a developer IMHO. But I will argue for that once (and if) I find one. Diez, > I checked, and matter of factly the patch would break buildroot if > BR2_PACKAGE_RPI_FIRMWARE_INSTALL_DTB_OVERLAYS was not selected. So > it's moot anyway. I was not suggesting that you submit a patch to Buildroot, but that you handle the customisation on your side. Use your own genimage.cfg file that suits your need. The manual has a section on how to handle local customisations: https://buildroot.org/downloads/manual/manual.html#customize One would argue that overlays are very often used so they should be installed by our defconfigs. However, there are so many overlays, for so different things, that usually only one or a few would be needed in a specific case. Or even that overlays from anotehr place are used. So I am still not sure we want to have our defconfigs install the overlays. Regards, Yann E. MORIN. |
When running a raspberry pi 3 buildroot, dtoverlay-parameters (e.g. dtoverlay=pi3-disable-bt) don't work, because the VFAT image doesn't contain the overlays-subdirectory. The following patch fixes this. diff --git a/board/raspberrypi/genimage-raspberrypi3.cfg b/board/raspberrypi/genimage-raspberrypi3.cfg index 7ba89c5..baab0c4 100644 --- a/board/raspberrypi/genimage-raspberrypi3.cfg +++ b/board/raspberrypi/genimage-raspberrypi3.cfg @@ -7,6 +7,7 @@ image boot.vfat { "rpi-firmware/config.txt", "rpi-firmware/fixup.dat", "rpi-firmware/start.elf", + "rpi-firmware/overlays", "kernel-marked/zImage" } }