Patchwork [v3,2/6] ux500-firmware: new package

login
register
mail settings
Submitter Gregory Hermant
Date Nov. 15, 2012, 8:25 a.m.
Message ID <1352967920-3164-3-git-send-email-gregory.hermant@calao-systems.com>
Download mbox | patch
Permalink /patch/199223/
State Superseded
Headers show

Comments

Gregory Hermant - Nov. 15, 2012, 8:25 a.m.
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 |   22 ++++++++++++++++++++++
 3 files changed, 30 insertions(+)
 create mode 100644 package/ux500-firmware/Config.in
 create mode 100644 package/ux500-firmware/ux500-firmware.mk
Arnout Vandecappelle - Nov. 21, 2012, 6:54 a.m.
On 15/11/12 09:25, Gregory Hermant wrote:
>
> Signed-off-by: Gregory Hermant<gregory.hermant@calao-systems.com>

Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

  Minor detail below.

[snip]

> +define UX500_FIRMWARE_INSTALL_TARGET_CMDS
> +	$(MAKE) -C $(@D) DESTDIR=$(TARGET_DIR) install
> +# The CG2900 linux driver has to load firmware named CG29XX_* but the firmware
> +# filenames contained in this package are CG2900_* hence the code below

  Comments inside the _CMDS macros don't work well: they are interpreted by
the shell.  It's better to put them before the macro definition.


  Regards,
  Arnout

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

Patch

diff --git a/package/Config.in b/package/Config.in
index e9fd4a9..4fa21ca 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..1fcb3b7
--- /dev/null
+++ b/package/ux500-firmware/ux500-firmware.mk
@@ -0,0 +1,22 @@ 
+#############################################################
+#
+# 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
+	$(MAKE) -C $(@D) DESTDIR=$(TARGET_DIR) install
+# The CG2900 linux driver has to load firmware named CG29XX_* but the firmware
+# filenames contained in this package are CG2900_* hence the code below
+	for f in $(TARGET_DIR)/lib/firmware/CG2900* ; do \
+		mv $$f $${f/CG2900/CG29XX}; \
+	done
+endef
+
+$(eval $(generic-package))