All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-zephyr][PATCH] zephyr: use TCLIBC=newlib directly
@ 2020-10-20 12:42 Ross Burton
  2020-10-20 12:44 ` [yocto] " Ross Burton
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Ross Burton @ 2020-10-20 12:42 UTC (permalink / raw)
  To: yocto; +Cc: naveen.kumar.saini

Instead of setting TCLIBC=baremetal and then adding newlib in various places,
just set TCLIBC=newlib directly.

This also means we can use the standard DEPENDS instead of reinventing them.

Signed-off-by: Ross Burton <ross.burton@arm.com>
---
 conf/distro/zephyr.conf                               | 7 +------
 recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc | 3 ---
 2 files changed, 1 insertion(+), 9 deletions(-)

diff --git a/conf/distro/zephyr.conf b/conf/distro/zephyr.conf
index 44448af..a98da32 100644
--- a/conf/distro/zephyr.conf
+++ b/conf/distro/zephyr.conf
@@ -5,16 +5,11 @@ DISTRO_VERSION = "1.0"
 
 TARGET_VENDOR = "-yocto"
 
-TCLIBC = "baremetal"
-TCLIBCAPPEND = ""
+TCLIBC = "newlib"
 
 TEST_TARGET = "QemuTargetZephyr"
 TEST_SUITES = "zephyr"
 
-PREFERRED_PROVIDER_virtual/libc = "newlib"
-PREFERRED_PROVIDER_virtual/libiconv = "newlib"
-
-TOOLCHAIN_TARGET_TASK += " newlib"
 INHERIT += "siteinfo-zephyr"
 
 INHERIT += "uninative"
diff --git a/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc b/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
index 7fa4b25..b947472 100644
--- a/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
+++ b/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
@@ -19,9 +19,6 @@ EXTRA_OECMAKE_append_arm = " -DZEPHYR_MODULES=${S}/modules/cmsis"
 export ZEPHYR_BASE="${S}"
 
 
-# We always need a toolchain to cross-compile.
-INHIBIT_DEFAULT_DEPS = "1"
-DEPENDS += "gcc-cross-${TARGET_ARCH} libgcc ${TOOLCHAIN_TARGET_TASK} gperf-native"
 DEPENDS += " python3-pyelftools-native python3-pyyaml-native python3-pykwalify-native"
 CROSS_COMPILE = "${STAGING_BINDIR_TOOLCHAIN}/${TARGET_PREFIX}"
 
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [yocto] [meta-zephyr][PATCH] zephyr: use TCLIBC=newlib directly
  2020-10-20 12:42 [meta-zephyr][PATCH] zephyr: use TCLIBC=newlib directly Ross Burton
@ 2020-10-20 12:44 ` Ross Burton
  2020-10-20 17:06 ` Khem Raj
  2020-10-21  5:35 ` Naveen Saini
  2 siblings, 0 replies; 4+ messages in thread
From: Ross Burton @ 2020-10-20 12:44 UTC (permalink / raw)
  Cc: yocto, naveen.kumar.saini

Please do review this one carefully, it continues to build for me and
solves a build failure when using meta-zephyr but not DISTRO=zephyr
(instead, a custom DISTRO and a MACHINE that sets TCLIBC=newlib) but
obviously it is rooting around the build depends and I didn't test the
SDK pieces.

Ross

On Tue, 20 Oct 2020 at 13:43, Ross Burton via lists.yoctoproject.org
<ross=burtonini.com@lists.yoctoproject.org> wrote:
>
> Instead of setting TCLIBC=baremetal and then adding newlib in various places,
> just set TCLIBC=newlib directly.
>
> This also means we can use the standard DEPENDS instead of reinventing them.
>
> Signed-off-by: Ross Burton <ross.burton@arm.com>
> ---
>  conf/distro/zephyr.conf                               | 7 +------
>  recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc | 3 ---
>  2 files changed, 1 insertion(+), 9 deletions(-)
>
> diff --git a/conf/distro/zephyr.conf b/conf/distro/zephyr.conf
> index 44448af..a98da32 100644
> --- a/conf/distro/zephyr.conf
> +++ b/conf/distro/zephyr.conf
> @@ -5,16 +5,11 @@ DISTRO_VERSION = "1.0"
>
>  TARGET_VENDOR = "-yocto"
>
> -TCLIBC = "baremetal"
> -TCLIBCAPPEND = ""
> +TCLIBC = "newlib"
>
>  TEST_TARGET = "QemuTargetZephyr"
>  TEST_SUITES = "zephyr"
>
> -PREFERRED_PROVIDER_virtual/libc = "newlib"
> -PREFERRED_PROVIDER_virtual/libiconv = "newlib"
> -
> -TOOLCHAIN_TARGET_TASK += " newlib"
>  INHERIT += "siteinfo-zephyr"
>
>  INHERIT += "uninative"
> diff --git a/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc b/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
> index 7fa4b25..b947472 100644
> --- a/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
> +++ b/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
> @@ -19,9 +19,6 @@ EXTRA_OECMAKE_append_arm = " -DZEPHYR_MODULES=${S}/modules/cmsis"
>  export ZEPHYR_BASE="${S}"
>
>
> -# We always need a toolchain to cross-compile.
> -INHIBIT_DEFAULT_DEPS = "1"
> -DEPENDS += "gcc-cross-${TARGET_ARCH} libgcc ${TOOLCHAIN_TARGET_TASK} gperf-native"
>  DEPENDS += " python3-pyelftools-native python3-pyyaml-native python3-pykwalify-native"
>  CROSS_COMPILE = "${STAGING_BINDIR_TOOLCHAIN}/${TARGET_PREFIX}"
>
> --
> 2.25.1
>
>
> 
>

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [yocto] [meta-zephyr][PATCH] zephyr: use TCLIBC=newlib directly
  2020-10-20 12:42 [meta-zephyr][PATCH] zephyr: use TCLIBC=newlib directly Ross Burton
  2020-10-20 12:44 ` [yocto] " Ross Burton
@ 2020-10-20 17:06 ` Khem Raj
  2020-10-21  5:35 ` Naveen Saini
  2 siblings, 0 replies; 4+ messages in thread
From: Khem Raj @ 2020-10-20 17:06 UTC (permalink / raw)
  To: Ross Burton, yocto; +Cc: naveen.kumar.saini



On 10/20/20 5:42 AM, Ross Burton wrote:
> Instead of setting TCLIBC=baremetal and then adding newlib in various places,
> just set TCLIBC=newlib directly.
> 
> This also means we can use the standard DEPENDS instead of reinventing them.
> 

I think this will also link the app with newlib now which previously was 
using baremetal.

> Signed-off-by: Ross Burton <ross.burton@arm.com>
> ---
>   conf/distro/zephyr.conf                               | 7 +------
>   recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc | 3 ---
>   2 files changed, 1 insertion(+), 9 deletions(-)
> 
> diff --git a/conf/distro/zephyr.conf b/conf/distro/zephyr.conf
> index 44448af..a98da32 100644
> --- a/conf/distro/zephyr.conf
> +++ b/conf/distro/zephyr.conf
> @@ -5,16 +5,11 @@ DISTRO_VERSION = "1.0"
>   
>   TARGET_VENDOR = "-yocto"
>   
> -TCLIBC = "baremetal"
> -TCLIBCAPPEND = ""
> +TCLIBC = "newlib"
>   
>   TEST_TARGET = "QemuTargetZephyr"
>   TEST_SUITES = "zephyr"
>   
> -PREFERRED_PROVIDER_virtual/libc = "newlib"
> -PREFERRED_PROVIDER_virtual/libiconv = "newlib"
> -
> -TOOLCHAIN_TARGET_TASK += " newlib"
>   INHERIT += "siteinfo-zephyr"
>   
>   INHERIT += "uninative"
> diff --git a/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc b/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
> index 7fa4b25..b947472 100644
> --- a/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
> +++ b/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
> @@ -19,9 +19,6 @@ EXTRA_OECMAKE_append_arm = " -DZEPHYR_MODULES=${S}/modules/cmsis"
>   export ZEPHYR_BASE="${S}"
>   
>   
> -# We always need a toolchain to cross-compile.
> -INHIBIT_DEFAULT_DEPS = "1"
> -DEPENDS += "gcc-cross-${TARGET_ARCH} libgcc ${TOOLCHAIN_TARGET_TASK} gperf-native"
>   DEPENDS += " python3-pyelftools-native python3-pyyaml-native python3-pykwalify-native"
>   CROSS_COMPILE = "${STAGING_BINDIR_TOOLCHAIN}/${TARGET_PREFIX}"
>   
> 
> 
> 
> 
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [meta-zephyr][PATCH] zephyr: use TCLIBC=newlib directly
  2020-10-20 12:42 [meta-zephyr][PATCH] zephyr: use TCLIBC=newlib directly Ross Burton
  2020-10-20 12:44 ` [yocto] " Ross Burton
  2020-10-20 17:06 ` Khem Raj
@ 2020-10-21  5:35 ` Naveen Saini
  2 siblings, 0 replies; 4+ messages in thread
From: Naveen Saini @ 2020-10-21  5:35 UTC (permalink / raw)
  To: Ross Burton, yocto

Few kernel testcases have dependencies on gperf-native, so we should keep it in DEPENDS.

Regards,
Naveen

-----Original Message-----
From: Ross Burton <ross.burton@arm.com> 
Sent: Tuesday, October 20, 2020 8:43 PM
To: yocto@lists.yoctoproject.org
Cc: Saini, Naveen Kumar <naveen.kumar.saini@intel.com>
Subject: [meta-zephyr][PATCH] zephyr: use TCLIBC=newlib directly

Instead of setting TCLIBC=baremetal and then adding newlib in various places, just set TCLIBC=newlib directly.

This also means we can use the standard DEPENDS instead of reinventing them.

Signed-off-by: Ross Burton <ross.burton@arm.com>
---
 conf/distro/zephyr.conf                               | 7 +------
 recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc | 3 ---
 2 files changed, 1 insertion(+), 9 deletions(-)

diff --git a/conf/distro/zephyr.conf b/conf/distro/zephyr.conf index 44448af..a98da32 100644
--- a/conf/distro/zephyr.conf
+++ b/conf/distro/zephyr.conf
@@ -5,16 +5,11 @@ DISTRO_VERSION = "1.0"
 
 TARGET_VENDOR = "-yocto"
 
-TCLIBC = "baremetal"
-TCLIBCAPPEND = ""
+TCLIBC = "newlib"
 
 TEST_TARGET = "QemuTargetZephyr"
 TEST_SUITES = "zephyr"
 
-PREFERRED_PROVIDER_virtual/libc = "newlib"
-PREFERRED_PROVIDER_virtual/libiconv = "newlib"
-
-TOOLCHAIN_TARGET_TASK += " newlib"
 INHERIT += "siteinfo-zephyr"
 
 INHERIT += "uninative"
diff --git a/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc b/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
index 7fa4b25..b947472 100644
--- a/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
+++ b/recipes-kernel/zephyr-kernel/zephyr-kernel-common.inc
@@ -19,9 +19,6 @@ EXTRA_OECMAKE_append_arm = " -DZEPHYR_MODULES=${S}/modules/cmsis"
 export ZEPHYR_BASE="${S}"
 
 
-# We always need a toolchain to cross-compile.
-INHIBIT_DEFAULT_DEPS = "1"
-DEPENDS += "gcc-cross-${TARGET_ARCH} libgcc ${TOOLCHAIN_TARGET_TASK} gperf-native"
 DEPENDS += " python3-pyelftools-native python3-pyyaml-native python3-pykwalify-native"
 CROSS_COMPILE = "${STAGING_BINDIR_TOOLCHAIN}/${TARGET_PREFIX}"
 
--
2.25.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-10-21  5:35 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-20 12:42 [meta-zephyr][PATCH] zephyr: use TCLIBC=newlib directly Ross Burton
2020-10-20 12:44 ` [yocto] " Ross Burton
2020-10-20 17:06 ` Khem Raj
2020-10-21  5:35 ` Naveen Saini

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.