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

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

[Ioan-Adrian Ratiu]

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}" \
+                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

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

[Martin Jansa]

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

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

[Denys Dmytriyenko]

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@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}" \

Reinventing the wheel much?

http://layers.openembedded.org/layerindex/branch/master/recipes/?q=ti-wifi-utils


> 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



> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

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

[Ioan-Adrian Ratiu]

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

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

[Ioan-Adrian Ratiu]

Hello

On Fri, 1 Apr 2016 15:53:41 -0400
Denys Dmytriyenko <denis@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@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}" \  
> 
> 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@gmail.com  
> 
> 
> 
> > -- 
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel  
> 

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

[Ioan-Adrian Ratiu]

On Mon, 11 Apr 2016 14:11:08 -0400
Denys Dmytriyenko <denis@denix.org> wrote:

> Resending again, since my previous reply got truncated when sending to the 
> list:
> 
> On Thu, Apr 07, 2016 at 11:50:10AM -0700, Denys Dmytriyenko wrote:
> > > 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?  
> > 
> > Well, it is currently NOT in the distribution layer, but rather in the
> > application layer... But I guess I'm fine submitting the latest version to
> > meta-oe, once I'm back from ELC.  

That would be really awesome. Thank you!

Adrian

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

[Denys Dmytriyenko]

Resending again, since my previous reply got truncated when sending to the 
list:

On Thu, Apr 07, 2016 at 11:50:10AM -0700, Denys Dmytriyenko wrote:
> > 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?
> 
> Well, it is currently NOT in the distribution layer, but rather in the
> application layer... But I guess I'm fine submitting the latest version to
> meta-oe, once I'm back from ELC.

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

[Denys Dmytriyenko]