Re: [meta-oe][PATCH] ti-wifi-utils: add recipe

[Martin Jansa <martin.jansa@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2834 bytes --]

On Tue, Mar 22, 2016 at 11:57:16AM +0200, Ioan-Adrian Ratiu wrote:
> This recipe is for a TI WiFi calibrator utility [1]. The firmware from
> this project is already shipped inside linux-firmware so we do not
> package it in this recipe.
> 
> [1] http://linuxwireless.sipsolutions.net/en/users/Drivers/wl12xx/#Calibration
> 
> Signed-off-by: James Minor <james.minor@ni.com>
> Signed-off-by: Ioan-Adrian Ratiu <adrian.ratiu@ni.com>
> ---
>  .../ti-wifi-utils/ti-wifi-utils_git.bb             | 24 ++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
>  create mode 100644 meta-oe/recipes-connectivity/ti-wifi-utils/ti-wifi-utils_git.bb
> 
> diff --git a/meta-oe/recipes-connectivity/ti-wifi-utils/ti-wifi-utils_git.bb b/meta-oe/recipes-connectivity/ti-wifi-utils/ti-wifi-utils_git.bb
> new file mode 100644
> index 0000000..fcae729
> --- /dev/null
> +++ b/meta-oe/recipes-connectivity/ti-wifi-utils/ti-wifi-utils_git.bb
> @@ -0,0 +1,24 @@
> +SUMMARY = "The calibrator and other useful utilities for TI wireless solution based on wl12xx driver"
> +LICENSE = "BSD-3-Clause"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=4725015cb0be7be389cf06deeae3683d"
> +
> +DEPENDS = "libnl"
> +
> +PV = "0.1+gitr${SRCPV}"
> +
> +SRCREV = "b03d9ce6362e6d22d6929f2736409af3b0fd3c88"
> +SRC_URI = "git://github.com/TI-OpenLink/ti-utils.git;branch=r5-jb"
> +
> +S = "${WORKDIR}/git"
> +
> +EXTRA_OEMAKE = 'CROSS_COMPILE=${TARGET_PREFIX} \
> +                CC="${TARGET_PREFIX}gcc ${TOOLCHAIN_OPTIONS}" \

This doesn't respect all flags usually available in $CC variable, why
haven't you used $CC?

Maybe it's other part of Makefiles which is using CROSS_COMPILE instead
of CC, but currently it's failing e.g. for hardfloat builds with
typical:

../SYSROOT/MACHINE/usr/include/gnu/stubs.h:7:29: fatal error: gnu/stubs-soft.h: No such file or directory

because the build doesn't have correct -mfloat-abi parameter

| arm-DISTRO-linux-gnueabi-gcc  --sysroot=/SYSROOTS/MACHINE -O2 -pipe -g -feliminate-unused-debug-types -fdebug-prefix-map=TMPDIRwork/MACHINE-DISTRO-linux-gnueabi/ti-wifi-utils/0.1+gitrAUTOINC+b03d9ce636-r0=/usr/src/debug/ti-wifi-utils/0.1+gitrAUTOINC+b03d9ce636-r0 -fdebug-prefix-map=TMPDIRsysroots/x86_64-linux= -fdebug-prefix-map=/SYSROOTS/MACHINE=  -I/SYSROOTS/MACHINE/usr/include/libnl3 -DCONFIG_LIBNL20   -c -o misc_cmds.o misc_cmds.c

> +                CFLAGS="${CFLAGS} -I${STAGING_INCDIR}/libnl3 -DCONFIG_LIBNL20" NLVER=3 \
> +                LDFLAGS="${LDFLAGS} ${TOOLCHAIN_OPTIONS}" \
> +'
> +
> +#only install the calibrator utility, firmware is already within linux-firmware
> +do_install() {
> +    install -d ${D}${bindir}
> +    install -m 0755 calibrator ${D}${bindir}/
> +}
> -- 
> 2.7.4
> 

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]