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

[Ioan-Adrian Ratiu]

Hello

On Fri, 1 Apr 2016 15:53:41 -0400
Denys Dmytriyenko <denis at denix.org> wrote:

> On Fri, Apr 01, 2016 at 01:36:08PM +0200, Martin Jansa 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 at ni.com>
> > > Signed-off-by: Ioan-Adrian Ratiu <adrian.ratiu at 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}" \  
> 
> Reinventing the wheel much?
> 
> http://layers.openembedded.org/layerindex/branch/master/recipes/?q=ti-wifi-utils
> 

Yes, it is a duplicate. How about keeping the recipe in meta-oe to make it usable without adding arago distributions layers?

Thank you,
Ioan-Adrian

> 
> > 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 at gmail.com  
> 
> 
> 
> > -- 
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel at lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel  
>