[Buildroot] [PATCH v2 2/6] ux500-firmware: new package

[Arnout Vandecappelle <arnout@mind.be>]

On 11/12/12 18:41, Gregory Hermant wrote:
>
> Signed-off-by: Gregory Hermant<gregory.hermant@calao-systems.com>
> ---
>   package/Config.in                        |    1 +
>   package/ux500-firmware/Config.in         |    7 +++++++
>   package/ux500-firmware/ux500-firmware.mk |   20 ++++++++++++++++++++
>   3 files changed, 28 insertions(+)
>   create mode 100644 package/ux500-firmware/Config.in
>   create mode 100644 package/ux500-firmware/ux500-firmware.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index cb78d2f..6da6e73 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -261,6 +261,7 @@ source "package/usb_modeswitch/Config.in"
>   source "package/usb_modeswitch_data/Config.in"
>   source "package/usbmount/Config.in"
>   source "package/usbutils/Config.in"
> +source "package/ux500-firmware/Config.in"
>   source "package/wipe/Config.in"
>   endmenu
>
> diff --git a/package/ux500-firmware/Config.in b/package/ux500-firmware/Config.in
> new file mode 100644
> index 0000000..7e8a94b
> --- /dev/null
> +++ b/package/ux500-firmware/Config.in
> @@ -0,0 +1,7 @@
> +config BR2_PACKAGE_UX500_FIRMWARE
> +	bool "ux500-firmware"
> +	help
> +	  This package provides various binary firmware files (closed binary blobs)
> +	  for the Azurewave AW-NH580 combo module (wifi, bt, gps).
> +
> +	  https://launchpad.net/~igloocommunity-maintainers/+archive/snowball/+files/
> diff --git a/package/ux500-firmware/ux500-firmware.mk b/package/ux500-firmware/ux500-firmware.mk
> new file mode 100644
> index 0000000..522113a
> --- /dev/null
> +++ b/package/ux500-firmware/ux500-firmware.mk
> @@ -0,0 +1,20 @@
> +#############################################################
> +#
> +# ux500-firmware
> +#
> +#############################################################
> +UX500_FIRMWARE_VERSION = 1.1.3-6
> +UX500_FIRMWARE_SOURCE = ux500-firmware_$(UX500_FIRMWARE_VERSION)linaro1.tar.gz
> +UX500_FIRMWARE_SITE = https://launchpad.net/~igloocommunity-maintainers/+archive/snowball/+files/
> +UX500_FIRMWARE_LICENSE = Snowball click-wrap license

  UX500_FIRMWARE_LICENSE_FILES = license.txt
> +UX500_FIRMWARE_REDISTRIBUTE = NO
> +
> +define UX500_FIRMWARE_INSTALL_TARGET_CMDS
> +	mkdir -p $(TARGET_DIR)/lib/firmware
> +	cp -r $(@D)/* $(TARGET_DIR)/lib/firmware/

  This also installs the debian directory etc. to the target.  Why not use
the Makefile that is delivered with the package?

> +	for f in $(TARGET_DIR)/lib/firmware/CG2900* ; do \
> +		mv $$f $${f/CG2900/CG29XX}; \
> +	done

  This warrants a comment why it is needed to rename some files.

> +endef
> +
> +$(eval $(generic-package))


  Regards,
  Arnout
-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F