[Buildroot] [PATCH 4/4] package/rpi-firmware: improve installation of dtb files

Martin Bark martin at barkynet.com
Thu Mar 15 19:58:35 UTC 2018


Hi Peter,

On 15 March 2018 at 19:40, Peter Seiderer <ps.report at gmx.net> wrote:
> Hello Martin,
>
> On Thu, 15 Mar 2018 13:24:51 +0000, Martin Bark <martin at barkynet.com> wrote:
>
>> Don't list specific dtb files, instead install all dtb files in the same
>> way as dtbo files are currently done.
>>
>> Signed-off-by: Martin Bark <martin at barkynet.com>
>> ---
>>  package/rpi-firmware/rpi-firmware.mk | 7 +++----
>>  1 file changed, 3 insertions(+), 4 deletions(-)
>>
>> diff --git a/package/rpi-firmware/rpi-firmware.mk b/package/rpi-firmware/rpi-firmware.mk
>> index 1ebdd96..9cc3d4a 100644
>> --- a/package/rpi-firmware/rpi-firmware.mk
>> +++ b/package/rpi-firmware/rpi-firmware.mk
>> @@ -12,10 +12,9 @@ RPI_FIRMWARE_INSTALL_IMAGES = YES
>>
>>  ifeq ($(BR2_PACKAGE_RPI_FIRMWARE_INSTALL_DTBS),y)
>>  define RPI_FIRMWARE_INSTALL_DTB
>> -     $(INSTALL) -D -m 0644 $(@D)/boot/bcm2708-rpi-b.dtb $(BINARIES_DIR)/rpi-firmware/bcm2708-rpi-b.dtb
>> -     $(INSTALL) -D -m 0644 $(@D)/boot/bcm2708-rpi-b-plus.dtb $(BINARIES_DIR)/rpi-firmware/bcm2708-rpi-b-plus.dtb
>> -     $(INSTALL) -D -m 0644 $(@D)/boot/bcm2709-rpi-2-b.dtb $(BINARIES_DIR)/rpi-firmware/bcm2709-rpi-2-b.dtb
>> -     $(INSTALL) -D -m 0644 $(@D)/boot/bcm2710-rpi-3-b.dtb $(BINARIES_DIR)/rpi-firmware/bcm2710-rpi-3-b.dtb
>> +     for dtb in  $(@D)/boot/*.dtb; do \
>> +             $(INSTALL) -D -m 0644 $${dtb} $(BINARIES_DIR)/rpi-firmware/$${dtb##*/} || exit 1; \
>> +     done
>
> Which will install three additional dtb files (bcm2708-rpi-0-w.dtb,
> bcm2708-rpi-cm.dtb and bcm2710-rpi-cm3.dtb), not a downside of the
> patch, but worth mentioning....

Yes, it will also install the new bcm2710-rpi-3-b-plus.dtb. This code
does not seem to have been kept updated, it's missing several dtb
files as you mention.  I think it's better to just install all dtb
files as we do for the dtbo files.

Thanks

Martin

>
> Regards,
> Peter
>
>>  endef
>>  endif
>>
>


More information about the buildroot mailing list