[Buildroot] [PATCH v2] wf111: new package

[Buildroot] [PATCH v2] wf111: new package

[Antoine Tenart]

Adds support for the BlueGiga WF111 WiFi driver and the binary utilities
distributed alongside the driver. An account is required to download the
sources from the BlueGiga website, which can be created freely. The
driver is available for armv5, arm7a and i386.

Since it is not possible to automatically retrieve the sources, because
of the required user account needed on the BlueGiga website, an option
is added to let the Buildroot user specify the directory where the
driver tarball was downloaded.

Finally, two options must be selected in the Linux kernel configuration:
CONFIG_WIRELESS_EXT and CONFIG_WEXT_PRIV. These are blind options (i.e.
not selectable directly) so they cannot be enabled by a change in
linux/linux.mk. The user as two choices to enable these options:
- By making them non blind, with a "WF111 support" configuration entry
  for example.
- By enabling another WiFi driver that select them.

The work behind this commit was funded by ECA Group
<http://www.ecagroup.com>. ECA Group is the copyright owner of the
contributed code.

Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>
---

Changes since v1:
	- fixed various typos
	- added explanations about needed (blind) kernel options
	- added a comment in the menu when wf111 requirements are not met
	- used qstrip for string options
	- updated to use $(LINUX_MAKE_FLAGS)
	- used cp instead of rsync
	- added a check to be sure the configuration is correctly filled

 package/Config.in       |  1 +
 package/wf111/Config.in | 36 ++++++++++++++++++++++++++++++++++++
 package/wf111/wf111.mk  | 34 ++++++++++++++++++++++++++++++++++
 3 files changed, 71 insertions(+)
 create mode 100644 package/wf111/Config.in
 create mode 100644 package/wf111/wf111.mk

diff --git a/package/Config.in b/package/Config.in
index d19b37f703db..e631fcb78f81 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -390,6 +390,7 @@ endif
 	source "package/usbmount/Config.in"
 	source "package/usbutils/Config.in"
 	source "package/w_scan/Config.in"
+	source "package/wf111/Config.in"
 	source "package/wipe/Config.in"
 	source "package/xorriso/Config.in"
 endmenu
diff --git a/package/wf111/Config.in b/package/wf111/Config.in
new file mode 100644
index 000000000000..d2ba440a90a0
--- /dev/null
+++ b/package/wf111/Config.in
@@ -0,0 +1,36 @@
+config BR2_PACKAGE_WF111
+	bool "wf111"
+	depends on BR2_LINUX_KERNEL
+	depends on BR2_ARM_CPU_ARMV5 || BR2_ARM_CPU_ARMV7A || BR2_i386
+	# Binary tools are distributed alongside the driver, and are
+	# dynamically linked against the glibc.
+	depends on BR2_TOOLCHAIN_USES_GLIBC
+	help
+	  BlueGiga WF111 WiFi driver and utilities.
+
+	  Warning: CONFIG_WIRELESS_EXT and CONFIG_WEXT_PRIV must be
+	  selected in the Linux kernel configuration. These are blind
+	  options (i.e. not selectable directly) so they cannot be
+	  enabled by a change in linux/linux.mk. There are two choices
+	  to enable these options:
+	  - By making them non blind, with a "WF111 support" configuration
+	    entry that selects them, for example.
+	  - By enabling another random WiFi driver that select them.
+
+	  http://www.bluegiga.com/en-US/products/wifi-modules/wf111-wifi-module/
+
+if BR2_PACKAGE_WF111
+
+config BR2_PACKAGE_WF111_TARBALL_PATH
+	string "Local tarball location"
+	help
+	  The WF111 tarball can be retrieved on the BlueGiga website
+	  after registration. This option specifies the path where the
+	  tarball is locally saved.
+
+endif
+
+comment "wf111 needs an (e)glibc toolchain"
+	depends on BR2_LINUX_KERNEL
+	depends on BR2_ARM_CPU_ARMV5 || BR2_ARM_CPU_ARMV7A || BR2_i386
+	depends on !BR2_TOOLCHAIN_USES_GLIBC
diff --git a/package/wf111/wf111.mk b/package/wf111/wf111.mk
new file mode 100644
index 000000000000..479d665760b1
--- /dev/null
+++ b/package/wf111/wf111.mk
@@ -0,0 +1,34 @@
+################################################################################
+#
+# wf111
+#
+################################################################################
+
+WF111_VERSION = 5.2.2
+WF111_SITE_METHOD = file
+WF111_SITE = $(call qstrip,$(BR2_PACKAGE_WF111_TARBALL_PATH))
+WF111_DEPENDENCIES = linux
+
+ifeq ($(BR2_PACKAGE_WF111)$(call qstrip,$(BR2_PACKAGE_WF111_TARBALL_PATH)),y)
+$(error No tarball location specified, check BR2_PACKAGE_WF111_TARBALL_PATH)
+endif
+
+ifeq ($(BR2_ARM_CPU_ARMV7A),y)
+WF111_SOURCE = wf111-linux-driver_5.2.2-r1_armv7-a.tar.gz
+else ifeq ($(BR2_ARM_CPU_ARMV5),y)
+WF111_SOURCE = wf111-linux-driver_5.2.2-r1_armv5t.tar.gz
+else ifeq ($(BR2_i386),y)
+WF111_SOURCE = wf111-linux-driver_5.2.2-r1_x86.tar.gz
+endif
+
+define WF111_BUILD_CMDS
+	$(MAKE) -C $(@D) PWD=$(@D) \
+		$(LINUX_MAKE_FLAGS) KDIR=$(LINUX_DIR) \
+		install_static
+endef
+
+define WF111_INSTALL_TARGET_CMDS
+	cp -dpfr $(@D)/output/* $(TARGET_DIR)
+endef
+
+$(eval $(generic-package))
-- 
2.2.2

[Buildroot] [PATCH v2] wf111: new package

[Thomas Petazzoni]

Dear Antoine Tenart,

On Tue,  3 Feb 2015 15:33:25 +0100, Antoine Tenart wrote:
> Adds support for the BlueGiga WF111 WiFi driver and the binary utilities
> distributed alongside the driver. An account is required to download the
> sources from the BlueGiga website, which can be created freely. The
> driver is available for armv5, arm7a and i386.
> 
> Since it is not possible to automatically retrieve the sources, because
> of the required user account needed on the BlueGiga website, an option
> is added to let the Buildroot user specify the directory where the
> driver tarball was downloaded.
> 
> Finally, two options must be selected in the Linux kernel configuration:
> CONFIG_WIRELESS_EXT and CONFIG_WEXT_PRIV. These are blind options (i.e.
> not selectable directly) so they cannot be enabled by a change in
> linux/linux.mk. The user as two choices to enable these options:
> - By making them non blind, with a "WF111 support" configuration entry
>   for example.
> - By enabling another WiFi driver that select them.
> 
> The work behind this commit was funded by ECA Group
> <http://www.ecagroup.com>. ECA Group is the copyright owner of the
> contributed code.
> 
> Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>

Thanks, applied to next. It would be good however to send a followup
patch adding <pkg>_LICENSE and <pkg>_LICENSE_FILES.

Best regards,

Thomas Petazzoni
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH v2] wf111: new package

[Antoine Tenart]

Thomas,

On Fri, Feb 20, 2015 at 10:07:05AM +0100, Thomas Petazzoni wrote:
> On Tue,  3 Feb 2015 15:33:25 +0100, Antoine Tenart wrote:
> > Adds support for the BlueGiga WF111 WiFi driver and the binary utilities
> > distributed alongside the driver. An account is required to download the
> > sources from the BlueGiga website, which can be created freely. The
> > driver is available for armv5, arm7a and i386.
> > 
> > Since it is not possible to automatically retrieve the sources, because
> > of the required user account needed on the BlueGiga website, an option
> > is added to let the Buildroot user specify the directory where the
> > driver tarball was downloaded.
> > 
> > Finally, two options must be selected in the Linux kernel configuration:
> > CONFIG_WIRELESS_EXT and CONFIG_WEXT_PRIV. These are blind options (i.e.
> > not selectable directly) so they cannot be enabled by a change in
> > linux/linux.mk. The user as two choices to enable these options:
> > - By making them non blind, with a "WF111 support" configuration entry
> >   for example.
> > - By enabling another WiFi driver that select them.
> > 
> > The work behind this commit was funded by ECA Group
> > <http://www.ecagroup.com>. ECA Group is the copyright owner of the
> > contributed code.
> > 
> > Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>
> 
> Thanks, applied to next. It would be good however to send a followup
> patch adding <pkg>_LICENSE and <pkg>_LICENSE_FILES.

I'd like to be able to provide a license, but all I can find is a
Copyright at the beginning of the source files, with the following
statement:

 * Refer to LICENSE.txt included with this source code for details on
 * the license terms.

And... the LICENSE.txt file can't be found :)

Antoine

-- 
Antoine T?nart, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com