Skip to contentSkip to conversation

18xx-ti-utils: new package

PATCH New Baruch Siach 2 participants 4 messages Mar 1, 2018

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 18xx-ti-utils: new package

These are spacial RF calibration utilities for TI Wilink 18xx modules.

---
v3:
* Propagate dependencies (Marcus)
* Add license information and hash

v2:
* Fix the config symbol as noted by ThomasP
* Use STAGING_DIR for NFSROOT

---
 DEVELOPERS                               |  1 +
 package/18xx-ti-utils/18xx-ti-utils.hash |  3 +++
 package/18xx-ti-utils/18xx-ti-utils.mk   | 30 ++++++++++++++++++++++++++++++
 package/18xx-ti-utils/Config.in          | 14 ++++++++++++++
 package/Config.in                        |  1 +
 5 files changed, 49 insertions(+)
 create mode 100644 package/18xx-ti-utils/18xx-ti-utils.hash
 create mode 100644 package/18xx-ti-utils/18xx-ti-utils.mk
 create mode 100644 package/18xx-ti-utils/Config.in
DEVELOPERS +1 -0
index e2b390142595..77b6df3cd37d 100644
@@ -205,6 +205,7 @@ F:	package/pulseview/
 F:	package/sigrok-cli/
 
 N:	Baruch Siach <[email protected]>
+F:	package/18xx-ti-utils/
 F:	package/daemon/
 F:	package/dropbear/
 F:	package/ebtables/
package/18xx-ti-utils/18xx-ti-utils.hash new file +3 -0
new file mode 100644
index 000000000000..faa23d0b3edc
@@ -0,0 +1,3 @@
+# Locally calculated
+sha256 9ce2c7228dd0ac8d3cc530a32bb92b532bdbc5dfb6a405c245abee678d878697  18xx-ti-utils-R8.7_SP3.tar.gz
+sha256 363cef40b276a7a91ebcff3f78626d9cfe5e5a51927c725236007259403d5fc4  COPYING
package/18xx-ti-utils/18xx-ti-utils.mk new file +30 -0
new file mode 100644
index 000000000000..d1f0388c3a87
@@ -0,0 +1,30 @@
+################################################################################
+#
+# 18xx-ti-utils
+#
+################################################################################
+
+18XX_TI_UTILS_VERSION = R8.7_SP3
+18XX_TI_UTILS_SITE = git://git.ti.com/wilink8-wlan/18xx-ti-utils
+18XX_TI_UTILS_DEPENDENCIES = libnl
+18XX_TI_UTILS_LICENSE = BSD-3-Clause
+18XX_TI_UTILS_LICENSE_FILES = COPYING
+
+18XX_TI_UTILS_CFLAGS = -I$(STAGING_DIR)/usr/include/libnl3 -DCONFIG_LIBNL32
+
+ifeq ($(BR2_STATIC_LIBS),y)
+18XX_TI_UTILS_BUILD_TARGET = static
+endif
+
+define 18XX_TI_UTILS_BUILD_CMDS
+	$(TARGET_MAKE_ENV) CROSS_COMPILE=$(TARGET_CROSS) \
+		NFSROOT="$(STAGING_DIR)" NLVER=3 $(MAKE) -C $(@D) \
+		CFLAGS="$(TARGET_CFLAGS) $(18XX_TI_UTILS_CFLAGS)" \
+		$(18XX_TI_UTILS_BUILD_TARGET)
+endef
+
+define 18XX_TI_UTILS_INSTALL_TARGET_CMDS
+	$(INSTALL) -m 0755 $(@D)/calibrator $(TARGET_DIR)/usr/bin/calibrator
+endef
+
+$(eval $(generic-package))
package/18xx-ti-utils/Config.in new file +14 -0
new file mode 100644
index 000000000000..571d196e091c
@@ -0,0 +1,14 @@
+config BR2_PACKAGE_18XX_TI_UTILS
+	bool "18xx-ti-utils"
+	depends on BR2_TOOLCHAIN_HAS_THREADS # libnl
+	select BR2_PACKAGE_LIBNL
+	help
+	  TI Wilink calibration utilities.
+
+	  CONFIG_NL80211_TESTMODE must be enabled in the kernel
+	  configuration for this package to work.
+
+	  http://processors.wiki.ti.com/index.php/WL18xx_TX_Testing
+
+comment "18xx-ti-utils needs a toolchain w/ threads"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS
package/Config.in +1 -0
index 97317fdb5ff0..12d3d2bbff11 100644
@@ -375,6 +375,7 @@ menu "Firmware"
 	source "package/wilink-bt-firmware/Config.in"
 	source "package/zd1211-firmware/Config.in"
 endmenu
+	source "package/18xx-ti-utils/Config.in"
 	source "package/a10disp/Config.in"
 	source "package/acpica/Config.in"
 	source "package/acpid/Config.in"
Signature · 1 line
2.16.1
#1 lore raw

PATCH 1/1 18xx-ti-utils: new package

Quoted 10 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Baruch" == Baruch Siach <[email protected]> writes:

These are spacial RF calibration utilities for TI Wilink 18xx modules.

---
v3:
* Propagate dependencies (Marcus)
* Add license information and hash

package/18xx-ti-utils/Config.in line 9

Quoted 5 lines
package/18xx-ti-utils/Config.in new file +5 -0
+	help
+	  TI Wilink calibration utilities.
+
+	  CONFIG_NL80211_TESTMODE must be enabled in the kernel
+	  configuration for this package to work.

Would it make sense to force this option on in linux.mk like we do for other packages?

Committed to next, thanks.

Signature · 1 line
Bye, Peter Korsgaard
#2 lore raw

PATCH 1/1 18xx-ti-utils: new package

Hi Peter,

Quoted 5 lines · Peter Korsgaard
Quoted 4 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Baruch" == Baruch Siach <[email protected]> writes:

These are spacial RF calibration utilities for TI Wilink 18xx modules.

Quoted 13 lines

package/18xx-ti-utils/Config.in line 9

Quoted 10 lines

---
v3:
* Propagate dependencies (Marcus)
* Add license information and hash

package/18xx-ti-utils/Config.in new file +5 -0
+	help
+	  TI Wilink calibration utilities.
+
+	  CONFIG_NL80211_TESTMODE must be enabled in the kernel
+	  configuration for this package to work.

Would it make sense to force this option on in linux.mk like we do for other packages?

I don't think so. I don't like this magic config manipulation in the general case. CONFIG_NL80211_TESTMODE in particular is meant for those who really know what they are doing. As the kernel config option help text put it:

Say N.

Thanks,
baruch

Signature · 3 lines
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
#3 lore raw

PATCH 1/1 18xx-ti-utils: new package

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Baruch" == Baruch Siach <[email protected]> writes:

Hi,

Quoted 8 lines
Quoted 2 lines

Would it make sense to force this option on in linux.mk like we do for other packages?

I don't think so. I don't like this magic config manipulation in the general case. CONFIG_NL80211_TESTMODE in particular is meant for those who really know what they are doing. As the kernel config option help text put it:

Say N.

OK. I agree that we should keep the "magic" kernel config tweaks to a minimum.

Signature · 1 line
Bye, Peter Korsgaard