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

[Ioan-Adrian Ratiu <adrian.ratiu@ni.com>]

Hello

On Fri, 1 Apr 2016 13:36:08 +0200
Martin Jansa <martin.jansa@gmail.com> wrote:

> 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
> 

Since this is already in master, I'll submit another patch to fix this and use $CC properly.

Thank you,
Ionel

> > +                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
> >   
>