[Buildroot] [PATCH v3] New package: linux-firmware

[Peter Korsgaard <jacmet@sunsite.dk>]

>>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:

Hi,

 Gustavo> These constructs can get too long too fast, why not something
 Gustavo> like...

 Gustavo> ifeq ($(BR2_PACKAGE_LINUX_FIRMWARE_RALINK_RT2XXX),y)
 Gustavo>     LINUX_FIRMWARE_FILES += rt2860.bin rt2870.bin rt30*.bin
 Gustavo> LICENCE.ralink-firmware.txt
 Gustavo> endif

 Gustavo> define LINUX_FIRMWARE_INSTALL_TARGET_CMDS
 Gustavo>     cp -af $(addprefix $(@D)/, $(LINUX_FIRMWARE_FILES))
 Gustavo> $(TARGET_DIR)/lib/firmware
 Gustavo> endef

Agreed, except you should use a loop and install -D, so subdirs are
supported.

E.G.

        for i in $(LINUX_FIRMWARE_FILES); do \
          $i(INSTALL) -m 0644 -D $(@D)/$$i $(TARGET_DIR)/lib/firmware/$$i; \
        done

-- 
Bye, Peter Korsgaard